Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 276d197e authored by Aya Levin's avatar Aya Levin Committed by Saeed Mahameed
Browse files

net/mlx5e: Fix error flow of CQE recovery on tx reporter 



CQE recovery function begins with test and set of recovery bit. Add an
error flow which ensures clearing of this bit when leaving the recovery
function, to allow further recoveries to take place. This allows removal
of clearing recovery bit on sq activate.

Fixes: de8650a8 ("net/mlx5e: Add tx reporter support")
Signed-off-by: default avatarAya Levin <ayal@mellanox.com>
Reviewed-by: default avatarTariq Toukan <tariqt@mellanox.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@mellanox.com>
parent d9a2fcf5

drivers/net/ethernet/mellanox/mlx5/core/en/reporter_tx.c

+8 −4
Original line number Diff line number Diff line
@@ -83,17 +83,17 @@ static int mlx5e_tx_reporter_err_cqe_recover(struct mlx5e_txqsq *sq) 
	if (err) {

		netdev_err(dev, "Failed to query SQ 0x%x state. err = %d\n",

			   sq->sqn, err);

		return err;

		goto out;

	}



	if (state != MLX5_SQC_STATE_ERR)

		return 0;

		goto out;



	mlx5e_tx_disable_queue(sq->txq);



	err = mlx5e_wait_for_sq_flush(sq);

	if (err)

		return err;

		goto out;



	/* At this point, no new packets will arrive from the stack as TXQ is

	 * marked with QUEUE_STATE_DRV_XOFF. In addition, NAPI cleared all
@@ -102,13 +102,17 @@ static int mlx5e_tx_reporter_err_cqe_recover(struct mlx5e_txqsq *sq) 


	err = mlx5e_sq_to_ready(sq, state);

	if (err)

		return err;

		goto out;



	mlx5e_reset_txqsq_cc_pc(sq);

	sq->stats->recover++;

	clear_bit(MLX5E_SQ_STATE_RECOVERING, &sq->state);

	mlx5e_activate_txqsq(sq);



	return 0;

out:

	clear_bit(MLX5E_SQ_STATE_RECOVERING, &sq->state);

	return err;

}



static int mlx5_tx_health_report(struct devlink_health_reporter *tx_reporter,

drivers/net/ethernet/mellanox/mlx5/core/en_main.c

+0 −1
Original line number Diff line number Diff line
@@ -1321,7 +1321,6 @@ static int mlx5e_open_txqsq(struct mlx5e_channel *c, 
void mlx5e_activate_txqsq(struct mlx5e_txqsq *sq)

{

	sq->txq = netdev_get_tx_queue(sq->channel->netdev, sq->txq_ix);

	clear_bit(MLX5E_SQ_STATE_RECOVERING, &sq->state);

	set_bit(MLX5E_SQ_STATE_ENABLED, &sq->state);

	netdev_tx_reset_queue(sq->txq);

	netif_tx_start_queue(sq->txq);