patch 'net/bnxt: skip timed Tx pacing setup when unsupported' has been queued to stable release 25.11.3
Kevin Traynor
ktraynor at redhat.com
Tue Jul 28 17:56:20 CEST 2026
Hi,
FYI, your patch has been queued to stable release 25.11.3
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 08/01/26. So please
shout if anyone has objections.
Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.
Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable
This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/bfb87c79c44532fc8cc36c232522bb6c068b2f8b
Thanks.
Kevin
---
>From bfb87c79c44532fc8cc36c232522bb6c068b2f8b Mon Sep 17 00:00:00 2001
From: Dakota Sicher <dakota.sicher at broadcom.com>
Date: Thu, 25 Jun 2026 15:29:51 -0600
Subject: [PATCH] net/bnxt: skip timed Tx pacing setup when unsupported
[ upstream commit edff93f638540783af448de1d40a365b9aee52a6 ]
Driver attempts to configure the TTX pacing TQM rings regardless
of firmware capabilities. When the card does not support timed TX
pacing the device fails to initialize.
Skip the TTX_PACING_TQM_RING backing store type when
BNXT_FW_CAP_TIMED_TX_PACING is not set. The capability is defined
but not set until timed TX pacing support is enabled, so the ring
type is unconditionally skipped on unaffected cards.
Fixes: fe2f715ca580 ("net/bnxt: support backing store v2")
Signed-off-by: Dakota Sicher <dakota.sicher at broadcom.com>
Signed-off-by: Mohammad Shuab Siddique <mohammad-shuab.siddique at broadcom.com>
---
.mailmap | 1 +
drivers/net/bnxt/bnxt.h | 1 +
drivers/net/bnxt/bnxt_hwrm.c | 10 ++++++++--
3 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/.mailmap b/.mailmap
index 62f9f6b3bc..e18f73f3d1 100644
--- a/.mailmap
+++ b/.mailmap
@@ -307,4 +307,5 @@ Cyril Chemparathy <cchemparathy at ezchip.com> <cchemparathy at tilera.com>
Cyril Cressent <cyril.cressent at intel.com>
Dahir Osman <dahir.osman at windriver.com>
+Dakota Sicher <dakota.sicher at broadcom.com>
Damian Milosek <damian.milosek at intel.com>
Damian Nowak <damianx.nowak at intel.com>
diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h
index ed80b29fb3..b125aa29f8 100644
--- a/drivers/net/bnxt/bnxt.h
+++ b/drivers/net/bnxt/bnxt.h
@@ -885,4 +885,5 @@ struct bnxt {
#define BNXT_FW_CAP_RX_ALL_PKT_TS BIT(11)
#define BNXT_FW_CAP_BACKING_STORE_V2 BIT(12)
+#define BNXT_FW_CAP_TIMED_TX_PACING BIT(15)
#define BNXT_FW_CAP_RX_RATE_PROFILE BIT(17)
#define BNXT_FW_BACKING_STORE_V2_EN(bp) \
diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
index bddd55d819..ceebb3531b 100644
--- a/drivers/net/bnxt/bnxt_hwrm.c
+++ b/drivers/net/bnxt/bnxt_hwrm.c
@@ -6823,5 +6823,8 @@ int bnxt_hwrm_func_backing_store_qcaps_v2(struct bnxt *bp)
flags = rte_le_to_cpu_32(resp->flags);
type = rte_le_to_cpu_16(resp->next_valid_type);
- if (!(flags & HWRM_FUNC_BACKING_STORE_QCAPS_V2_OUTPUT_FLAGS_TYPE_VALID)) {
+ if (!(flags & HWRM_FUNC_BACKING_STORE_QCAPS_V2_OUTPUT_FLAGS_TYPE_VALID) ||
+ (rte_le_to_cpu_16(req.type) ==
+ HWRM_FUNC_BACKING_STORE_QCAPS_V2_INPUT_TYPE_TTX_PACING_TQM_RING &&
+ !(bp->fw_cap & BNXT_FW_CAP_TIMED_TX_PACING))) {
cnt = false;
goto next;
@@ -6900,5 +6903,8 @@ int bnxt_hwrm_func_backing_store_types_count(struct bnxt *bp)
HWRM_UNLOCK();
- if (flags & HWRM_FUNC_BACKING_STORE_QCAPS_V2_OUTPUT_FLAGS_TYPE_VALID) {
+ if ((flags & HWRM_FUNC_BACKING_STORE_QCAPS_V2_OUTPUT_FLAGS_TYPE_VALID) &&
+ !(rte_le_to_cpu_16(req.type) ==
+ HWRM_FUNC_BACKING_STORE_QCAPS_V2_INPUT_TYPE_TTX_PACING_TQM_RING &&
+ !(bp->fw_cap & BNXT_FW_CAP_TIMED_TX_PACING))) {
PMD_DRV_LOG_LINE(DEBUG, "Valid types 0x%x", req.type);
types++;
--
2.55.0
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2026-07-28 16:54:52.262798382 +0100
+++ 0050-net-bnxt-skip-timed-Tx-pacing-setup-when-unsupported.patch 2026-07-28 16:54:50.792728682 +0100
@@ -1 +1 @@
-From edff93f638540783af448de1d40a365b9aee52a6 Mon Sep 17 00:00:00 2001
+From bfb87c79c44532fc8cc36c232522bb6c068b2f8b Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit edff93f638540783af448de1d40a365b9aee52a6 ]
+
@@ -16 +17,0 @@
-Cc: stable at dpdk.org
@@ -27 +28 @@
-index 594936b725..b21cc2c287 100644
+index 62f9f6b3bc..e18f73f3d1 100644
@@ -30 +31 @@
-@@ -312,4 +312,5 @@ Cyril Chemparathy <cchemparathy at ezchip.com> <cchemparathy at tilera.com>
+@@ -307,4 +307,5 @@ Cyril Chemparathy <cchemparathy at ezchip.com> <cchemparathy at tilera.com>
@@ -37 +38 @@
-index 0500414ae0..336de75da0 100644
+index ed80b29fb3..b125aa29f8 100644
@@ -40 +41 @@
-@@ -898,4 +898,5 @@ struct bnxt {
+@@ -885,4 +885,5 @@ struct bnxt {
@@ -45 +46 @@
- #define BNXT_FW_CAP_MULTI_DB BIT(18)
+ #define BNXT_FW_BACKING_STORE_V2_EN(bp) \
@@ -47 +48 @@
-index 66237bc219..1615b36aae 100644
+index bddd55d819..ceebb3531b 100644
@@ -50 +51 @@
-@@ -6846,5 +6846,8 @@ int bnxt_hwrm_func_backing_store_qcaps_v2(struct bnxt *bp)
+@@ -6823,5 +6823,8 @@ int bnxt_hwrm_func_backing_store_qcaps_v2(struct bnxt *bp)
@@ -60 +61 @@
-@@ -6923,5 +6926,8 @@ int bnxt_hwrm_func_backing_store_types_count(struct bnxt *bp)
+@@ -6900,5 +6903,8 @@ int bnxt_hwrm_func_backing_store_types_count(struct bnxt *bp)
More information about the stable
mailing list