On Thu, Jun 04, 2026 at 07:13:53PM +0300, Artem Novikov wrote:
> The hash action parser stops iterating when it reaches
> RTE_FLOW_ACTION_TYPE_END. The switch below uses action_type copied
> directly from action->type inside that loop, so the END case cannot be
> selected.
>
> Remove the unreachable case. No functional change.
>
> Fixes: 7be10c3004be ("net/iavf: add RSS configuration for VF")
> Cc: [email protected]
> Cc: [email protected]
>
> Signed-off-by: Artem Novikov <[email protected]>
> ---
> drivers/net/intel/iavf/iavf_hash.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/intel/iavf/iavf_hash.c
> b/drivers/net/intel/iavf/iavf_hash.c
> index 217f0500ab..7ce7b86d8e 100644
> --- a/drivers/net/intel/iavf/iavf_hash.c
> +++ b/drivers/net/intel/iavf/iavf_hash.c
> @@ -1490,8 +1490,6 @@ iavf_hash_parse_action(struct iavf_pattern_match_item
> *match_item,
> rss_type, pattern_hint);
> break;
>
> - case RTE_FLOW_ACTION_TYPE_END:
> - break;
>
> default:
> rte_flow_error_set(error, EINVAL,
This code is no longer present, following the recent rework of Intel driver
flow parsing, done by Anatoly. Therefore marking this as "not applicable"
in patchwork.
Thanks,
/Bruce
