[PATCH] net/i40e: remove redundant judgment in fdir parse
Kaiwen Deng
kaiwenx.deng at intel.com
Fri Nov 24 08:01:03 CET 2023
if (eth_spec && eth_mask &&
next_type == RTE_FLOW_ITEM_TYPE_END) {
...
if (next_type == RTE_FLOW_ITEM_TYPE_VLAN || ...) {
...
}
...
}
Clearly, that condition in the inner "if" is always "false".
This commit will remove the redundant judgment.
Fixes: 7d83c152a207 ("net/i40e: parse flow director filter")
Cc: stable at dpdk.org
Signed-off-by: Kaiwen Deng <kaiwenx.deng at intel.com>
---
drivers/net/i40e/i40e_flow.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/i40e/i40e_flow.c b/drivers/net/i40e/i40e_flow.c
index 877e49151e..92165c8422 100644
--- a/drivers/net/i40e/i40e_flow.c
+++ b/drivers/net/i40e/i40e_flow.c
@@ -1708,8 +1708,7 @@ i40e_flow_parse_fdir_pattern(struct rte_eth_dev *dev,
ether_type = rte_be_to_cpu_16(eth_spec->hdr.ether_type);
- if (next_type == RTE_FLOW_ITEM_TYPE_VLAN ||
- ether_type == RTE_ETHER_TYPE_IPV4 ||
+ if (ether_type == RTE_ETHER_TYPE_IPV4 ||
ether_type == RTE_ETHER_TYPE_IPV6 ||
ether_type == i40e_get_outer_vlan(dev)) {
rte_flow_error_set(error, EINVAL,
--
2.34.1
More information about the dev
mailing list