Re: [Xen-devel] [PATCH] xen-pciback: Mark a PCIe device to be hidden on AER error

[Juergen Gross <jgross@xxxxxxxx>]

On 27/07/17 17:41, Boris Ostrovsky wrote:
> On 07/27/2017 11:31 AM, Juergen Gross wrote:
>> On 27/06/17 19:21, Venu Busireddy wrote:
>>> This patch set is part of a set of patchs that together allow containment
>>> of unrecoverable AER errors from PCIe devices assigned to guests in
>>> passthrough mode. The containment is achieved by killing the guest and
>>> hiding the device upon receiving the fatal AER error.
>>>
>>> The Xen patch related to this patch is:
>>>
>>> https://lists.xen.org/archives/html/xen-devel/2017-06/msg03269.html
>>>
>>>
>>> This patch stores in xenstore the <s:b:d.f> of the passed through device
>>> that triggered the AER unrecoverable error. This will allow xen (with a
>>> watcher setup to watch "aerFailedSBDF") to make the device unassignable
>>> until the next reboot or operator intervention using the xen tool stack.
>>>
>>> Note:
>>> When unrecoverable AER errors are detected from the PCIe devices
>>> assigned to guests in passthrough mode, BIOS's bring down the server,
>>> thus bringing down the entire hypervisor. For this patch set to work,
>>> the AER error handling needs to be delegated to the host operating system.
>>>
>>> Signed-off-by: Venu Busireddy <venu.busireddy@xxxxxxxxxx>
>>> Signed-off-by: Elena Ufimtseva <elena.ufimtseva@xxxxxxxxxx>
>> Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
>>
> 
> Isn't this dependent on Xen-side patches? I think we should wait until
> those are accepted (if nothing else, the watch name should be agreed upon).

Right. I wouldn't commit the patch until the Xen side is ready. Just
wanted to give a "go ahead" for the Xen side to avoid a deadlock. :-)


Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel