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
