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
