From: Mohammad Shuab Siddique <mohammad-shuab.siddique@broadcom.com>
To: dev@dpdk.org
Cc: kishore.padmanabha@broadcom.com,
Mohammad Shuab Siddique <mohammad-shuab.siddique@broadcom.com>,
stable@dpdk.org
Subject: [PATCH] net/bnxt: fix l2_db_multi_page_size_kb position in func_qcfg output
Date: Tue, 24 Mar 2026 14:48:06 -0600 [thread overview]
Message-ID: <20260324204815.54465-1-Mohammad-Shuab.Siddique@broadcom.com> (raw)
From: Mohammad Shuab Siddique <mohammad-shuab.siddique@broadcom.com>
The field was placed at incorrect offset in hwrm_func_qcfg_output causing
legacy_l2_db_size_kb to read svif_info instead, producing a doorbell offset
exceeding BAR2 and resulting in SIGSEGV on BCM57608.
Move l2_db_multi_page_size_kb after mirror_vnic_id to match firmware
response layout.
Fixes: 04102f2ffc8c ("net/bnxt: update HWRM API to version 1.10.1.6")
Fixes: 7a1f9c782b50 ("net/bnxt: add multi-doorbell support")
Cc: stable@dpdk.org
Signed-off-by: Mohammad Shuab Siddique <mohammad-shuab.siddique@broadcom.com>
---
drivers/net/bnxt/hsi_struct_def_dpdk.h | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/net/bnxt/hsi_struct_def_dpdk.h b/drivers/net/bnxt/hsi_struct_def_dpdk.h
index 26d5e9ca78..1a8874f5b6 100644
--- a/drivers/net/bnxt/hsi_struct_def_dpdk.h
+++ b/drivers/net/bnxt/hsi_struct_def_dpdk.h
@@ -16277,12 +16277,6 @@ struct __rte_packed_begin hwrm_func_qcfg_output {
* of the doorbell BAR between L2 and RoCE is required.
*/
uint16_t l2_doorbell_bar_size_kb;
- /*
- * The size of the doorbell BAR in KBytes reserved for multi-L2
- * doorbell pages. This area is a subset of l2_doorbell_bar_size_kb,
- * which is size of the total doorbell BAR space reserved for L2.
- */
- uint16_t l2_db_multi_page_size_kb;
/*
* A bitmask indicating the active endpoints. Each bit represents a
* specific endpoint, with bit 0 indicating EP 0 and bit 3 indicating
@@ -16605,6 +16599,12 @@ struct __rte_packed_begin hwrm_func_qcfg_output {
* or a raw QP.
*/
uint16_t mirror_vnic_id;
+ /*
+ * The size of the doorbell BAR in KBytes reserved for multi-L2
+ * doorbell pages. This area is a subset of l2_doorbell_bar_size_kb,
+ * which is size of the total doorbell BAR space reserved for L2.
+ */
+ uint16_t l2_db_multi_page_size_kb;
uint8_t unused_7[7];
/*
* This field is used in Output records to indicate that the output
--
2.47.3
next reply other threads:[~2026-03-24 20:46 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-24 20:48 Mohammad Shuab Siddique [this message]
2026-03-24 21:02 ` [PATCH] net/bnxt: fix l2_db_multi_page_size_kb position in func_qcfg output Kishore Padmanabha
2026-03-25 14:17 ` Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260324204815.54465-1-Mohammad-Shuab.Siddique@broadcom.com \
--to=mohammad-shuab.siddique@broadcom.com \
--cc=dev@dpdk.org \
--cc=kishore.padmanabha@broadcom.com \
--cc=stable@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox