diff options
author | Sujith <Sujith.Manoharan@atheros.com> | 2009-03-16 23:20:06 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-03-27 20:12:55 -0400 |
commit | 722f069a6dc95d7c6c2cdfbe3413899a3b768f9c (patch) | |
tree | 3f9ccbd9e2fdf75bd27675c48b238adc911ae626 /net/mac80211/agg-tx.c | |
parent | e23a9014fd4d502a419255a83e2479ab804c6f16 (diff) |
mac80211: Tear down aggregation sessions for suspend/resume
When the driver has been notified with a STA_REMOVE, it tears down
the internal ADDBA state. On resume, trying to initiate aggregation would
fail because mac80211 has not cleared the operational state for that <TID,STA>.
This can be fixed by tearing down the existing sessions on a suspend.
Also, the driver can initiate a new BA session when suspend is in progress.
This is fixed by marking the station as being in suspend state and
denying ADDBA requests for such STAs.
Signed-off-by: Sujith <Sujith.Manoharan@atheros.com>
Acked-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/agg-tx.c')
-rw-r--r-- | net/mac80211/agg-tx.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/net/mac80211/agg-tx.c b/net/mac80211/agg-tx.c index 1df116d4d6e7..e5776ef1717a 100644 --- a/net/mac80211/agg-tx.c +++ b/net/mac80211/agg-tx.c | |||
@@ -257,6 +257,15 @@ int ieee80211_start_tx_ba_session(struct ieee80211_hw *hw, u8 *ra, u16 tid) | |||
257 | goto unlock; | 257 | goto unlock; |
258 | } | 258 | } |
259 | 259 | ||
260 | if (test_sta_flags(sta, WLAN_STA_SUSPEND)) { | ||
261 | #ifdef CONFIG_MAC80211_HT_DEBUG | ||
262 | printk(KERN_DEBUG "Suspend in progress. " | ||
263 | "Denying BA session request\n"); | ||
264 | #endif | ||
265 | ret = -EINVAL; | ||
266 | goto unlock; | ||
267 | } | ||
268 | |||
260 | spin_lock_bh(&sta->lock); | 269 | spin_lock_bh(&sta->lock); |
261 | 270 | ||
262 | sdata = sta->sdata; | 271 | sdata = sta->sdata; |