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

[Paolo Bonzini]

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