On Thu, May 23, 2013 at 11:37:59AM +0800, Fam Zheng wrote: > @@ -90,7 +98,16 @@ static int curl_aio_flush(void *opaque); > static int curl_sock_cb(CURL *curl, curl_socket_t fd, int action, > void *s, void *sp) > { > + BDRVCURLState *bs = s;
bs is used for BlockDriverState in block/* code. I find this confusing. Usually the custom state is called 's'. Perhaps just rename the void* arguments s_ and sp_.