[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v4 4/9] aer: impove pcie_aer_init to support vfio
From: |
Alex Williamson |
Subject: |
Re: [Qemu-devel] [RFC v4 4/9] aer: impove pcie_aer_init to support vfio device |
Date: |
Mon, 09 Mar 2015 14:29:21 -0600 |
On Mon, 2015-03-02 at 15:16 +0800, Chen Fan wrote:
> extend pcie_aer_init arguments to adjust vfio device.
Some discussion of why vfio wants this would be useful.
> Signed-off-by: Chen Fan <address@hidden>
> ---
> hw/pci-bridge/ioh3420.c | 3 ++-
> hw/pci-bridge/xio3130_downstream.c | 3 ++-
> hw/pci-bridge/xio3130_upstream.c | 3 ++-
> hw/pci/pcie_aer.c | 7 ++++---
> include/hw/pci/pcie_aer.h | 4 +++-
> 5 files changed, 13 insertions(+), 7 deletions(-)
>
> diff --git a/hw/pci-bridge/ioh3420.c b/hw/pci-bridge/ioh3420.c
> index cce2fdd..354574f 100644
> --- a/hw/pci-bridge/ioh3420.c
> +++ b/hw/pci-bridge/ioh3420.c
> @@ -129,7 +129,8 @@ static int ioh3420_initfn(PCIDevice *d)
> goto err_pcie_cap;
> }
> pcie_cap_root_init(d);
> - rc = pcie_aer_init(d, IOH_EP_AER_OFFSET);
> + rc = pcie_aer_init(d, PCI_ERR_VER,
> + IOH_EP_AER_OFFSET, PCI_ERR_SIZEOF);
> if (rc < 0) {
> goto err;
> }
> diff --git a/hw/pci-bridge/xio3130_downstream.c
> b/hw/pci-bridge/xio3130_downstream.c
> index b3a6479..407f75f 100644
> --- a/hw/pci-bridge/xio3130_downstream.c
> +++ b/hw/pci-bridge/xio3130_downstream.c
> @@ -92,7 +92,8 @@ static int xio3130_downstream_initfn(PCIDevice *d)
> goto err_pcie_cap;
> }
> pcie_cap_arifwd_init(d);
> - rc = pcie_aer_init(d, XIO3130_AER_OFFSET);
> + rc = pcie_aer_init(d, PCI_ERR_VER,
> + XIO3130_AER_OFFSET, PCI_ERR_SIZEOF);
> if (rc < 0) {
> goto err;
> }
> diff --git a/hw/pci-bridge/xio3130_upstream.c
> b/hw/pci-bridge/xio3130_upstream.c
> index eada582..52b130f 100644
> --- a/hw/pci-bridge/xio3130_upstream.c
> +++ b/hw/pci-bridge/xio3130_upstream.c
> @@ -81,7 +81,8 @@ static int xio3130_upstream_initfn(PCIDevice *d)
> }
> pcie_cap_flr_init(d);
> pcie_cap_deverr_init(d);
> - rc = pcie_aer_init(d, XIO3130_AER_OFFSET);
> + rc = pcie_aer_init(d, PCI_ERR_VER,
> + XIO3130_AER_OFFSET, PCI_ERR_SIZEOF);
> if (rc < 0) {
> goto err;
> }
> diff --git a/hw/pci/pcie_aer.c b/hw/pci/pcie_aer.c
> index ece1487..a76a943 100644
> --- a/hw/pci/pcie_aer.c
> +++ b/hw/pci/pcie_aer.c
> @@ -94,12 +94,13 @@ static void aer_log_clear_all_err(PCIEAERLog *aer_log)
> aer_log->log_num = 0;
> }
>
> -int pcie_aer_init(PCIDevice *dev, uint16_t offset)
> +int pcie_aer_init(PCIDevice *dev, uint8_t cap_ver,
> + uint16_t offset, uint16_t size)
> {
> PCIExpressDevice *exp;
>
> - pcie_add_capability(dev, PCI_EXT_CAP_ID_ERR, PCI_ERR_VER,
> - offset, PCI_ERR_SIZEOF);
> + pcie_add_capability(dev, PCI_EXT_CAP_ID_ERR,
> + cap_ver, offset, size);
> exp = &dev->exp;
> exp->aer_cap = offset;
>
> diff --git a/include/hw/pci/pcie_aer.h b/include/hw/pci/pcie_aer.h
> index bcac80a..afa074e 100644
> --- a/include/hw/pci/pcie_aer.h
> +++ b/include/hw/pci/pcie_aer.h
> @@ -87,7 +87,9 @@ struct PCIEAERErr {
>
> extern const VMStateDescription vmstate_pcie_aer_log;
>
> -int pcie_aer_init(PCIDevice *dev, uint16_t offset);
> +int pcie_aer_init(PCIDevice *dev, uint8_t cap_ver,
> + uint16_t offset, uint16_t size);
> +
> void pcie_aer_exit(PCIDevice *dev);
> void pcie_aer_write_config(PCIDevice *dev,
> uint32_t addr, uint32_t val, int len);
- [Qemu-devel] [RFC v4 0/9] pass aer error to guest for vfio device, Chen Fan, 2015/03/02
- [Qemu-devel] [RFC v4 1/9] pcie_aer: fix typos in pcie_aer_inject_error comment, Chen Fan, 2015/03/02
- [Qemu-devel] [RFC v4 3/9] vfio: add pcie extanded capability support, Chen Fan, 2015/03/02
- [Qemu-devel] [RFC v4 2/9] aer: fix a wrong init PCI_ERR_COR_STATUS w1cmask type register, Chen Fan, 2015/03/02
- [Qemu-devel] [RFC v4 4/9] aer: impove pcie_aer_init to support vfio device, Chen Fan, 2015/03/02
- Re: [Qemu-devel] [RFC v4 4/9] aer: impove pcie_aer_init to support vfio device,
Alex Williamson <=
- [Qemu-devel] [RFC v4 6/9] vfio: add 'x-aer' option to disable aer capability, Chen Fan, 2015/03/02
- [Qemu-devel] [RFC v4 7/9] pcie_aer: expose pcie_aer_msg() interface, Chen Fan, 2015/03/02
- [Qemu-devel] [RFC v4 5/9] vfio: add aer support for vfio device, Chen Fan, 2015/03/02
- [Qemu-devel] [RFC v4 8/9] vfio-pci: pass the aer error to guest, Chen Fan, 2015/03/02