Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-07 Thread Richard W.M. Jones
On Fri, Jun 07, 2013 at 03:47:23PM +0800, Fam Zheng wrote: > On Fri, 06/07 08:27, Richard W.M. Jones wrote: > > On Fri, Jun 07, 2013 at 09:54:42AM +0800, Fam Zheng wrote: > > > The weird thing is it doesn't work for me, I'm sure something is wrong > > > with current upstream, although not totally b

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-07 Thread Fam Zheng
On Fri, 06/07 09:46, Stefan Hajnoczi wrote: > On Thu, Jun 06, 2013 at 12:01:57PM +0100, Richard W.M. Jones wrote: > > On Thu, Jun 06, 2013 at 02:25:46PM +0800, Fam Zheng wrote: > > > v7: > > > 13: Added: > > > curl: change timeout to 30 seconds > > > > I tested this against: > > > > (1) H

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-07 Thread Fam Zheng
On Fri, 06/07 08:27, Richard W.M. Jones wrote: > On Fri, Jun 07, 2013 at 09:54:42AM +0800, Fam Zheng wrote: > > The weird thing is it doesn't work for me, I'm sure something is wrong > > with current upstream, although not totally broken. > > > > $./qemu-io http://localhost/vm/arch.raw -c 'read

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-07 Thread Stefan Hajnoczi
On Thu, Jun 06, 2013 at 12:01:57PM +0100, Richard W.M. Jones wrote: > On Thu, Jun 06, 2013 at 02:25:46PM +0800, Fam Zheng wrote: > > v7: > > 13: Added: > > curl: change timeout to 30 seconds > > I tested this against: > > (1) HTTP to Apache server over slow but local wifi. > > (2) HTTP t

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-07 Thread Richard W.M. Jones
On Fri, Jun 07, 2013 at 09:54:42AM +0800, Fam Zheng wrote: > The weird thing is it doesn't work for me, I'm sure something is wrong > with current upstream, although not totally broken. > > $./qemu-io http://localhost/vm/arch.raw -c 'read -v 0 512' > (stuck here forever, no output) This doesn

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-06 Thread Fam Zheng
On Thu, 06/06 12:01, Richard W.M. Jones wrote: > On Thu, Jun 06, 2013 at 02:25:46PM +0800, Fam Zheng wrote: > > v7: > > 13: Added: > > curl: change timeout to 30 seconds > > I tested this against: > > (1) HTTP to Apache server over slow but local wifi. > > (2) HTTP to a remote ISO (on an

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-06 Thread Richard W.M. Jones
On Thu, Jun 06, 2013 at 02:25:46PM +0800, Fam Zheng wrote: > v7: > 13: Added: > curl: change timeout to 30 seconds I tested this against: (1) HTTP to Apache server over slow but local wifi. (2) HTTP to a remote ISO (on another continent). Test (1) is fine. Test (2) gives plenty of I/O

[Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-05 Thread Fam Zheng
CURL library API has changed, the current curl driver is not working with current libcurl. It may or may not have worked with old libcurl, but currently, when testing with apache http URL, it just hangs before fetching any data. The problem is because the mismatch of our code and how libcurl wants