diff options
author | Mohamad Haj Yahia <mohamad@mellanox.com> | 2016-12-06 17:32:48 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-12-06 11:44:45 -0500 |
commit | c0f1147d14e4b09018a495c5095094e5707a4f44 (patch) | |
tree | 3dfd05fe193895006dd58375674129bca7952cd6 /drivers/net/ethernet/mellanox/mlx5/core/en_tx.c | |
parent | 3c8591d593a3da9ae8e8342acb1f6ab9ab478e92 (diff) | |
download | op-kernel-dev-c0f1147d14e4b09018a495c5095094e5707a4f44.zip op-kernel-dev-c0f1147d14e4b09018a495c5095094e5707a4f44.tar.gz |
net/mlx5e: Change the SQ/RQ operational state to positive logic
When using the negative logic (i.e. FLUSH state), after the RQ/SQ reopen
we will have a time interval that the RQ/SQ is not really ready and the
state indicates that its not in FLUSH state because the initial SQ/RQ struct
memory starts as zeros.
Now we changed the state to indicate if the SQ/RQ is opened and we will
set the READY state after finishing preparing all the SQ/RQ resources.
Fixes: 6e8dd6d6f4bd ("net/mlx5e: Don't wait for SQ completions on close")
Fixes: f2fde18c52a7 ("net/mlx5e: Don't wait for RQ completions on close")
Signed-off-by: Mohamad Haj Yahia <mohamad@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/mellanox/mlx5/core/en_tx.c')
-rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/en_tx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c index 70a7173..cfb6837 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c @@ -409,7 +409,7 @@ bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq, int napi_budget) sq = container_of(cq, struct mlx5e_sq, cq); - if (unlikely(test_bit(MLX5E_SQ_STATE_FLUSH, &sq->state))) + if (unlikely(!test_bit(MLX5E_SQ_STATE_ENABLED, &sq->state))) return false; npkts = 0; |