Re: [Qemu-devel] [PATCHv2 4/7] spapr_pci: Allow EEH on spapr-pci-host-bridge

[Alexey Kardashevskiy <aik@ozlabs.ru>]

On 02/29/2016 06:06 PM, David Gibson wrote:
> Now that the EEH code is independent of the special
> spapr-vfio-pci-host-bridge device, we can allow it on all spapr PCI
> host bridges instead.  We do this by changing spapr_phb_eeh_available()
> to be based on the vfio_eeh_as_ok() call instead of the host bridge class.
>
> Because the value of vfio_eeh_as_ok() can change with devices being
> hotplugged or unplugged, this can potentially lead to some strange edge
> cases where the guest starts using EEH, then it starts failing because
> of a change in status.
>
> However, it's not really any worse than the current situation.  Cases that
> would have worked previously will still work (i.e. VFIO devices from at
> most one VFIO IOMMU group per vPHB), it's just that it's no longer
> necessary to use spapr-vfio-pci-host-bridge with the groupid pre-specified.
>
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

Reviewed-by: Alexey Kardashevskiy <aik@ozlabs.ru>




-- 
Alexey