[Make-wifi-fast] Fwd: [PATCH] rtl8xxxu: Use lower tx rates for the ack packet

[Dave Taht]

I am glad more folk are looking at the excessive retransmit problem

---------- Forwarded message ---------
From: Chris Chiu <chris.chiu at canonical.com>
Date: Thu, Sep 30, 2021 at 7:31 AM
Subject: [PATCH] rtl8xxxu: Use lower tx rates for the ack packet
To: <Jes.Sorensen at gmail.com>, <kvalo at codeaurora.org>,
<davem at davemloft.net>, <kuba at kernel.org>
Cc: <linux-wireless at vger.kernel.org>, <netdev at vger.kernel.org>,
<linux-kernel at vger.kernel.org>, Chris Chiu <chris.chiu at canonical.com>


According to the Realtek propritary driver and the rtw88 driver, the
tx rates of the ack (includes block ack) are initialized with lower
tx rates (no HT rates) which is set by the RRSR register value. In
real cases, ack rate higher than current tx rate could lead to
difficulty for the receiving end to receive management/control frames.
The retransmission rate would be higher then expected when the driver
is acting as receiver and the RSSI is not good.

Cross out higer rates for ack packet before implementing dynamic rrsr
configuration

Signed-off-by: Chris Chiu <chris.chiu at canonical.com>
---
 drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c | 7 ++++++-
 drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_regs.h | 2 ++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
index 774341b0005a..413cccd88f5c 100644
--- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
+++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
@@ -4460,13 +4460,18 @@ void rtl8xxxu_gen1_init_aggregation(struct
rtl8xxxu_priv *priv)

 static void rtl8xxxu_set_basic_rates(struct rtl8xxxu_priv *priv, u32 rate_cfg)
 {
+       struct ieee80211_hw *hw = priv->hw;
        u32 val32;
        u8 rate_idx = 0;

        rate_cfg &= RESPONSE_RATE_BITMAP_ALL;

        val32 = rtl8xxxu_read32(priv, REG_RESPONSE_RATE_SET);
-       val32 &= ~RESPONSE_RATE_BITMAP_ALL;
+              val32 = rtl8xxxu_read32(priv, REG_RESPONSE_RATE_SET);
+       if (hw->conf.chandef.chan->band == NL80211_BAND_5GHZ)
+               val32 &= RESPONSE_RATE_RRSR_INIT_5G;
+       else
+               val32 &= RESPONSE_RATE_RRSR_INIT_2G;
        val32 |= rate_cfg;
        rtl8xxxu_write32(priv, REG_RESPONSE_RATE_SET, val32);

diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_regs.h
b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_regs.h
index a2a31f374a82..438b65ba9640 100644
--- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_regs.h
+++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_regs.h
@@ -516,6 +516,8 @@
 #define REG_RESPONSE_RATE_SET          0x0440
 #define  RESPONSE_RATE_BITMAP_ALL      0xfffff
 #define  RESPONSE_RATE_RRSR_CCK_ONLY_1M        0xffff1
+#define  RESPONSE_RATE_RRSR_INIT_2G    0x15f
+#define  RESPONSE_RATE_RRSR_INIT_5G    0x150
 #define  RSR_1M                                BIT(0)
 #define  RSR_2M                                BIT(1)
 #define  RSR_5_5M                      BIT(2)
--
2.20.1



-- 
Fixing Starlink's Latencies: https://www.youtube.com/watch?v=c9gLo6Xrwgw

Dave Täht CEO, TekLibre, LLC