RE: [PATCH] net/i40e: remove redundant judgment in fdir parse

[Zhang, Qi Z]

> -----Original Message-----
> From: Kaiwen Deng <kaiwenx.d...@intel.com>
> Sent: Friday, November 24, 2023 3:01 PM
> To: dev@dpdk.org
> Cc: sta...@dpdk.org; Yang, Qiming <qiming.y...@intel.com>; Zhou, YidingX
> <yidingx.z...@intel.com>; Deng, KaiwenX <kaiwenx.d...@intel.com>; Zhang,
> Yuying <yuying.zh...@intel.com>; Xing, Beilei <beilei.x...@intel.com>; Wu,
> Jingjing <jingjing...@intel.com>
> Subject: [PATCH] net/i40e: remove redundant judgment in fdir parse
> 
> 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: sta...@dpdk.org
> 
> Signed-off-by: Kaiwen Deng <kaiwenx.d...@intel.com>

Acked-by: Qi Zhang <qi.z.zh...@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi