Acked-by: Vladimir Medvedkin <[email protected]>
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: [email protected] Signed-off-by: Anatoly Burakov <[email protected]> --- 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
