Re: [Qemu-devel] [PATCH for-2.9 v3 0/5] Sheepdog cleanups

[Jeff Cody <jcody@redhat.com>]

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