[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read |
Date: |
Fri, 7 Jun 2013 15:47:23 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
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 -v 0 512'
> > (stuck here forever, no output)
>
> This doesn't work for me either.
>
> *However* it only doesn't work if I use the Fedora version of curl,
> which I'm convinced is broken.
I see, it turns out upstream curl works in my case too, thanks.
So it's more of curl broken rather than qemu driver. For the timeout
issue introduced here, I think it's the effect of implementing timer in
the driver, I can't think of a good way to workaround. However the
master code not using timer at all is an undocumented usage of libcurl.
--
Fam
- [Qemu-devel] [PATCH v7 07/13] curl: make use of CURLDataCache., (continued)
- [Qemu-devel] [PATCH v7 07/13] curl: make use of CURLDataCache., Fam Zheng, 2013/06/06
- [Qemu-devel] [PATCH v7 08/13] curl: use list to store CURLState, Fam Zheng, 2013/06/06
- [Qemu-devel] [PATCH v7 09/13] curl: add cache quota., Fam Zheng, 2013/06/06
- [Qemu-devel] [PATCH v7 10/13] curl: introduce ssl_no_cert runtime option., Fam Zheng, 2013/06/06
- [Qemu-devel] [PATCH v7 11/13] block/curl.c: Refuse to open the handle for writes., Fam Zheng, 2013/06/06
- [Qemu-devel] [PATCH v7 12/13] curl: set s->url to NULL after free., Fam Zheng, 2013/06/06
- [Qemu-devel] [PATCH v7 13/13] curl: change timeout to 30 seconds, Fam Zheng, 2013/06/06
- Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read, Richard W.M. Jones, 2013/06/06
- Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read, Stefan Hajnoczi, 2013/06/07