[PATCH] net/mlx5: fix creation of drop flows

[PATCH] net/mlx5: fix creation of drop flows

[Nelio Laranjeiro]

Drop flows being created when the port is stop should not access to the drop
table hash queues as it is invalid.

Fixes: 028761059aeb ("net/mlx5: use an RSS drop queue")

Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: Shahaf Shuler <shahafs@mellanox.com>
---
 drivers/net/mlx5/mlx5_flow.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
index 8b3957b..96bbf42 100644
--- a/drivers/net/mlx5/mlx5_flow.c
+++ b/drivers/net/mlx5/mlx5_flow.c
@@ -1004,9 +1004,9 @@ priv_flow_create_action_queue_drop(struct priv *priv,
 	}
 	rte_flow->drop = 1;
 	rte_flow->ibv_attr = flow->ibv_attr;
-	rte_flow->qp = priv->flow_drop_queue->qp;
 	if (!priv->started)
 		return rte_flow;
+	rte_flow->qp = priv->flow_drop_queue->qp;
 	rte_flow->ibv_flow = ibv_exp_create_flow(rte_flow->qp,
 						 rte_flow->ibv_attr);
 	if (!rte_flow->ibv_flow) {
-- 
2.1.4

Re: [PATCH] net/mlx5: fix creation of drop flows

[Ferruh Yigit]

On 6/15/2017 9:25 AM, Nelio Laranjeiro wrote:
> Drop flows being created when the port is stop should not access to the drop
> table hash queues as it is invalid.
> 
> Fixes: 028761059aeb ("net/mlx5: use an RSS drop queue")
> 
> Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
> Acked-by: Shahaf Shuler <shahafs@mellanox.com>

Applied to dpdk-next-net/master, thanks.

This required solving merge conflict, although it looks simple, can you
please confirm the final commit.

Re: [PATCH] net/mlx5: fix creation of drop flows

[Nélio Laranjeiro]

On Thu, Jun 15, 2017 at 01:34:31PM +0100, Ferruh Yigit wrote:
> On 6/15/2017 9:25 AM, Nelio Laranjeiro wrote:
> > Drop flows being created when the port is stop should not access to the drop
> > table hash queues as it is invalid.
> > 
> > Fixes: 028761059aeb ("net/mlx5: use an RSS drop queue")
> > 
> > Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
> > Acked-by: Shahaf Shuler <shahafs@mellanox.com>
> 
> Applied to dpdk-next-net/master, thanks.
> 
> This required solving merge conflict, although it looks simple, can you
> please confirm the final commit.

it is good for me.

Thanks Ferruh,

-- 
Nélio Laranjeiro
6WIND