From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55830) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cYNZC-0007Lm-7U for qemu-devel@nongnu.org; Mon, 30 Jan 2017 20:49:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cYNZB-0003hq-DP for qemu-devel@nongnu.org; Mon, 30 Jan 2017 20:49:06 -0500 Sender: Paolo Bonzini References: <20161129113245.32724-1-pbonzini@redhat.com> <3f948687-3169-c304-3ef8-31347f08857f@redhat.com> <20170104040728.GA30425@localhost.localdomain> <20170104164752.GC30425@localhost.localdomain> From: Paolo Bonzini Message-ID: Date: Mon, 30 Jan 2017 19:48:56 -0600 MIME-Version: 1.0 In-Reply-To: <20170104164752.GC30425@localhost.localdomain> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit 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: Jeff Cody Cc: Hitoshi Mitake , Liu Yuan , qemu-devel@nongnu.org, qemu-block@nongnu.org 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? Paolo