diff options
| author | Marc Kleine-Budde <mkl@pengutronix.de> | 2024-01-12 17:13:14 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2024-01-13 18:26:23 +0000 |
| commit | 894d7508316e7ad722df597d68b4b1797a9eee11 (patch) | |
| tree | eb2d4b06d498502881d5059209a88c879cc4a90f | |
| parent | 9181d6f8a2bb32d158de66a84164fac05e3ddd18 (diff) | |
net: netdev_queue: netdev_txq_completed_mb(): fix wake condition
netif_txq_try_stop() uses "get_desc >= start_thrs" as the check for
the call to netif_tx_start_queue().
Use ">=" i netdev_txq_completed_mb(), too.
Fixes: c91c46de6bbc ("net: provide macros for commonly copied lockless queue stop/wake code")
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Acked-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | include/net/netdev_queues.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/net/netdev_queues.h b/include/net/netdev_queues.h index d68b0a483431..8b8ed4e13d74 100644 --- a/include/net/netdev_queues.h +++ b/include/net/netdev_queues.h @@ -128,7 +128,7 @@ netdev_txq_completed_mb(struct netdev_queue *dev_queue, netdev_txq_completed_mb(txq, pkts, bytes); \ \ _res = -1; \ - if (pkts && likely(get_desc > start_thrs)) { \ + if (pkts && likely(get_desc >= start_thrs)) { \ _res = 1; \ if (unlikely(netif_tx_queue_stopped(txq)) && \ !(down_cond)) { \ |