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