Skip to content

Commit 04e35ca

Browse files
nbd168jmberg-intel
authored andcommitted
mac80211: remove STA txq pending airtime underflow warning
This warning can trigger if there is a mismatch between frames that were sent with the sta pointer set vs tx status frames reported for the sta address. This can happen due to race conditions on re-creating stations, or even in the case of .sta_add/remove being used instead of .sta_state, which can cause frames to be sent to a station that has not been uploaded yet. If there is an actual underflow issue, it should show up in the device airtime warning below, so it is better to remove this one. Signed-off-by: Felix Fietkau <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Johannes Berg <[email protected]>
1 parent e61fbfc commit 04e35ca

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

net/mac80211/sta_info.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1923,9 +1923,7 @@ void ieee80211_sta_update_pending_airtime(struct ieee80211_local *local,
19231923
if (sta) {
19241924
tx_pending = atomic_sub_return(tx_airtime,
19251925
&sta->airtime[ac].aql_tx_pending);
1926-
if (WARN_ONCE(tx_pending < 0,
1927-
"STA %pM AC %d txq pending airtime underflow: %u, %u",
1928-
sta->addr, ac, tx_pending, tx_airtime))
1926+
if (tx_pending < 0)
19291927
atomic_cmpxchg(&sta->airtime[ac].aql_tx_pending,
19301928
tx_pending, 0);
19311929
}

0 commit comments

Comments
 (0)