mbox series

[net-next,v5,00/11] bnxt_en: Add NPAR 1.2 and TPH support

Message ID 20250213011240.1640031-1-michael.chan@broadcom.com (mailing list archive)
Headers show
Series bnxt_en: Add NPAR 1.2 and TPH support | expand

Message

Michael Chan Feb. 13, 2025, 1:12 a.m. UTC
The first patch adds NPAR 1.2 support.  Patches 2 to 11 add TPH
(TLP Processing Hints) support.  These TPH driver patches are new
revisions originally posted as part of the TPH PCI patch series.
Additional driver refactoring has been done so that we can free
and allocate RX completion ring and the TX rings if the channel is
a combined channel.  We also add napi_disable() and napi_enable()
during queue_stop() and queue_start() respectively, and reset for
error handling in queue_start().

v5:
Split bnxt_hwrm_tx_ring_free() refactoring from patch #9 into a new patch
Remove reset counters increment
Add comments that ring alloc should not fail in this code path
Add comments about napi_disable()

v4:
Fix NPAR typo in patch #1 and improve the description of NPAR

https://lore.kernel.org/netdev/20250208202916.1391614-1-michael.chan@broadcom.com/

v3:
Fix build bot warning in patch #9
Add MODULE_IMPORT_NS("NETDEV_INTERNAL") to patch #10
Optimize ring operations when TPH is not enabled in patch #8 and #9

https://lore.kernel.org/netdev/20250204004609.1107078-1-michael.chan@broadcom.com/

v2:
Major change is error handling in patch #9.

https://lore.kernel.org/netdev/20250116192343.34535-1-michael.chan@broadcom.com/

v1: 
https://lore.kernel.org/netdev/20250113063927.4017173-1-michael.chan@broadcom.com/

Discussion about adding napi_disable()/napi_enable():

https://lore.kernel.org/netdev/5336d624-8d8b-40a6-b732-b020e4a119a2@davidwei.uk/#t

Previous driver series fixing rtnl_lock and empty release function:

https://lore.kernel.org/netdev/20241115200412.1340286-1-wei.huang2@amd.com/

v5 of the PCI series using netdev_rx_queue_restart():

https://lore.kernel.org/netdev/20240916205103.3882081-5-wei.huang2@amd.com/

v1 of the PCI series using open/close:

https://lore.kernel.org/netdev/20240509162741.1937586-9-wei.huang2@amd.com/

Manoj Panicker (1):
  bnxt_en: Add TPH support in BNXT driver

Michael Chan (6):
  bnxt_en: Set NPAR 1.2 support when registering with firmware
  bnxt_en: Refactor completion ring allocation logic for P5_PLUS chips
  bnxt_en: Refactor TX ring allocation logic
  bnxt_en: Refactor RX/RX AGG ring parameters setup for P5_PLUS
  bnxt_en: Pass NQ ID to the FW when allocating RX/RX AGG rings
  bnxt_en: Refactor TX ring free logic

Somnath Kotur (4):
  bnxt_en: Refactor completion ring free routine
  bnxt_en: Refactor bnxt_free_tx_rings() to free per TX ring
  bnxt_en: Reallocate RX completion ring for TPH support
  bnxt_en: Extend queue stop/start for TX rings

 drivers/net/ethernet/broadcom/bnxt/bnxt.c | 554 ++++++++++++++++------
 drivers/net/ethernet/broadcom/bnxt/bnxt.h |   8 +
 2 files changed, 416 insertions(+), 146 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Feb. 15, 2025, 4 a.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 12 Feb 2025 17:12:28 -0800 you wrote:
> The first patch adds NPAR 1.2 support.  Patches 2 to 11 add TPH
> (TLP Processing Hints) support.  These TPH driver patches are new
> revisions originally posted as part of the TPH PCI patch series.
> Additional driver refactoring has been done so that we can free
> and allocate RX completion ring and the TX rings if the channel is
> a combined channel.  We also add napi_disable() and napi_enable()
> during queue_stop() and queue_start() respectively, and reset for
> error handling in queue_start().
> 
> [...]

Here is the summary with links:
  - [net-next,v5,01/11] bnxt_en: Set NPAR 1.2 support when registering with firmware
    https://git.kernel.org/netdev/net-next/c/ebdf7fe488c5
  - [net-next,v5,02/11] bnxt_en: Refactor completion ring allocation logic for P5_PLUS chips
    https://git.kernel.org/netdev/net-next/c/0fed290525d5
  - [net-next,v5,03/11] bnxt_en: Refactor TX ring allocation logic
    https://git.kernel.org/netdev/net-next/c/e6ec50485659
  - [net-next,v5,04/11] bnxt_en: Refactor completion ring free routine
    https://git.kernel.org/netdev/net-next/c/f33a508c23a4
  - [net-next,v5,05/11] bnxt_en: Refactor bnxt_free_tx_rings() to free per TX ring
    https://git.kernel.org/netdev/net-next/c/09cc58d59441
  - [net-next,v5,06/11] bnxt_en: Refactor RX/RX AGG ring parameters setup for P5_PLUS
    https://git.kernel.org/netdev/net-next/c/e1714de53218
  - [net-next,v5,07/11] bnxt_en: Pass NQ ID to the FW when allocating RX/RX AGG rings
    https://git.kernel.org/netdev/net-next/c/4c8e612c9a36
  - [net-next,v5,08/11] bnxt_en: Reallocate RX completion ring for TPH support
    https://git.kernel.org/netdev/net-next/c/6b6bf60fc95f
  - [net-next,v5,09/11] bnxt_en: Refactor TX ring free logic
    https://git.kernel.org/netdev/net-next/c/c8a0f7652d61
  - [net-next,v5,10/11] bnxt_en: Extend queue stop/start for TX rings
    https://git.kernel.org/netdev/net-next/c/fe96d717d38e
  - [net-next,v5,11/11] bnxt_en: Add TPH support in BNXT driver
    https://git.kernel.org/netdev/net-next/c/c214410c47d6

You are awesome, thank you!