diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2009-08-21 08:44:45 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-08-28 14:40:34 -0400 |
commit | ea77f12f2cc0f31168f2e0259e65a22202ac4dc2 (patch) | |
tree | 8ac1288e1680ae2b6093b4fdca978a4ea86f7e21 /net/mac80211/ieee80211_i.h | |
parent | 3d54d25515838543e56889aa7e48f40d00719368 (diff) |
mac80211: remove tasklet enable/disable
Due to the way the tasklets work in mac80211 there's
no need to ever disable them.
However, we need to clear the pending packets when
taking down the last interface because otherwise
the tx_pending_tasklet might be queued if the
driver mucks with the queues (which it shouldn't).
I've had a situation occasionally with ar9170 in
which ksoftirq was using 100% CPU time because
a disabled tasklet was scheduled, and I think that
was due to ar9170 receiving a packet while the
tasklet was disabled. That's strange and it really
should not do that for other reasons, but there's
no need to waste that much CPU time over it, it
should just warn instead.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/ieee80211_i.h')
-rw-r--r-- | net/mac80211/ieee80211_i.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h index fa930e01295f..dbd8411cc1bd 100644 --- a/net/mac80211/ieee80211_i.h +++ b/net/mac80211/ieee80211_i.h | |||
@@ -667,6 +667,9 @@ struct ieee80211_local { | |||
667 | */ | 667 | */ |
668 | bool quiescing; | 668 | bool quiescing; |
669 | 669 | ||
670 | /* device is started */ | ||
671 | bool started; | ||
672 | |||
670 | int tx_headroom; /* required headroom for hardware/radiotap */ | 673 | int tx_headroom; /* required headroom for hardware/radiotap */ |
671 | 674 | ||
672 | /* Tasklet and skb queue to process calls from IRQ mode. All frames | 675 | /* Tasklet and skb queue to process calls from IRQ mode. All frames |