aboutsummaryrefslogtreecommitdiffstats
path: root/net/mac80211/sta_info.h
diff options
context:
space:
mode:
authorJohannes Berg <johannes@sipsolutions.net>2009-02-10 15:25:51 -0500
committerJohn W. Linville <linville@tuxdriver.com>2009-02-13 13:45:43 -0500
commit55687e380a3965ac448e03281e027553a6ae6dac (patch)
treeba7c079d3ce4d7278eeb45317ba3baf54f45c96c /net/mac80211/sta_info.h
parent23e6a7ea5cb1a902d37ab0c783709c178fa834df (diff)
mac80211: fix aggregation timer lockups
As far as I can tell, there are possible lockups because both the RX session_timer and TX addba_resp_timer are del_timer_sync'ed under the sta spinlock which both timer functions take. Additionally, the TX agg code seems to leak memory when TX aggregation is not disabled before the sta_info is freed. Fix this by making the free code a little smarter in the RX agg case, and actually make the sta_info_destroy code free the TX agg info in the TX agg case. We won't notify the peer, but it'll notice something is wrong anyway, and normally this only happens after we've told it in some other way we will no longer talk to it. Signed-off-by: Johannes Berg <johannes@sipsolutions.net> Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/sta_info.h')
-rw-r--r--net/mac80211/sta_info.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/net/mac80211/sta_info.h b/net/mac80211/sta_info.h
index a070bd929e00..d9653231992f 100644
--- a/net/mac80211/sta_info.h
+++ b/net/mac80211/sta_info.h
@@ -100,6 +100,7 @@ struct tid_ampdu_rx {
100 u16 buf_size; 100 u16 buf_size;
101 u16 timeout; 101 u16 timeout;
102 u8 dialog_token; 102 u8 dialog_token;
103 bool shutdown;
103}; 104};
104 105
105/** 106/**