diff options
| author | Tejun Heo <tj@kernel.org> | 2010-10-15 11:35:28 -0400 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2010-10-25 14:43:12 -0400 |
| commit | 99b88a0ecbdbc6df03527292571b2b442965814a (patch) | |
| tree | 71a5c8b9e0dcb6e6a07c182e05b97bff8095a348 /net | |
| parent | 229aebb873e29726b91e076161649cf45154b0bf (diff) | |
mac80211: cancel restart_work explicitly instead of depending on flush_scheduled_work()
iee80211_hw->restart_work is the only work which uses the system
workqueue. Instead of calling flush_scheduled_work() during
iee80211_exit(), cancel the work during unregistration.
This is to prepare for the deprecation and removal of
flush_scheduled_work().
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/mac80211/main.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/net/mac80211/main.c b/net/mac80211/main.c index 22bc42b18991..6b322fa681f5 100644 --- a/net/mac80211/main.c +++ b/net/mac80211/main.c | |||
| @@ -748,7 +748,7 @@ int ieee80211_register_hw(struct ieee80211_hw *hw) | |||
| 748 | hw->queues = IEEE80211_MAX_QUEUES; | 748 | hw->queues = IEEE80211_MAX_QUEUES; |
| 749 | 749 | ||
| 750 | local->workqueue = | 750 | local->workqueue = |
| 751 | create_singlethread_workqueue(wiphy_name(local->hw.wiphy)); | 751 | alloc_ordered_workqueue(wiphy_name(local->hw.wiphy), 0); |
| 752 | if (!local->workqueue) { | 752 | if (!local->workqueue) { |
| 753 | result = -ENOMEM; | 753 | result = -ENOMEM; |
| 754 | goto fail_workqueue; | 754 | goto fail_workqueue; |
| @@ -962,12 +962,6 @@ static void __exit ieee80211_exit(void) | |||
| 962 | rc80211_minstrel_ht_exit(); | 962 | rc80211_minstrel_ht_exit(); |
| 963 | rc80211_minstrel_exit(); | 963 | rc80211_minstrel_exit(); |
| 964 | 964 | ||
| 965 | /* | ||
| 966 | * For key todo, it'll be empty by now but the work | ||
| 967 | * might still be scheduled. | ||
| 968 | */ | ||
| 969 | flush_scheduled_work(); | ||
| 970 | |||
| 971 | if (mesh_allocated) | 965 | if (mesh_allocated) |
| 972 | ieee80211s_stop(); | 966 | ieee80211s_stop(); |
| 973 | 967 | ||
