From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49236) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dAdaK-0007uV-JP for qemu-devel@nongnu.org; Tue, 16 May 2017 10:36:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dAdaG-0005dr-4V for qemu-devel@nongnu.org; Tue, 16 May 2017 10:36:24 -0400 Date: Tue, 16 May 2017 10:35:59 -0400 From: Jeff Cody Message-ID: <20170516143559.GJ19824@localhost.localdomain> References: <20170515100059.15795-1-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170515100059.15795-1-pbonzini@redhat.com> Subject: Re: [Qemu-devel] [PATCH v3 0/7] curl: locking cleanups/fixes, coroutine conversion, remove aio_poll List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: qemu-devel@nongnu.org, rjones@redhat.com, qemu-block@nongnu.org On Mon, May 15, 2017 at 12:00:52PM +0200, Paolo Bonzini wrote: > Compared to v2, this silences checkpatch and correctly destroy the mutex on > exiting from curl_open with an error. > > Paolo > > Paolo Bonzini (7): > curl: strengthen assertion in curl_clean_state > curl: never invoke callbacks with s->mutex held > curl: avoid recursive locking of BDRVCURLState mutex > curl: split curl_find_state/curl_init_state > curl: convert CURLAIOCB to byte values > curl: convert readv to coroutines > curl: do not do aio_poll when waiting for a free CURLState > > block/curl.c | 217 +++++++++++++++++++++++++++++++++-------------------------- > 1 file changed, 121 insertions(+), 96 deletions(-) > > -- > 2.12.2 > Thanks, I fixed up the coding styling issues in patch 1 and 4. Applied to my block branch: git://github.com/codyprime/qemu-kvm-jtc block -Jeff