patch 'net/bnxt: fix statistics for high number of queues' has been queued to stable release 25.11.1

[Kishore Padmanabha]

Hi Kevin,

Please go ahead and merge this change to 25.11.1.

Thanks,
Kishore


-----Original Message-----
From: Kevin Traynor <ktraynor at redhat.com>
Sent: Thursday, February 26, 2026 8:09 AM
To: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
Cc: dpdk stable <stable at dpdk.org>
Subject: patch 'net/bnxt: fix statistics for high number of queues' has
been queued to stable release 25.11.1

Hi,

FYI, your patch has been queued to stable release 25.11.1

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 03/02/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/4631430b86eea5353914b36
51cbf8d232ed39042

Thanks.

Kevin

---
>From 4631430b86eea5353914b3651cbf8d232ed39042 Mon Sep 17 00:00:00 2001
From: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
Date: Tue, 13 Jan 2026 09:59:34 -0500
Subject: [PATCH] net/bnxt: fix statistics for high number of queues

[ upstream commit e0f92e31f72790b63897942e06189a02a12d50b5 ]

Stats collection was not aggregating stats for rings greater than
RTE_ETHDEV_QUEUE_STAT_CNTRS  when the application did not increase the
stats counters but supports more queues than the limit. Add checks to
increment aggregated stats from queues greater than the limit. The fill
functions already handle NULL qstats for aggregate-only collection.

Bugzilla ID: 1836
Fixes: 57d5e5bc86e4 ("net/bnxt: add statistics")

Signed-off-by: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
---
 drivers/net/bnxt/bnxt_stats.c | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/drivers/net/bnxt/bnxt_stats.c b/drivers/net/bnxt/bnxt_stats.c
index 3ed1dc8101..88cfbaf9ff 100644
--- a/drivers/net/bnxt/bnxt_stats.c
+++ b/drivers/net/bnxt/bnxt_stats.c
@@ -660,5 +660,5 @@ static int bnxt_stats_get_ext(struct rte_eth_dev
*eth_dev,
 			      (unsigned int)RTE_ETHDEV_QUEUE_STAT_CNTRS);

-	for (i = 0; i < num_q_stats; i++) {
+	for (i = 0; i < bp->rx_cp_nr_rings; i++) {
 		struct bnxt_rx_queue *rxq = bp->rx_queues[i];
 		struct bnxt_cp_ring_info *cpr = rxq->cp_ring; @@ -676,5
+676,6 @@ static int bnxt_stats_get_ext(struct rte_eth_dev *eth_dev,
 			return rc;

-		bnxt_fill_rte_eth_stats_ext(bnxt_stats, &ring_stats,
qstats, i, true);
+		bnxt_fill_rte_eth_stats_ext(bnxt_stats, &ring_stats,
+					    i < num_q_stats ? qstats :
NULL, i, true);
 		bnxt_stats->rx_nombuf +=

rte_atomic_load_explicit(&rxq->rx_mbuf_alloc_fail,
@@ -685,5 +686,5 @@ static int bnxt_stats_get_ext(struct rte_eth_dev
*eth_dev,
 			      (unsigned int)RTE_ETHDEV_QUEUE_STAT_CNTRS);

-	for (i = 0; i < num_q_stats; i++) {
+	for (i = 0; i < bp->tx_cp_nr_rings; i++) {
 		struct bnxt_tx_queue *txq = bp->tx_queues[i];
 		struct bnxt_cp_ring_info *cpr = txq->cp_ring; @@ -698,5
+699,6 @@ static int bnxt_stats_get_ext(struct rte_eth_dev *eth_dev,
 			return rc;

-		bnxt_fill_rte_eth_stats_ext(bnxt_stats, &ring_stats,
qstats, i, false);
+		bnxt_fill_rte_eth_stats_ext(bnxt_stats, &ring_stats,
+					    i < num_q_stats ? qstats :
NULL, i, false);
 	}

@@ -725,5 +727,5 @@ int bnxt_stats_get_op(struct rte_eth_dev *eth_dev,
 			      (unsigned int)RTE_ETHDEV_QUEUE_STAT_CNTRS);

-	for (i = 0; i < num_q_stats; i++) {
+	for (i = 0; i < bp->rx_cp_nr_rings; i++) {
 		struct bnxt_rx_queue *rxq = bp->rx_queues[i];
 		struct bnxt_cp_ring_info *cpr = rxq->cp_ring; @@ -740,5
+742,6 @@ int bnxt_stats_get_op(struct rte_eth_dev *eth_dev,
 			return rc;

-		bnxt_fill_rte_eth_stats(bnxt_stats, &ring_stats, qstats,
i, true);
+		bnxt_fill_rte_eth_stats(bnxt_stats, &ring_stats,
+					i < num_q_stats ? qstats : NULL,
i, true);
 		bnxt_stats->rx_nombuf +=

rte_atomic_load_explicit(&rxq->rx_mbuf_alloc_fail,
@@ -749,5 +752,5 @@ int bnxt_stats_get_op(struct rte_eth_dev *eth_dev,
 			      (unsigned int)RTE_ETHDEV_QUEUE_STAT_CNTRS);

-	for (i = 0; i < num_q_stats; i++) {
+	for (i = 0; i < bp->tx_cp_nr_rings; i++) {
 		struct bnxt_tx_queue *txq = bp->tx_queues[i];
 		struct bnxt_cp_ring_info *cpr = txq->cp_ring; @@ -762,5
+765,6 @@ int bnxt_stats_get_op(struct rte_eth_dev *eth_dev,
 			return rc;

-		bnxt_fill_rte_eth_stats(bnxt_stats, &ring_stats, qstats,
i, false);
+		bnxt_fill_rte_eth_stats(bnxt_stats, &ring_stats,
+					i < num_q_stats ? qstats : NULL,
i, false);
 		bnxt_stats->oerrors +=

rte_atomic_load_explicit(&txq->tx_mbuf_drop,
--
2.53.0

---
  Diff of the applied patch vs upstream commit (please double-check if
non-empty:
---
--- -	2026-02-26 10:16:48.923905642 +0000
+++ 0046-net-bnxt-fix-statistics-for-high-number-of-queues.patch
2026-02-26 10:16:46.961459281 +0000
@@ -1 +1 @@
-From e0f92e31f72790b63897942e06189a02a12d50b5 Mon Sep 17 00:00:00 2001
+From 4631430b86eea5353914b3651cbf8d232ed39042 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit e0f92e31f72790b63897942e06189a02a12d50b5 ]
+
@@ -15 +16,0 @@
-Cc: stable at dpdk.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5493 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://mails.dpdk.org/archives/stable/attachments/20260226/204c18d4/attachment.bin>