diff options
| author | Matt Chen <matt.chen@intel.com> | 2017-01-22 02:16:58 +0800 |
|---|---|---|
| committer | Willy Tarreau <w@1wt.eu> | 2017-06-20 14:04:26 +0200 |
| commit | 6958f50ab442e17848b8949e8bc468d09b823c5a (patch) | |
| tree | b71969a9855b0530112122c91df2646e9db1d922 /net | |
| parent | 0c51e5d46aeba36a10f0b7a71901bde4ff4bc76e (diff) | |
| download | linux-6958f50ab442e17848b8949e8bc468d09b823c5a.tar.gz linux-6958f50ab442e17848b8949e8bc468d09b823c5a.tar.bz2 linux-6958f50ab442e17848b8949e8bc468d09b823c5a.zip | |
mac80211: flush delayed work when entering suspend
commit a9e9200d8661c1a0be8c39f93deb383dc940de35 upstream.
The issue was found when entering suspend and resume.
It triggers a warning in:
mac80211/key.c: ieee80211_enable_keys()
...
WARN_ON_ONCE(sdata->crypto_tx_tailroom_needed_cnt ||
sdata->crypto_tx_tailroom_pending_dec);
...
It points out sdata->crypto_tx_tailroom_pending_dec isn't cleaned up successfully
in a delayed_work during suspend. Add a flush_delayed_work to fix it.
Signed-off-by: Matt Chen <matt.chen@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Willy Tarreau <w@1wt.eu>
Diffstat (limited to 'net')
| -rw-r--r-- | net/mac80211/pm.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/mac80211/pm.c b/net/mac80211/pm.c index efb510e6f206..a1f47b8d8013 100644 --- a/net/mac80211/pm.c +++ b/net/mac80211/pm.c @@ -114,6 +114,7 @@ int __ieee80211_suspend(struct ieee80211_hw *hw, struct cfg80211_wowlan *wowlan) break; } + flush_delayed_work(&sdata->dec_tailroom_needed_wk); drv_remove_interface(local, sdata); } |