[v4] net/iavf: revert fix VLAN insertion
Checks
Commit Message
The vector Tx path does not support VLAN insertion via the L2TAG2 field,
but the scalar path supports. The earlier commit was to force to select
scalar path as soon as kernel driver requests to use L2TAG2. That logic is
incorrect. Because other case like VLAN offloading not required but scalar
path selected would have a significant performance drop.
Therefore the following commit was reverted accordingly.
commit 0d58caa7d6d1 ("net/iavf: fix VLAN insertion")
After reverting this commit, the AVX512 Tx path would insert the VLAN tag
into the wrong location(inner of QinQ) when the kernel driver requested
L2TAG2. This is inconsistent with the behavior of PF(outer of QinQ).
It is currently known that ice kernel drivers newer than 1.8.9 will request
the use of L2TAG2. User can set parameter '--force-max-simd-bitwidth' to
64/128/256 to avoid this issue.
Fixes: 0d58caa7d6d1 ("net/iavf: fix VLAN insertion")
Cc: stable@dpdk.org
Signed-off-by: Yiding Zhou <yidingx.zhou@intel.com>
---
v4: document this issue as kown issue and add some commit log
v3: rebase and change commit log
---
doc/guides/rel_notes/known_issues.rst | 23 +++++++++++++++++++++++
drivers/net/iavf/iavf_rxtx_vec_common.h | 3 ---
2 files changed, 23 insertions(+), 3 deletions(-)
@@ -906,3 +906,26 @@ Vhost multi-queue reconnection failed with QEMU version 4.2.0 to 5.1.0
**Driver/Module**:
Virtual Device Poll Mode Driver (PMD).
+
+IAVF inserts VLAN tag incorrectly on AVX-512 Tx path
+----------------------------------------------------------------------
+
+**Description**
+ When the kernel driver requests the VF to use the L2TAG2 field of the Tx context
+ descriptor to insert the hardware offload VLAN tag, AVX-512 Tx path cannot handle
+ this case correctly due to its lack of support for the Tx context descriptor.
+
+**Implication**
+ The VLAN tag will be inserted to the wrong location(inner of QinQ) on AVX-512 Tx
+ path. That is inconsistent with the behavior of PF(outer of QinQ). The ice kernel
+ driver's version newer than 1.8.9 requests to use L2TAG2 and has this issue.
+
+**Resolution/Workaround**:
+ Set the parameter `--force-max-simd-bitwidth` as 64/128/256 to avoid selecting AVX-512
+ Tx path
+
+**Affected Environment/Platform**:
+ ALL.
+
+**Driver/Module**:
+ Poll Mode Driver (PMD).
@@ -253,9 +253,6 @@ iavf_tx_vec_queue_default(struct iavf_tx_queue *txq)
if (txq->offloads & IAVF_TX_NO_VECTOR_FLAGS)
return -1;
- if (txq->vlan_flag == IAVF_TX_FLAGS_VLAN_TAG_LOC_L2TAG2)
- return -1;
-
if (txq->offloads & IAVF_TX_VECTOR_OFFLOAD)
return IAVF_VECTOR_OFFLOAD_PATH;