diff options
author | Pavel Roskin <proski@gnu.org> | 2010-02-13 22:13:06 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-02-16 14:16:24 -0500 |
commit | 04662360854aefcb6337d93701892ad97411fdd0 (patch) | |
tree | 5d3330040a35c5b03bae63fc848d3e79526c7589 /drivers/net/wireless/mwl8k.c | |
parent | 026331c4d9b526561ea96f95fac4bfc52b69e316 (diff) |
mwl8k: disable softirqs when accessing sta_notify_list
Use spin_[un]lock_bh in mwl8k_sta_notify(). The sta_notify handler is
required to be atomic, yet it can be called in process context, so make
sure one call won't preempt another.
Signed-off-by: Pavel Roskin <proski@gnu.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/mwl8k.c')
-rw-r--r-- | drivers/net/wireless/mwl8k.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wireless/mwl8k.c b/drivers/net/wireless/mwl8k.c index 0cfdb9db66f7..6497c841fb21 100644 --- a/drivers/net/wireless/mwl8k.c +++ b/drivers/net/wireless/mwl8k.c | |||
@@ -3784,9 +3784,9 @@ mwl8k_sta_notify(struct ieee80211_hw *hw, struct ieee80211_vif *vif, | |||
3784 | s->cmd = cmd; | 3784 | s->cmd = cmd; |
3785 | s->sta = *sta; | 3785 | s->sta = *sta; |
3786 | 3786 | ||
3787 | spin_lock(&priv->sta_notify_list_lock); | 3787 | spin_lock_bh(&priv->sta_notify_list_lock); |
3788 | list_add_tail(&s->list, &priv->sta_notify_list); | 3788 | list_add_tail(&s->list, &priv->sta_notify_list); |
3789 | spin_unlock(&priv->sta_notify_list_lock); | 3789 | spin_unlock_bh(&priv->sta_notify_list_lock); |
3790 | 3790 | ||
3791 | ieee80211_queue_work(hw, &priv->sta_notify_worker); | 3791 | ieee80211_queue_work(hw, &priv->sta_notify_worker); |
3792 | } | 3792 | } |