mac80211: always use mac80211 loss detection
authorDavid Bauer <mail@david-bauer.net>
Wed, 17 May 2023 10:04:00 +0000 (12:04 +0200)
committerDavid Bauer <mail@david-bauer.net>
Thu, 22 Jun 2023 22:23:58 +0000 (00:23 +0200)
ath10k does not report excessive loss in case of broken block-ack
sessions. The loss is communicated to the host-os, but ath10k does not
trigger a low-ack events by itself.

The mac80211 framework for loss detection however detects this
circumstance well in case of ath10k. So use it regardless of ath10k's
own loss detection mechanism.

Signed-off-by: David Bauer <mail@david-bauer.net>
(cherry picked from commit ed816f6ba8b5e3df609e5fc53f7bcb09bdaa16be)

package/kernel/mac80211/patches/subsys/340-mac80211-always-use-mac80211-loss-detection.patch [new file with mode: 0644]

diff --git a/package/kernel/mac80211/patches/subsys/340-mac80211-always-use-mac80211-loss-detection.patch b/package/kernel/mac80211/patches/subsys/340-mac80211-always-use-mac80211-loss-detection.patch
new file mode 100644 (file)
index 0000000..e084773
--- /dev/null
@@ -0,0 +1,36 @@
+From cdf461888f900c3a149b10a04d72b4a590ecdec3 Mon Sep 17 00:00:00 2001
+From: David Bauer <mail@david-bauer.net>
+Date: Tue, 16 May 2023 23:11:32 +0200
+Subject: [PATCH] mac80211: always use mac80211 loss detection
+
+ath10k does not report excessive loss in case of broken block-ack
+sessions. The loss is communicated to the host-os, but ath10k does not
+trigger a low-ack events by itself.
+
+The mac80211 framework for loss detection however detects this
+circumstance well in case of ath10k. So use it regardless of ath10k's
+own loss detection mechanism.
+
+Patching this in mac80211 does allow this hack to be used with any
+flavor of ath10k/ath11k.
+
+Signed-off-by: David Bauer <mail@david-bauer.net>
+---
+ net/mac80211/status.c | 6 ------
+ 1 file changed, 6 deletions(-)
+
+--- a/net/mac80211/status.c
++++ b/net/mac80211/status.c
+@@ -794,12 +794,6 @@ static void ieee80211_lost_packet(struct
+       unsigned long pkt_time = STA_LOST_PKT_TIME;
+       unsigned int pkt_thr = STA_LOST_PKT_THRESHOLD;
+-      /* If driver relies on its own algorithm for station kickout, skip
+-       * mac80211 packet loss mechanism.
+-       */
+-      if (ieee80211_hw_check(&sta->local->hw, REPORTS_LOW_ACK))
+-              return;
+-
+       /* This packet was aggregated but doesn't carry status info */
+       if ((info->flags & IEEE80211_TX_CTL_AMPDU) &&
+           !(info->flags & IEEE80211_TX_STAT_AMPDU))