Commit 9a4f92a6 authored by Yevgeny Petrilin's avatar Yevgeny Petrilin Committed by David S. Miller

mlx4_en: Fix error handling while activating RX rings

In case of failure of either srq creation or page allocation,
the cleanup code handled the failed ring as well, and tried
to destroy resources that where not allocated.
Signed-off-by: default avatarYevgeny Petrilin <yevgenyp@mellanox.co.il>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 775d8d93
...@@ -436,8 +436,9 @@ int mlx4_en_activate_rx_rings(struct mlx4_en_priv *priv) ...@@ -436,8 +436,9 @@ int mlx4_en_activate_rx_rings(struct mlx4_en_priv *priv)
/* Initialize page allocators */ /* Initialize page allocators */
err = mlx4_en_init_allocator(priv, ring); err = mlx4_en_init_allocator(priv, ring);
if (err) { if (err) {
mlx4_err(mdev, "Failed initializing ring allocator\n"); mlx4_err(mdev, "Failed initializing ring allocator\n");
goto err_allocator; ring_ind--;
goto err_allocator;
} }
/* Fill Rx buffers */ /* Fill Rx buffers */
...@@ -467,6 +468,7 @@ int mlx4_en_activate_rx_rings(struct mlx4_en_priv *priv) ...@@ -467,6 +468,7 @@ int mlx4_en_activate_rx_rings(struct mlx4_en_priv *priv)
ring->wqres.db.dma, &ring->srq); ring->wqres.db.dma, &ring->srq);
if (err){ if (err){
mlx4_err(mdev, "Failed to allocate srq\n"); mlx4_err(mdev, "Failed to allocate srq\n");
ring_ind--;
goto err_srq; goto err_srq;
} }
ring->srq.event = mlx4_en_srq_event; ring->srq.event = mlx4_en_srq_event;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment