Re: [PATCH net v3 0/3] bnxt_en: PCIe FLR/AER fixes

[Jacob Keller <jacob.e.keller@intel.com>]

On 10/5/2026 1:42 PM, Michael Chan wrote:
> This patchset is split from the earlier 9-patch series to limit the
> fixes to the PCIe FLR/AER issues only (patches 7 to 9).  The other
> fixes (patches 1 to 6) will be sent to net-next separately.
> 
> The 1st patch is unchanged since v1.  It refactors a bnxt_clear_bars()
> helper needed by patch 2 and 3.  Patch 2 fixes an issue during driver
> init.  in the kdump kernel on a Dell system with a smart PCIe switch
> upstream from the NIC.  Patch 3 fixes an issue in the AER code path for
> non-fatal errors found by a partner.
> 
> These patches have been regression tested on Broadcom 5750X and 5760X
> NICs running 237.x.x.x production FW.
> 
> v3:
> Only include the 3 PCIe FLR/AER fixes.
> Updated to address some more Sashiko comments.
> 

The unconditional sleeps are a bit annoying, but given what you
described (and that they preexist this series and are simply expanded to
cover more cases) it makes sense.

Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>

> v2:
> Updated to address some Sashiko comments.
> 
> https://lore.kernel.org/netdev/20260928041712.3467803-1-michael.chan@broadcom.com/
> 
> v1:
> https://lore.kernel.org/netdev/20260831024342.2161156-1-michael.chan@broadcom.com/
> 
> Michael Chan (2):
>   bnxt_en: Add bnxt_clear_bars() helper
>   bnxt_en: Fix driver init in kdump kernel
> 
> Pavan Chebbi (1):
>   bnxt_en: Re-write the BARs following any type of PCIe errors
> 
>  drivers/net/ethernet/broadcom/bnxt/bnxt.c | 164 ++++++++++++++--------
>  drivers/net/ethernet/broadcom/bnxt/bnxt.h |   1 -
>  2 files changed, 108 insertions(+), 57 deletions(-)
>