From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56464) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cYnaF-0004uM-An for qemu-devel@nongnu.org; Wed, 01 Feb 2017 00:35:56 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cYnaE-0006qS-Eu for qemu-devel@nongnu.org; Wed, 01 Feb 2017 00:35:55 -0500 Date: Wed, 1 Feb 2017 00:35:48 -0500 From: Jeff Cody Message-ID: <20170201053548.GA6500@localhost.localdomain> References: <20161129113245.32724-1-pbonzini@redhat.com> <3f948687-3169-c304-3ef8-31347f08857f@redhat.com> <20170104040728.GA30425@localhost.localdomain> <20170104164752.GC30425@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH for-2.9 v3 0/5] Sheepdog cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Hitoshi Mitake , Liu Yuan , qemu-devel@nongnu.org, qemu-block@nongnu.org On Mon, Jan 30, 2017 at 07:48:56PM -0600, Paolo Bonzini wrote: > > > On 04/01/2017 11:47, Jeff Cody wrote: > > On Wed, Jan 04, 2017 at 12:42:53PM +0100, Paolo Bonzini wrote: > >> > >> > >> On 04/01/2017 05:07, Jeff Cody wrote: > >>> On Wed, Dec 21, 2016 at 03:07:07PM +0100, Paolo Bonzini wrote: > >>>> > >>>> > >>>> On 29/11/2016 12:32, Paolo Bonzini wrote: > >>>>> Cleaning up the code and removing duplication makes it simpler to > >>>>> later adapt it for the multiqueue work. > >>>>> > >>>>> Tested against sheepdog 1.0. I also tested taking snapshots and reverting > >>>>> to older snapshots, but the latter only worked with "dog vdi rollback". > >>>>> Neither loadvm nor qemu-img worked for me. > >>>>> > >>>>> Paolo > >>>>> > >>>>> v1->v2: placate patchew > >>>>> v2->v3: rebase > >>>>> > >>>>> Paolo Bonzini (5): > >>>>> sheepdog: remove unused cancellation support > >>>>> sheepdog: reorganize coroutine flow > >>>>> sheepdog: do not use BlockAIOCB > >>>>> sheepdog: simplify inflight_aio_head management > >>>>> sheepdog: reorganize check for overlapping requests > >>>>> > >>>>> block/sheepdog.c | 289 ++++++++++++++++--------------------------------------- > >>>>> 1 file changed, 83 insertions(+), 206 deletions(-) > >>>>> > >>>> > >>>> 2.8 is now out, so ping. > >>>> > >>> > >>> I don't have a functional sheepdog setup at the moment; have you tested > >>> these patches, or should I set up a test rig for them? (I am guessing I > >>> should do the latter; either way, I'll pull the patches in once I or someone > >>> else has tested them). > >> > >> Yes, I have tested them. Fedora's sheepdog package is fubar right now. > >> I've built an updated one and asked for ownership but haven't had any > >> answer yet. > >> > >> Paolo > > > > Great, thanks. > > > > Applied to my block branch: > > > > git://github.com/codyprime/qemu-kvm-jtc.git block > > > > -Jeff > > > > > > Ping? > Sorry, never flushed my pull request queue. Flushed now. Thanks, Jeff