[PATCH v6 08/25] net/ixgbe: fix syn filter priority

[Medvedkin, Vladimir]

Acked-by: Vladimir Medvedkin <vladimir.medvedkin at intel.com>

On 10/9/2026 11:58 AM, Anatoly Burakov wrote:
> In rte_flow API, 0 means highest priority. Fix SYN filter priority being
> wrongly set to lower priority when API requests priority 0.
>
> Fixes: 429f6ebb42cc ("net/ixgbe: parse TCP SYN filter")
> Cc: stable at dpdk.org
>
> Signed-off-by: Anatoly Burakov <anatoly.burakov at intel.com>
> ---
>   drivers/net/intel/ixgbe/ixgbe_flow.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/intel/ixgbe/ixgbe_flow.c b/drivers/net/intel/ixgbe/ixgbe_flow.c
> index a2b9403d3a1..6ba36a8b44a 100644
> --- a/drivers/net/intel/ixgbe/ixgbe_flow.c
> +++ b/drivers/net/intel/ixgbe/ixgbe_flow.c
> @@ -804,10 +804,10 @@ cons_parse_syn_filter(const struct rte_flow_attr *attr, const struct rte_flow_it
>   	filter->queue = q_act->index;
>   
>   	/* Support 2 priorities, the lowest or highest. */
> -	if (!attr->priority) {
> -		filter->hig_pri = 0;
> -	} else if (attr->priority == (uint32_t)~0U) {
> +	if (attr->priority == 0) {
>   		filter->hig_pri = 1;
> +	} else if (attr->priority == (uint32_t)~0U) {
> +		filter->hig_pri = 0;
>   	} else {
>   		memset(filter, 0, sizeof(struct rte_eth_syn_filter));
>   		rte_flow_error_set(error, EINVAL,

-- 
Regards,
Vladimir