diff options
author | Johannes Berg <johannes.berg@intel.com> | 2013-06-03 07:51:59 -0400 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2013-06-03 07:51:59 -0400 |
commit | ed405be5cb5e8e494e8dcd83b6eb1add34dc752a (patch) | |
tree | 97e167b6dd4cc16aa31adcbba575c381013574c3 /net/mac80211/cfg.c | |
parent | a8cd57b3042446250f19daf98d120802dffc768d (diff) |
mac80211: fix sdata locking around __ieee80211_request_smps
My cfg80211/mac80211 locking unification broke the sdata
locking in ieee80211_set_power_mgmt, it needs to acquire
the lock for __ieee80211_request_smps(). Add the locking.
Reported-by: Jakub Kicinski <kubakici@wp.pl>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/cfg.c')
-rw-r--r-- | net/mac80211/cfg.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c index 9034da16cf1b..30622101d3b5 100644 --- a/net/mac80211/cfg.c +++ b/net/mac80211/cfg.c | |||
@@ -2378,7 +2378,9 @@ static int ieee80211_set_power_mgmt(struct wiphy *wiphy, struct net_device *dev, | |||
2378 | local->dynamic_ps_forced_timeout = timeout; | 2378 | local->dynamic_ps_forced_timeout = timeout; |
2379 | 2379 | ||
2380 | /* no change, but if automatic follow powersave */ | 2380 | /* no change, but if automatic follow powersave */ |
2381 | sdata_lock(sdata); | ||
2381 | __ieee80211_request_smps(sdata, sdata->u.mgd.req_smps); | 2382 | __ieee80211_request_smps(sdata, sdata->u.mgd.req_smps); |
2383 | sdata_unlock(sdata); | ||
2382 | 2384 | ||
2383 | if (local->hw.flags & IEEE80211_HW_SUPPORTS_DYNAMIC_PS) | 2385 | if (local->hw.flags & IEEE80211_HW_SUPPORTS_DYNAMIC_PS) |
2384 | ieee80211_hw_config(local, IEEE80211_CONF_CHANGE_PS); | 2386 | ieee80211_hw_config(local, IEEE80211_CONF_CHANGE_PS); |