Hi Denis,
I have accepted the change and post it for merging the change. Thanks, Kishore *From:* Денис Люлин <[email protected]> *Sent:* Thursday, July 23, 2026 5:35 AM *To:* Kishore Padmanabha <[email protected]> *Cc:* Ajit Khaparde <[email protected]>; Manish Kurup < [email protected]>; Shuanglin Wang <[email protected]>; Randy Schacher <[email protected]>; [email protected]; [email protected] *Subject:* Re: [PATCH] net/bnxt: fix memset size mismatch in Tx queue start Kindly reminder Sorry, But nobody has reviewed my patch on Patchwork https://patches.dpdk.org/project/dpdk/patch/[email protected]/ Could you please review this? On Thu, Apr 16, 2026 at 5:03 AM Denis Lyulin <[email protected]> wrote: When the tx_queue is started, previous stats are reset. For tpa_v2 and P7, extended stats structure is used: `struct bnxt_ring_stats_ext`, so passing wrong size to memset makes not all stats to be reset. This commit sets correct size of the structure passed to memset when zeroing-out previous stats. Fixes: dd0191d5e70d ("net/bnxt/tf_ulp: support Thor2 ULP layer") Cc: [email protected] Cc: [email protected] Signed-off-by: Denis Lyulin <[email protected]> Acked-by: Kishore Padmanabha <[email protected]> --- drivers/net/bnxt/bnxt_txr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/bnxt/bnxt_txr.c b/drivers/net/bnxt/bnxt_txr.c index 27758898b0..b6ad37f872 100644 --- a/drivers/net/bnxt/bnxt_txr.c +++ b/drivers/net/bnxt/bnxt_txr.c @@ -729,7 +729,7 @@ int bnxt_tx_queue_start(struct rte_eth_dev *dev, uint16_t tx_queue_id) */ if (BNXT_TPA_V2_P7(bp)) memset(&bp->prev_tx_ring_stats_ext[tx_queue_id], 0, - sizeof(struct bnxt_ring_stats)); + sizeof(struct bnxt_ring_stats_ext)); else memset(&bp->prev_tx_ring_stats[tx_queue_id], 0, sizeof(struct bnxt_ring_stats)); -- 2.34.1
smime.p7s
Description: S/MIME Cryptographic Signature
