Re: [Qemu-devel] [PATCH v2 00/11] curl: fix curl read

2013-05-15 Thread Stefan Hajnoczi
On Wed, May 15, 2013 at 11:46:36AM +0800, Fam Zheng wrote: On Tue, 05/14 10:22, Stefan Hajnoczi wrote: On Tue, May 14, 2013 at 10:26:19AM +0800, Fam Zheng wrote: - CURLDataCache holds the user data read from libcurl, it is in a list ordered by access, the used cache is moved to list

Re: [Qemu-devel] [PATCH v2 00/11] curl: fix curl read

2013-05-14 Thread Stefan Hajnoczi
On Tue, May 14, 2013 at 10:26:19AM +0800, Fam Zheng wrote: - CURLDataCache holds the user data read from libcurl, it is in a list ordered by access, the used cache is moved to list head on access, so the tail element is freed first. BDRVCURLState.cache_quota is the threshold to start

Re: [Qemu-devel] [PATCH v2 00/11] curl: fix curl read

2013-05-14 Thread Fam Zheng
On Tue, 05/14 10:22, Stefan Hajnoczi wrote: On Tue, May 14, 2013 at 10:26:19AM +0800, Fam Zheng wrote: - CURLDataCache holds the user data read from libcurl, it is in a list ordered by access, the used cache is moved to list head on access, so the tail element is freed first.

Re: [Qemu-devel] [PATCH v2 00/11] curl: fix curl read

2013-05-14 Thread Fam Zheng
On Tue, 05/14 10:22, Stefan Hajnoczi wrote: On Tue, May 14, 2013 at 10:26:19AM +0800, Fam Zheng wrote: - CURLDataCache holds the user data read from libcurl, it is in a list ordered by access, the used cache is moved to list head on access, so the tail element is freed first.

[Qemu-devel] [PATCH v2 00/11] curl: fix curl read

2013-05-13 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. (It is hard to split this to multiple patches as basically all these changes need to work together.) BDRVCURLState holds the pointer to curl multi