From 0b5ca74dda4085a80c4b8cdc1933377a9b4444d5 Mon Sep 17 00:00:00 2001 From: Ajit Khaparde Date: Mon, 16 Apr 2018 18:11:18 -0700 Subject: [PATCH] net/bnxt: fix L2 filter cleanup We are wrongly freeing up a filter in the driver while it is still configured in the HW. This can cause incorrect L2 filter id to be used for filters created subsequently. This filter will be cleared on cleanup anyway. Fixes: 5ef3b79fdfe6 ("net/bnxt: support flow filter ops") Cc: stable@dpdk.org Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_filter.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/drivers/net/bnxt/bnxt_filter.c b/drivers/net/bnxt/bnxt_filter.c index 5f3154060c..d28c04038f 100644 --- a/drivers/net/bnxt/bnxt_filter.c +++ b/drivers/net/bnxt/bnxt_filter.c @@ -919,11 +919,6 @@ bnxt_validate_and_parse_flow(struct rte_eth_dev *dev, goto ret; } - if (filter1) { - bnxt_free_filter(bp, filter1); - filter1->fw_l2_filter_id = -1; - } - act = nxt_non_void_action(++act); if (act->type != RTE_FLOW_ACTION_TYPE_END) { rte_flow_error_set(error, EINVAL, -- 2.20.1