[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [qemu-s390x] [PATCH v4 2/2] vfio-ccw: support async command subregio

From: Eric Farman
Subject: Re: [qemu-s390x] [PATCH v4 2/2] vfio-ccw: support async command subregion
Date: Wed, 29 May 2019 09:47:55 -0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1

On 5/29/19 5:48 AM, Cornelia Huck wrote:
> On Tue, 21 May 2019 16:47:45 -0400
> Eric Farman <address@hidden> wrote:
>> On 5/21/19 12:32 PM, Cornelia Huck wrote:
>>> Why mostly? I'm not sure yet whether we handling multiple requests for
>>> passthrough devices correctly yet (virtual should be fine.)
>>> Start vs. (start|halt|clear) is fine, as the code checks whether
>>> something is already pending before poking the kernel interface.
>>> Likewise, halt vs. (start|halt|clear) is fine, as the code checks for
>>> halt or clear and start and halt use different regions. The problematic
>>> one is clear, as that's something that's always supposed to go through.
>>> Probably fine if clear should always "win", but I need to think some
>>> more about that.  
>> I suspect you are right, because of the check on the halt side, and
>> considering that the clear is the biggest recovery action we have.  So
>> this does seem like things are okay.  I'll ponder this overnight and
>> finish my review tomorrow.
> Ok, what's the verdict here? :)

Gah, I left this sit in my draft folder before holiday.  Sorry about that!

> (I'm trying to clean up my pending stuff :)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]