Re: [PATCH v3 0/3] AER-KVM: Error containment of VFIO devicesassigned to KVM guests

From: Alex Williamson
Date: Mon Feb 04 2013 - 12:02:19 EST


On Sun, 2013-02-03 at 14:10 +0000, Pandarathil, Vijaymohan R wrote:
> Add support for error containment when a VFIO device assigned to a KVM
> guest encounters an error. This is for PCIe devices/drivers that support AER
> functionality. When the host OS is notified of an error in a device either
> through the firmware first approach or through an interrupt handled by the AER
> root port driver, the error handler registered by the vfio-pci driver gets
> invoked. The qemu process is signaled through an eventfd registered per
> VFIO device by the qemu process. In the eventfd handler, qemu decides on
> what action to take. In this implementation, guest is brought down to
> contain the error.
>
>
> v3:
> - Removed PCI_AER* flags from device info ioctl.
> - Incorporated feedback

Hi Vijay,

It's getting much closer, just a few comments in each patch. As Gleb
points out, please try to use git send-mail (or even stg mail) so that
threading is maintained for these. A side effect will be that you can't
send kernel & qemu patches in the same series, but that's not
necessarily a bad thing. Thanks,

Alex

> v2:
> - Rebased to latest upstream stable bits
> - Changed the new ioctl to be part of VFIO_SET_IRQs ioctl
> - Added a new patch to get/put reference to a vfio device from struct device
> - Incorporated all other feedback.
>
> ---
>
> Vijay Mohan Pandarathil(3):
>
> [PATCH 1/3] VFIO: Wrapper for getting reference to vfio_device from device
> [PATCH 2/3] VFIO-AER: Vfio-pci driver changes for supporting AER
> [PATCH 3/3] QEMU-AER: Qemu changes to support AER for VFIO-PCI devices
>
> Kernel files changed
>
> drivers/vfio/vfio.c | 41 ++++++++++++++++++++++++++++++++---------
> include/linux/vfio.h | 3 +++
> 2 files changed, 35 insertions(+), 9 deletions(-)
>
> drivers/vfio/pci/vfio_pci.c | 43 ++++++++++++++++++++++++++++++++++++-
> drivers/vfio/pci/vfio_pci_intrs.c | 30 ++++++++++++++++++++++++++
> drivers/vfio/pci/vfio_pci_private.h | 1 +
> include/uapi/linux/vfio.h | 1 +
> 4 files changed, 74 insertions(+), 1 deletion(-)
>
> Qemu files changed
>
> hw/vfio_pci.c | 105 +++++++++++++++++++++++++++++++++++++++++++++
> linux-headers/linux/vfio.h | 1 +
> 2 files changed, 106 insertions(+)



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/