net/mlx5: fix device stop with multiple regions

LIST macro are not safe when inside a LIST_FOREACH() a LIST_REMOVE() is
called to remove an entry, this behavior is undefined causing some entries
to disappear from the list.

Fixes: 6e78005a9b30 ("net/mlx5: add reference counter on DPDK Tx queues")

Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>

diff --git a/drivers/net/mlx5/mlx5_trigger.c b/drivers/net/mlx5/mlx5_trigger.c
index 982d0a2..5de2d02 100644 (file)
--- a/drivers/net/mlx5/mlx5_trigger.c
+++ b/drivers/net/mlx5/mlx5_trigger.c
@@ -188,7 +188,7 @@ mlx5_dev_start(struct rte_eth_dev *dev)
 error:
        /* Rollback. */
        dev->data->dev_started = 0;
-       LIST_FOREACH(mr, &priv->mr, next)
+       for (mr = LIST_FIRST(&priv->mr); mr; mr = LIST_FIRST(&priv->mr))
                priv_mr_release(priv, mr);
        priv_flow_stop(priv, &priv->flows);
        priv_dev_traffic_disable(priv, dev);
@@ -230,9 +230,8 @@ mlx5_dev_stop(struct rte_eth_dev *dev)
        priv_dev_interrupt_handler_uninstall(priv, dev);
        priv_txq_stop(priv);
        priv_rxq_stop(priv);
-       LIST_FOREACH(mr, &priv->mr, next) {
+       for (mr = LIST_FIRST(&priv->mr); mr; mr = LIST_FIRST(&priv->mr))
                priv_mr_release(priv, mr);
-       }
        priv_flow_delete_drop_queue(priv);
        priv_unlock(priv);
 }