Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-21 Thread Fam Zheng
On Tue, 05/21 08:39, Richard W.M. Jones wrote: > On Tue, May 21, 2013 at 09:54:15AM +0800, Fam Zheng wrote: > > On Mon, 05/20 09:49, Richard W.M. Jones wrote: > > > On Mon, May 20, 2013 at 09:41:06AM +0100, Richard W.M. Jones wrote: > > > > On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote:

Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-21 Thread Richard W.M. Jones
On Tue, May 21, 2013 at 09:54:15AM +0800, Fam Zheng wrote: > On Mon, 05/20 09:49, Richard W.M. Jones wrote: > > On Mon, May 20, 2013 at 09:41:06AM +0100, Richard W.M. Jones wrote: > > > On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote: > > > > CURL library API has changed, the current curl

Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-20 Thread Fam Zheng
On Mon, 05/20 09:49, Richard W.M. Jones wrote: > On Mon, May 20, 2013 at 09:41:06AM +0100, Richard W.M. Jones wrote: > > On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote: > > > CURL library API has changed, the current curl driver is not working. > > > This patch rewrites the use of API as

Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-20 Thread Richard W.M. Jones
On Mon, May 20, 2013 at 09:41:06AM +0100, Richard W.M. Jones wrote: > On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote: > > CURL library API has changed, the current curl driver is not working. > > This patch rewrites the use of API as well as the structure of internal > > states. > > I

Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-20 Thread Richard W.M. Jones
On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote: > CURL library API has changed, the current curl driver is not working. > This patch rewrites the use of API as well as the structure of internal > states. I tried this, but it segfaults: Program terminated with signal 11, Segmentation f

[Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-20 Thread Fam Zheng
CURL library API has changed, the current curl driver is not working. This patch rewrites the use of API as well as the structure of internal states. BDRVCURLState holds the pointer to curl multi interface (man 3 libcurl-multi), and 4 lists for internal states: - CURLState holds state for libcur