Those two if statements are useless as there is a verification on the drop
field of the flow to jump to the end of the function just above.

Signed-off-by: Nelio Laranjeiro <nelio.laranje...@6wind.com>
Acked-by: Shahaf Shuler <shah...@mellanox.com>
---
 drivers/net/mlx5/mlx5_flow.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
index 86be929..d62ccff 100644
--- a/drivers/net/mlx5/mlx5_flow.c
+++ b/drivers/net/mlx5/mlx5_flow.c
@@ -1279,10 +1279,6 @@ priv_flow_destroy(struct priv *priv,
                claim_zero(ibv_destroy_qp(flow->qp));
        if (flow->ind_table)
                claim_zero(ibv_exp_destroy_rwq_ind_table(flow->ind_table));
-       if (flow->drop && flow->wq)
-               claim_zero(ibv_exp_destroy_wq(flow->wq));
-       if (flow->drop && flow->cq)
-               claim_zero(ibv_destroy_cq(flow->cq));
        if (flow->mark) {
                struct rte_flow *tmp;
                struct rxq *rxq;
-- 
2.1.4

Reply via email to