net/mlx5: fix redundant flow after RSS expansion
authorXiaoyu Min <jackmin@nvidia.com>
Tue, 30 Mar 2021 13:40:32 +0000 (21:40 +0800)
committerRaslan Darawsheh <rasland@nvidia.com>
Tue, 13 Apr 2021 11:23:01 +0000 (13:23 +0200)
commitc1d397a67eed3b9d7adce5e73c5352de040b3b65
treecb54048376c2f0435dd1ba2dce7789f8946314a5
parent61cfa035d3712243d0b724e6e3d76607520671e2
net/mlx5: fix redundant flow after RSS expansion

When RSS expand, if there is no expansion happened but completion
happened because user only input next protocol field instead of item
i.e, ether type == 0x8100 instead of VLAN, an extra flow is created with
missing item in order to filter traffic strictly.

However, after [1] and [2] the rte_flow_item_eth itself is enough to
filter out VLAN traffic, the VLAN item is not needed.

[1]: commit 09315fc83861 ("ethdev: add VLAN attributes to ethernet and VLAN items")
[2]: commit 86b59a1af671 ("net/mlx5: support VLAN matching fields")

This redundant flow will cause failure in some scenarios on group 0 due
to they are the same FTE.

Fixes: fc2dd8dd492f ("ethdev: fix expand RSS flows")
Cc: stable@dpdk.org
Signed-off-by: Xiaoyu Min <jackmin@nvidia.com>
Acked-by: Matan Azrad <matan@nvidia.com>
drivers/net/mlx5/mlx5_flow.c