2021-04-24 00:19:22 +00:00
|
|
|
From 16ef670789b252b221700adc413497ed2f941d8a Mon Sep 17 00:00:00 2001
|
2020-10-24 19:14:16 +00:00
|
|
|
From: Felix Fietkau <nbd@nbd.name>
|
2021-04-24 00:19:22 +00:00
|
|
|
Date: Thu, 22 Apr 2021 22:20:59 -0700
|
|
|
|
Subject: [PATCH] net: ethernet: mtk_eth_soc: remove unnecessary TX queue stops
|
2020-10-24 19:14:16 +00:00
|
|
|
|
2021-04-24 00:19:22 +00:00
|
|
|
When running short on descriptors, only stop the queue for the netdev that
|
|
|
|
tx was attempted for. By the time something tries to send on the other
|
|
|
|
netdev, the ring might have some more room already.
|
2020-10-24 19:14:16 +00:00
|
|
|
|
|
|
|
Signed-off-by: Felix Fietkau <nbd@nbd.name>
|
2021-04-24 00:19:22 +00:00
|
|
|
Signed-off-by: Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com>
|
|
|
|
Signed-off-by: David S. Miller <davem@davemloft.net>
|
2020-10-24 19:14:16 +00:00
|
|
|
---
|
2021-04-24 00:19:22 +00:00
|
|
|
drivers/net/ethernet/mediatek/mtk_eth_soc.c | 15 ++-------------
|
|
|
|
1 file changed, 2 insertions(+), 13 deletions(-)
|
2020-10-24 19:14:16 +00:00
|
|
|
|
|
|
|
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
|
|
|
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
2022-06-22 15:58:33 +00:00
|
|
|
@@ -1163,17 +1163,6 @@ static void mtk_wake_queue(struct mtk_et
|
2020-10-24 19:14:16 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
-static void mtk_stop_queue(struct mtk_eth *eth)
|
|
|
|
-{
|
|
|
|
- int i;
|
|
|
|
-
|
|
|
|
- for (i = 0; i < MTK_MAC_COUNT; i++) {
|
|
|
|
- if (!eth->netdev[i])
|
|
|
|
- continue;
|
|
|
|
- netif_stop_queue(eth->netdev[i]);
|
|
|
|
- }
|
|
|
|
-}
|
|
|
|
-
|
|
|
|
static netdev_tx_t mtk_start_xmit(struct sk_buff *skb, struct net_device *dev)
|
|
|
|
{
|
|
|
|
struct mtk_mac *mac = netdev_priv(dev);
|
2022-06-22 15:58:33 +00:00
|
|
|
@@ -1194,7 +1183,7 @@ static netdev_tx_t mtk_start_xmit(struct
|
2020-10-24 19:14:16 +00:00
|
|
|
|
|
|
|
tx_num = mtk_cal_txd_req(skb);
|
|
|
|
if (unlikely(atomic_read(&ring->free_count) <= tx_num)) {
|
|
|
|
- mtk_stop_queue(eth);
|
|
|
|
+ netif_stop_queue(dev);
|
|
|
|
netif_err(eth, tx_queued, dev,
|
|
|
|
"Tx Ring full when queue awake!\n");
|
|
|
|
spin_unlock(ð->page_lock);
|
2022-06-22 15:58:33 +00:00
|
|
|
@@ -1220,7 +1209,7 @@ static netdev_tx_t mtk_start_xmit(struct
|
2020-10-24 19:14:16 +00:00
|
|
|
goto drop;
|
|
|
|
|
|
|
|
if (unlikely(atomic_read(&ring->free_count) <= ring->thresh))
|
|
|
|
- mtk_stop_queue(eth);
|
|
|
|
+ netif_stop_queue(dev);
|
|
|
|
|
|
|
|
spin_unlock(ð->page_lock);
|
|
|
|
|