net/mlx5: remove flow drop useless if branches
authorNélio Laranjeiro <nelio.laranjeiro@6wind.com>
Wed, 23 Aug 2017 08:15:08 +0000 (10:15 +0200)
committerFerruh Yigit <ferruh.yigit@intel.com>
Fri, 6 Oct 2017 00:49:47 +0000 (02:49 +0200)
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.laranjeiro@6wind.com>
Acked-by: Shahaf Shuler <shahafs@mellanox.com>
drivers/net/mlx5/mlx5_flow.c

index 86be929..d62ccff 100644 (file)
@@ -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;