On 11/18/19 1:13 PM, Cornelia Huck wrote:
> On Fri, 15 Nov 2019 04:34:30 +0100
> Eric Farman <far...@linux.ibm.com> wrote:
>
>> From: Farhan Ali <al...@linux.ibm.com>
>>
>> EIO is returned by vfio-ccw mediated device when the backing
>> host subchannel is not operational anymore. So return cc=3
>> back to the guest, rather than returning a unit check.
>> This way the guest can take appropriate action such as
>> issue an 'stsch'.
>
> Hnm, I'm trying to recall whether that was actually a conscious choice,
> but I can't quite remember... the change does make sense at a glance,
> however.
>
>>
>> Signed-off-by: Farhan Ali <al...@linux.ibm.com>
>
> I would need your s-o-b for that one, though :)
Oops. :)
>
>> ---
>> hw/vfio/ccw.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/hw/vfio/ccw.c b/hw/vfio/ccw.c
>> index 6863f6c69f..0919ddbeb8 100644
>> --- a/hw/vfio/ccw.c
>> +++ b/hw/vfio/ccw.c
>> @@ -114,6 +114,7 @@ again:
>> return IOINST_CC_BUSY;
>> case -ENODEV:
>> case -EACCES:
>> + case -EIO:
>> return IOINST_CC_NOT_OPERATIONAL;
>> case -EFAULT:
>> default:
>