On Wed, Jul 08, 2026 at 06:58:47PM +0000, Anurag Mandal wrote:
> In Double VLAN Mode (DVM) i.e. when RTE_ETH_RX_OFFLOAD_VLAN_EXTEND
> is enabled iavf_config_vlan_strip_v2() only toggles the inner
> VLAN strip setting (0x8100) for the RTE_ETH_RX_OFFLOAD_VLAN_STRIP
> offload, while the outer VLAN strip is controlled independently
> through the QinQ strip path.
>
> However, in DVM, most hardwares place a single (non-QinQ) VLAN
> tag in the outer position
> As a result, when a single VLAN packet is received and the
> application disables single VLAN tag stripping, only the inner
> VLAN tag strip gets turned off and the outer VLAN tag strip
> stays enabled.
> Hence, the single VLAN tag still gets stripped.
> This does not honor the application's request.
>
> This patch fixes the aforesaid case.
> When DVM and QinQ stripping are both enabled, also apply the strip
> enable/disable to the outer VLAN so that single VLAN packets received
> in double VLAN mode respect the single VLAN strip configuration.
>
> Fixes: 8599d7604e0a ("net/iavf: support QinQ strip")
> Cc: [email protected]
>
> Signed-off-by: Anurag Mandal <[email protected]>
> Tested-by: Ye Chen <[email protected]>
> ---
> V3: Restricted to VLAN TPID 0x8100
> V2: Spelling correction in commit message
>
Applied to dpdk-next-net-intel.
Thanks,
/Bruce
