[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