diff options
author | Johannes Berg <johannes.berg@intel.com> | 2011-05-13 07:35:40 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2011-05-16 14:10:40 -0400 |
commit | ec034b208dc8aa5dc73ec46c3f27e34c5efbf113 (patch) | |
tree | 56140fc3a4115441822b5cc2aa9de5224bcb8654 /net/mac80211/sta_info.h | |
parent | 7527a782e187d1214a5b3dc2897ce441033bb4ef (diff) |
mac80211: fix TX a-MPDU locking
During my quest to make mac80211 not have any RCU
warnings from sparse, I came across the a-MPDU code
again and it wasn't quite clear why it isn't racy.
So instead of assigning the tid_tx array with just
the spinlock held in ieee80211_start_tx_ba_session
use a separate temporary array protected only by
the spinlock and protect all assignments to the
"live" array by both the spinlock and the mutex so
that other code is easily verified to be correct.
Due to pointer assignment atomicity I don't think
this is a real issue, but I'm not sure, especially
on Alpha the current code might be problematic.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/sta_info.h')
-rw-r--r-- | net/mac80211/sta_info.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/net/mac80211/sta_info.h b/net/mac80211/sta_info.h index f00b4dcb49d7..55c51855ceb7 100644 --- a/net/mac80211/sta_info.h +++ b/net/mac80211/sta_info.h | |||
@@ -152,6 +152,7 @@ struct tid_ampdu_rx { | |||
152 | * | 152 | * |
153 | * @tid_rx: aggregation info for Rx per TID -- RCU protected | 153 | * @tid_rx: aggregation info for Rx per TID -- RCU protected |
154 | * @tid_tx: aggregation info for Tx per TID | 154 | * @tid_tx: aggregation info for Tx per TID |
155 | * @tid_start_tx: sessions where start was requested | ||
155 | * @addba_req_num: number of times addBA request has been sent. | 156 | * @addba_req_num: number of times addBA request has been sent. |
156 | * @dialog_token_allocator: dialog token enumerator for each new session; | 157 | * @dialog_token_allocator: dialog token enumerator for each new session; |
157 | * @work: work struct for starting/stopping aggregation | 158 | * @work: work struct for starting/stopping aggregation |
@@ -168,6 +169,7 @@ struct sta_ampdu_mlme { | |||
168 | /* tx */ | 169 | /* tx */ |
169 | struct work_struct work; | 170 | struct work_struct work; |
170 | struct tid_ampdu_tx *tid_tx[STA_TID_NUM]; | 171 | struct tid_ampdu_tx *tid_tx[STA_TID_NUM]; |
172 | struct tid_ampdu_tx *tid_start_tx[STA_TID_NUM]; | ||
171 | u8 addba_req_num[STA_TID_NUM]; | 173 | u8 addba_req_num[STA_TID_NUM]; |
172 | u8 dialog_token_allocator; | 174 | u8 dialog_token_allocator; |
173 | }; | 175 | }; |
@@ -398,6 +400,8 @@ static inline u32 get_sta_flags(struct sta_info *sta) | |||
398 | return ret; | 400 | return ret; |
399 | } | 401 | } |
400 | 402 | ||
403 | void ieee80211_assign_tid_tx(struct sta_info *sta, int tid, | ||
404 | struct tid_ampdu_tx *tid_tx); | ||
401 | 405 | ||
402 | 406 | ||
403 | #define STA_HASH_SIZE 256 | 407 | #define STA_HASH_SIZE 256 |