<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"><meta name="Generator" content="Microsoft Word 15 (filtered medium)"><style></style></head><body lang="EN-US" link="blue" vlink="purple" style="word-wrap:break-word"><div class="WordSection1">Hi Denis, I have accepted the change and post it for merging the change. Thanks,Kishore <div><div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in">From: Денис Люлин <<a href="mailto:lyulin.2003@mail.ru">lyulin.2003@mail.ru</a>> Sent: Thursday, July 23, 2026 5:35 AM To: Kishore Padmanabha <<a href="mailto:kishore.padmanabha@broadcom.com">kishore.padmanabha@broadcom.com</a>> Cc: Ajit Khaparde <<a href="mailto:ajit.khaparde@broadcom.com">ajit.khaparde@broadcom.com</a>>; Manish Kurup <<a href="mailto:manish.kurup@broadcom.com">manish.kurup@broadcom.com</a>>; Shuanglin Wang <<a href="mailto:shuanglin.wang@broadcom.com">shuanglin.wang@broadcom.com</a>>; Randy Schacher <<a href="mailto:stuart.schacher@broadcom.com">stuart.schacher@broadcom.com</a>>; <a href="mailto:dev@dpdk.org">dev@dpdk.org</a>; <a href="mailto:stable@dpdk.org">stable@dpdk.org</a> Subject: Re: [PATCH] net/bnxt: fix memset size mismatch in Tx queue start</div></div> Kindly reminder Sorry, But nobody has reviewed my patch on Patchwork <a href="https://patches.dpdk.org/project/dpdk/patch/20260416090252.2211405-1-lyulin.2003@mail.ru/">https://patches.dpdk.org/project/dpdk/patch/20260416090252.2211405-1-lyulin.2003@mail.ru/</a> Could you please review this? <blockquote style="border:none;border-left:solid #0857a6 1.0pt;padding:0in 0in 0in 8.0pt;margin-left:7.5pt;margin-top:7.5pt;margin-right:7.5pt;margin-bottom:7.5pt"><div id=""><div id="style_17764436580970480114_mr_css_attr"><div id="style_17764436580970480114_BODY_mr_css_attr">On Thu, Apr 16, 2026 at 5:03 AM Denis Lyulin <<a href="mailto:lyulin.2003@mail.ru">lyulin.2003@mail.ru</a>> wrote:<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">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: <a href="mailto:shuanglin.wang@broadcom.com">shuanglin.wang@broadcom.com</a> Cc: <a href="mailto:stable@dpdk.org">stable@dpdk.org</a> Signed-off-by: Denis Lyulin <<a href="mailto:lyulin.2003@mail.ru">lyulin.2003@mail.ru</a>></blockquote>Acked-by: Kishore Padmanabha <<a href="mailto:kishore.padmanabha@broadcom.com">kishore.padmanabha@broadcom.com</a>> <blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">--- 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 </blockquote></div></div></div></blockquote></div></body></html>