<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif">I will restore the explicit rejection in v2.  </div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, Sep 22, 2026 at 1:50 AM David Marchand <<a href="mailto:david.marchand@redhat.com">david.marchand@redhat.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">On Mon, 21 Sept 2026 at 18:21, Stephen Hemminger<br>
<<a href="mailto:stephen@networkplumber.org" target="_blank">stephen@networkplumber.org</a>> wrote:<br>
><br>
> On Sun, 20 Sep 2026 21:08:11 -0600<br>
> Mohammad Shuab Siddique <<a href="mailto:mohammad-shuab.siddique@broadcom.com" target="_blank">mohammad-shuab.siddique@broadcom.com</a>> wrote:<br>
><br>
> > From: Keegan Freyhof <<a href="mailto:keegan.freyhof@broadcom.com" target="_blank">keegan.freyhof@broadcom.com</a>><br>
> ><br>
> > Remove support for the VMDq feature from the driver: the multi-pool<br>
> > Rx-queue configuration path in bnxt_mq_rx_configure(), the VMDQ VLAN<br>
> > pool wiring in the L2 filter alloc path, and the max_vmdq_pools/<br>
> > vmdq_queue_num reporting in dev_info (now always reported as 0).<br>
> ><br>
> > Preserved two pieces of logic that live in the same<br>
> > function/switch statement but are not VMDq-pool-specific: the<br>
> > RTE_ETH_MQ_RX_RSS pools=rx_cosq_cnt case and the<br>
> > per-non-default-VNIC skip_filter_allocation guard in<br>
> > bnxt_mq_rx_configure(), both used by the plain multi-COS RSS path.<br>
> > Also kept the RTE_ETH_VMDQ_ACCEPT_HASH_UC/BROADCAST/MULTICAST -><br>
> > l2_rx_mask flag mapping in rte_pmd_bnxt_set_vf_rxmode(): despite the<br>
> > name, those bits are the standard DPDK VF-rxmode API used regardless<br>
> > of hardware VMDq pool support, not a VMDq-pool-specific control.<br>
> ><br>
> > Signed-off-by: Keegan Freyhof <<a href="mailto:keegan.freyhof@broadcom.com" target="_blank">keegan.freyhof@broadcom.com</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>
> > ---<br>
><br>
> [PATCH] net/bnxt: remove VMDq support<br>
><br>
> Warning: mq_mode validation is lost. Before, VMDq, DCB and DCB_RSS<br>
> modes hit the default case and returned -EINVAL. Now any mode other<br>
> than NONE or RSS silently configures a single pool and succeeds.<br>
> Reject every other mq_mode explicitly in bnxt_mq_rx_configure() or<br>
> dev_configure.<br>
<br>
mq_mode should be validated in ethdev now, in next-net.<br>
93b017ff2ffa ("ethdev: check VMDq availability")<br>
<br>
Is there something missing?<br>
<br>
-- <br>
David Marchand<br>
<br>
</blockquote></div><div><br clear="all"></div><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Shuab</div></div></div></div>