diff options
| author | Alexander Wetzel <alexander.wetzel@web.de> | 2018-05-14 23:33:34 +0300 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2018-05-18 12:14:36 +0300 |
| commit | 39c1134c66b4552f665da576cb625f184a44a8a3 (patch) | |
| tree | b725235aefb3cb2c970d2dc1e443647100b9a819 /net/mac80211/sta_info.c | |
| parent | 4a22b00b288649f6d697b350ec606574479a2df3 (diff) | |
| download | linux-39c1134c66b4552f665da576cb625f184a44a8a3.tar.xz | |
mac80211: fix TX aggregation stop race
The mac80211 tear down code is not waiting for the driver call back.
This can bring down the the TX path (TID) till the user manually
reconnects. (Observed with iwldvm and enabled TX aggregation.)
The race can be prevented when the ampdu_mlme worker handles the tear
down.
The race:
* ieee80211_sta_tear_down_BA_sessions calls
___ieee80211_stop_tx_ba_session for all TIDs,
* then cancels the ampdu_mlme worker
* and cleanups the TIDs the driver already has called back for.
* ieee80211_stop_tx_ba_cb will never be called for a TID if the callback
came after the the check in ieee80211_sta_tear_down_BA_sessions.
Signed-off-by: Alexander Wetzel <Alexander.Wetzel@web.de>
[johannes: "enabled" -> "blocked" and invert logic, simplify init]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/sta_info.c')
0 files changed, 0 insertions, 0 deletions