27c9d80f51
Some checks failed
Build Kernel / Build all affected Kernels (push) Has been cancelled
Build all core packages / Build all core packages for selected target (push) Has been cancelled
Build and Push prebuilt tools container / Build and Push all prebuilt containers (push) Has been cancelled
Build Toolchains / Build Toolchains for each target (push) Has been cancelled
Build host tools / Build host tools for linux and macos based systems (push) Has been cancelled
Coverity scan build / Coverity x86/64 build (push) Has been cancelled
43 lines
1.4 KiB
Diff
43 lines
1.4 KiB
Diff
From: Felix Fietkau <nbd@nbd.name>
|
|
Date: Wed, 29 Mar 2023 16:02:54 +0200
|
|
Subject: [PATCH] net: ethernet: mtk_eth_soc: fix remaining throughput
|
|
regression
|
|
|
|
Based on further tests, it seems that the QDMA shaper is not able to
|
|
perform shaping close to the MAC link rate without throughput loss.
|
|
This cannot be compensated by increasing the shaping rate, so it seems
|
|
to be an internal limit.
|
|
|
|
Fix the remaining throughput regression by detecting that condition and
|
|
limiting shaping to ports with lower link speed.
|
|
|
|
This patch intentionally ignores link speed gain from TRGMII, because
|
|
even on such links, shaping to 1000 Mbit/s incurs some throughput
|
|
degradation.
|
|
|
|
Fixes: f63959c7eec3 ("net: ethernet: mtk_eth_soc: implement multi-queue support for per-port queues")
|
|
Reported-by: Frank Wunderlich <frank-w@public-files.de>
|
|
Signed-off-by: Felix Fietkau <nbd@nbd.name>
|
|
---
|
|
|
|
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
|
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
|
@@ -766,6 +766,7 @@ static void mtk_mac_link_up(struct phyli
|
|
MAC_MCR_FORCE_RX_FC);
|
|
|
|
/* Configure speed */
|
|
+ mac->speed = speed;
|
|
switch (speed) {
|
|
case SPEED_2500:
|
|
case SPEED_1000:
|
|
@@ -3347,6 +3348,9 @@ found:
|
|
if (dp->index >= MTK_QDMA_NUM_QUEUES)
|
|
return NOTIFY_DONE;
|
|
|
|
+ if (mac->speed > 0 && mac->speed <= s.base.speed)
|
|
+ s.base.speed = 0;
|
|
+
|
|
mtk_set_queue_speed(eth, dp->index + 3, s.base.speed);
|
|
|
|
return NOTIFY_DONE;
|