<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Jun 3, 2026 at 1:50 PM Mohammad Shuab Siddique <<a href="mailto:mohammad-shuab.siddique@broadcom.com">mohammad-shuab.siddique@broadcom.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Mohammad Shuab Siddique <<a href="mailto:mohammad-shuab.siddique@broadcom.com" target="_blank">mohammad-shuab.siddique@broadcom.com</a>><br>
<br>
Fixed VFs attempting global RSS configuration which is not<br>
permitted by firmware. VFs (including trusted VFs) must use<br>
per-VNIC RSS configuration with actual vnic_id and rss_ctx_idx<br>
values.<br>
<br>
Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
Signed-off-by: Mohammad Shuab Siddique <<a href="mailto:mohammad-shuab.siddique@broadcom.com" target="_blank">mohammad-shuab.siddique@broadcom.com</a>><br>
</blockquote><div><span style="font-family:Calibri,sans-serif;font-size:11pt">Acked-by:</span><span style="font-family:Calibri,sans-serif;font-size:11pt"> </span><span style="font-family:Calibri,sans-serif;font-size:11pt">Kishore
Padmanabha <</span><a href="mailto:kishore.padmanabha@broadcom.com" style="font-family:Calibri,sans-serif;font-size:11pt;color:rgb(5,99,193)">kishore.padmanabha@broadcom.com</a><span style="font-family:Calibri,sans-serif;font-size:11pt">></span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">---<br>
drivers/net/bnxt/bnxt_hwrm.c | 18 ++++++++++++++++--<br>
1 file changed, 16 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c<br>
index 0c82935de9..afc948ac29 100644<br>
--- a/drivers/net/bnxt/bnxt_hwrm.c<br>
+++ b/drivers/net/bnxt/bnxt_hwrm.c<br>
@@ -2970,8 +2970,22 @@ bnxt_hwrm_vnic_rss_cfg_hash_mode_p5(struct bnxt *bp, struct bnxt_vnic_info *vnic<br>
req.hash_mode_flags = BNXT_HASH_MODE_INNERMOST;<br>
else<br>
req.hash_mode_flags = vnic->hash_mode;<br>
- req.vnic_id = rte_cpu_to_le_16(BNXT_DFLT_VNIC_ID_INVALID);<br>
- req.rss_ctx_idx = rte_cpu_to_le_16(BNXT_RSS_CTX_IDX_INVALID);<br>
+<br>
+ /* VFs must use actual vnic_id for per-VNIC configuration.<br>
+ * PFs can use INVALID vnic_id for global configuration.<br>
+ * This is because VFs don't have permission to configure<br>
+ * global hash mode, even if they're trusted.<br>
+ */<br>
+ if (BNXT_VF(bp)) {<br>
+ req.vnic_id = rte_cpu_to_le_16(vnic->fw_vnic_id);<br>
+ req.rss_ctx_idx = rte_cpu_to_le_16(vnic->fw_grp_ids[0]);<br>
+ PMD_DRV_LOG_LINE(DEBUG, "VF using per-VNIC RSS config (vnic_id=%u)",<br>
+ vnic->fw_vnic_id);<br>
+ } else {<br>
+ req.vnic_id = rte_cpu_to_le_16(BNXT_DFLT_VNIC_ID_INVALID);<br>
+ req.rss_ctx_idx = rte_cpu_to_le_16(BNXT_RSS_CTX_IDX_INVALID);<br>
+ PMD_DRV_LOG_LINE(DEBUG, "PF using global RSS config");<br>
+ }<br>
<br>
PMD_DRV_LOG_LINE(DEBUG, "RSS CFG: Hash level %d", req.hash_mode_flags);<br>
rc = bnxt_hwrm_send_message(bp, &req, sizeof(req),<br>
-- <br>
2.47.3<br>
<br>
</blockquote></div></div>