86bc29e4a8
[mac80211] ca5ee6e mac80211: Fix potential endless loop 2c14710 mac80211: add more AQL fixes/improvements 91fb3ce mac80211: remove an obsolete patch that is no longer doing anything useful acf1733 mac80211: add preliminary support for enabling 802.11ax in config d717343 mac80211: update encap offload patches to the latest version 673062f mac80211: allow bigger A-MSDU sizes in VHT, even if HT is limited caf7277 mac80211: do not allow bigger VHT MPDUs than the hardware supports cd36c0d mac80211: select the first available channel for 5GHz interfaces 1c6d456 mac80211: fix regression in station connection monitor optimization 4bd7689 mac80211: update sta connection monitor regression fix [target] Sync: at91, ath25, ath79, lantiq, mediatek, mvebu.
39 lines
1.4 KiB
Diff
39 lines
1.4 KiB
Diff
From: Felix Fietkau <nbd@nbd.name>
|
|
Date: Mon, 21 Sep 2020 17:43:06 +0200
|
|
Subject: [PATCH] mac80211: fix regression in sta connection monitor
|
|
|
|
When a frame was acked and probe frames were sent, the connection monitoring
|
|
needs to be reset, otherwise it will keep probing until the connection is
|
|
considered dead, even though frames have been acked in the mean time.
|
|
|
|
Fixes: 9abf4e49830d ("mac80211: optimize station connection monitor")
|
|
Reported-by: Georgi Valkov <gvalkov@abv.bg>
|
|
Signed-off-by: Felix Fietkau <nbd@nbd.name>
|
|
---
|
|
|
|
--- a/net/mac80211/status.c
|
|
+++ b/net/mac80211/status.c
|
|
@@ -1129,6 +1129,8 @@ void ieee80211_tx_status_ext(struct ieee
|
|
noack_success = !!(info->flags & IEEE80211_TX_STAT_NOACK_TRANSMITTED);
|
|
|
|
if (pubsta) {
|
|
+ struct ieee80211_sub_if_data *sdata = sta->sdata;
|
|
+
|
|
if (!acked && !noack_success)
|
|
sta->status_stats.retry_failed++;
|
|
sta->status_stats.retry_count += retry_count;
|
|
@@ -1143,6 +1145,13 @@ void ieee80211_tx_status_ext(struct ieee
|
|
/* Track when last packet was ACKed */
|
|
sta->status_stats.last_pkt_time = jiffies;
|
|
|
|
+ /* Reset connection monitor */
|
|
+ if (sdata->vif.type == NL80211_IFTYPE_STATION &&
|
|
+ unlikely(sdata->u.mgd.probe_send_count > 0)) {
|
|
+ sdata->u.mgd.probe_send_count = 0;
|
|
+ ieee80211_queue_work(&local->hw, &sdata->work);
|
|
+ }
|
|
+
|
|
if (info->status.is_valid_ack_signal) {
|
|
sta->status_stats.last_ack_signal =
|
|
(s8)info->status.ack_signal;
|