Re: [Qemu-devel] [PATCH v5 08/11] curl: use list to store CURLState

2013-05-23 Thread Fam Zheng
On Thu, 05/23 16:32, Stefan Hajnoczi wrote: > On Thu, May 23, 2013 at 11:38:06AM +0800, Fam Zheng wrote: > > @@ -660,9 +651,13 @@ static BlockDriverAIOCB > > *curl_aio_readv(BlockDriverState *bs, > > static void curl_close(BlockDriverState *bs) > > { > > BDRVCURLState *s = bs->opaque; > > -

Re: [Qemu-devel] [PATCH v5 08/11] curl: use list to store CURLState

2013-05-23 Thread Stefan Hajnoczi
On Thu, May 23, 2013 at 11:38:06AM +0800, Fam Zheng wrote: > @@ -660,9 +651,13 @@ static BlockDriverAIOCB *curl_aio_readv(BlockDriverState > *bs, > static void curl_close(BlockDriverState *bs) > { > BDRVCURLState *s = bs->opaque; > -int i; > > DPRINTF("CURL: Close\n"); > +if

[Qemu-devel] [PATCH v5 08/11] curl: use list to store CURLState

2013-05-22 Thread Fam Zheng
Make it consistent to other structures to use QLIST to store CURLState. It also simplifies initialization and releasing of data. Signed-off-by: Fam Zheng --- block/curl.c | 96 1 file changed, 51 insertions(+), 45 deletions(-) diff --