[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] igb: Add Function Level Reset to PF and VF
From: |
Jason Wang |
Subject: |
Re: [PATCH 2/2] igb: Add Function Level Reset to PF and VF |
Date: |
Mon, 23 Oct 2023 11:11:32 +0800 |
On Fri, Oct 20, 2023 at 5:41 PM Cédric Le Goater <clg@redhat.com> wrote:
>
> On 10/20/23 09:40, Cédric Le Goater wrote:
> > On 10/20/23 06:24, Jason Wang wrote:
> >> On Tue, Aug 29, 2023 at 5:06 PM Cédric Le Goater <clg@kaod.org> wrote:
> >>>
> >>> From: Cédric Le Goater <clg@redhat.com>
> >>>
> >>> The Intel 82576EB GbE Controller say that the Physical and Virtual
> >>> Functions support Function Level Reset. Add the capability to each
> >>> device model.
> >>>
> >>
> >> Do we need to do migration compatibility for this?
> >
> > Yes. it does. the config space is now different.
>
> Jason,
>
> To avoid an extra compat property, would it be ok to let the VF peek into
> the PF capabilities to set FLR or not ? Something like below.
I might be wrong, but it looks to me it's still a behaviour change?
Thanks
>
> Thanks,
>
> C.
>
>
> @@ -238,6 +238,12 @@ static const MemoryRegionOps mmio_ops =
> },
> };
>
> +static bool igbvf_check_pf_flr(PCIDevice *dev)
> +{
> + return !!(pci_get_long(dev->config + dev->exp.exp_cap + PCI_EXP_DEVCAP)
> + & PCI_EXP_DEVCAP_FLR);
> +}
> +
> static void igbvf_pci_realize(PCIDevice *dev, Error **errp)
> {
> IgbVfState *s = IGBVF(dev);
> @@ -267,7 +273,9 @@ static void igbvf_pci_realize(PCIDevice
> hw_error("Failed to initialize PCIe capability");
> }
>
> - pcie_cap_flr_init(dev);
> + if (igbvf_check_pf_flr(pcie_sriov_get_pf(dev))) {
> + pcie_cap_flr_init(dev);
> + }
>
> if (pcie_aer_init(dev, 1, 0x100, 0x40, errp) < 0) {
> hw_error("Failed to initialize AER capability");
>
>
> >
> > Thanks,
> >
> > C.
> >
> >
> >>
> >> Thanks
> >>
> >>> Cc: Sriram Yagnaraman <sriram.yagnaraman@est.tech>
> >>> Fixes: 3a977deebe6b ("Intrdocue igb device emulation")
> >>> Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> >>> Tested-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> >>> Signed-off-by: Cédric Le Goater <clg@redhat.com>
> >>> ---
> >>> hw/net/igb.c | 3 +++
> >>> hw/net/igbvf.c | 3 +++
> >>> 2 files changed, 6 insertions(+)
> >>>
> >>> diff --git a/hw/net/igb.c b/hw/net/igb.c
> >>> index e70a66ee038e..b8c170ad9b1a 100644
> >>> --- a/hw/net/igb.c
> >>> +++ b/hw/net/igb.c
> >>> @@ -101,6 +101,7 @@ static void igb_write_config(PCIDevice *dev, uint32_t
> >>> addr,
> >>>
> >>> trace_igb_write_config(addr, val, len);
> >>> pci_default_write_config(dev, addr, val, len);
> >>> + pcie_cap_flr_write_config(dev, addr, val, len);
> >>>
> >>> if (range_covers_byte(addr, len, PCI_COMMAND) &&
> >>> (dev->config[PCI_COMMAND] & PCI_COMMAND_MASTER)) {
> >>> @@ -433,6 +434,8 @@ static void igb_pci_realize(PCIDevice *pci_dev, Error
> >>> **errp)
> >>> }
> >>>
> >>> /* PCIe extended capabilities (in order) */
> >>> + pcie_cap_flr_init(pci_dev);
> >>> +
> >>> if (pcie_aer_init(pci_dev, 1, 0x100, 0x40, errp) < 0) {
> >>> hw_error("Failed to initialize AER capability");
> >>> }
> >>> diff --git a/hw/net/igbvf.c b/hw/net/igbvf.c
> >>> index 07343fa14a89..55e321e4ec20 100644
> >>> --- a/hw/net/igbvf.c
> >>> +++ b/hw/net/igbvf.c
> >>> @@ -204,6 +204,7 @@ static void igbvf_write_config(PCIDevice *dev,
> >>> uint32_t addr, uint32_t val,
> >>> {
> >>> trace_igbvf_write_config(addr, val, len);
> >>> pci_default_write_config(dev, addr, val, len);
> >>> + pcie_cap_flr_write_config(dev, addr, val, len);
> >>> }
> >>>
> >>> static uint64_t igbvf_mmio_read(void *opaque, hwaddr addr, unsigned
> >>> size)
> >>> @@ -266,6 +267,8 @@ static void igbvf_pci_realize(PCIDevice *dev, Error
> >>> **errp)
> >>> hw_error("Failed to initialize PCIe capability");
> >>> }
> >>>
> >>> + pcie_cap_flr_init(dev);
> >>> +
> >>> if (pcie_aer_init(dev, 1, 0x100, 0x40, errp) < 0) {
> >>> hw_error("Failed to initialize AER capability");
> >>> }
> >>> --
> >>> 2.41.0
> >>>
> >>>
> >>
> >
>