[Intel-wired-lan] [PATCH iwl-net v5] i40e: Restore VF MSI-X state during PCI reset
Romanowski, Rafal
rafal.romanowski at intel.com
Wed Jan 3 09:55:51 UTC 2024
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf Of
> Przemek Kitszel
> Sent: Thursday, December 21, 2023 2:49 PM
> To: Staikov, Andrii <andrii.staikov at intel.com>; intel-wired-lan at lists.osuosl.org
> Cc: Ostrowska, Karen <karen.ostrowska at intel.com>; netdev at vger.kernel.org;
> Drewek, Wojciech <wojciech.drewek at intel.com>; Mateusz Palczewski
> <mateusz.palczewski at intel.com>
> Subject: Re: [Intel-wired-lan] [PATCH iwl-net v5] i40e: Restore VF MSI-X state
> during PCI reset
>
> On 12/21/23 14:27, Andrii Staikov wrote:
> > During a PCI FLR the MSI-X Enable flag in the VF PCI MSI-X capability
> > register will be cleared. This can lead to issues when a VF is
> > assigned to a VM because in these cases the VF driver receives no
> > indication of the PF PCI error/reset and additionally it is incapable
> > of restoring the cleared flag in the hypervisor configuration space
> > without fully reinitializing the driver interrupt functionality.
> >
> > Since the VF driver is unable to easily resolve this condition on its
> > own, restore the VF MSI-X flag during the PF PCI reset handling.
> >
> > Fixes: 19b7960b2da1 ("i40e: implement split PCI error reset handler")
> > Co-developed-by: Karen Ostrowska <karen.ostrowska at intel.com>
> > Signed-off-by: Karen Ostrowska <karen.ostrowska at intel.com>
> > Co-developed-by: Mateusz Palczewski <mateusz.palczewski at intel.com>
> > Signed-off-by: Mateusz Palczewski <mateusz.palczewski at intel.com>
> > Reviewed-by: Wojciech Drewek <wojciech.drewek at intel.com>
> > Reviewed-by: Przemyslaw Kitszel <przemyslaw.kitszel at intel.com>
>
> should be "Przemek", but no need to send next version for that (but pls
> promise it's last time ;))
>
> > Signed-off-by: Andrii Staikov <andrii.staikov at intel.com>
> > ---
> > v1 -> v2: Fix signed-off tags
> > https://patchwork.ozlabs.org/project/intel-wired-lan/patch/20231204131
> > 041.3369693-1-andrii.staikov at intel.com/
> >
> > v2 -> v3: use @vf_dev in pci_get_device() instead of NULL and remove
> > unnecessary call
> > https://patchwork.ozlabs.org/project/intel-wired-lan/patch/20231206125
> > 127.218350-1-andrii.staikov at intel.com/
> >
> > v3 -> v4: wrap the added functionality into the CONFIG_PCI_IOV define
> > as this is VF-related functionality
> > https://patchwork.ozlabs.org/project/intel-wired-lan/patch/20231212122
> > 452.3250691-1-andrii.staikov at intel.com/
> >
> > v4 -> v5: fix RB tags
>
> there was a question in v4 from Jake to "perhaps do it like in ice", but I think
> that for a bug fix it's better to keep this patch as-is (review always welcomed
> through).
>
> > ---
> > drivers/net/ethernet/intel/i40e/i40e_main.c | 3 +++
> > .../ethernet/intel/i40e/i40e_virtchnl_pf.c | 26 +++++++++++++++++++
> > .../ethernet/intel/i40e/i40e_virtchnl_pf.h | 3 +++
> > 3 files changed, 32 insertions(+)
> >
> > diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c
> > b/drivers/net/ethernet/intel/i40e/i40e_main.c
> > index 7bb1f64833eb..bbe2d115fb15 100644
> > --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
> > +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
Tested-by: Rafal Romanowski <rafal.romanowski at intel.com>
More information about the Intel-wired-lan
mailing list