diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2010-01-06 15:30:58 +0100 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-01-12 13:51:28 -0500 |
commit | 81ac3462d346ee7aaf037a35156b0a7a354e98cf (patch) | |
tree | 8ad3bd87fc62db3b5a0bd69c5b4a2fc8613414b9 /net/mac80211/work.c | |
parent | 34a6eddbabd704b3c7dae9362234552267573be2 (diff) | |
download | kernel_samsung_crespo-81ac3462d346ee7aaf037a35156b0a7a354e98cf.zip kernel_samsung_crespo-81ac3462d346ee7aaf037a35156b0a7a354e98cf.tar.gz kernel_samsung_crespo-81ac3462d346ee7aaf037a35156b0a7a354e98cf.tar.bz2 |
mac80211: fix a few work bugs
Kalle and Lennert reported problems with the new work
code, and at least Kalle's problem I was able to trace
to a missing jiffies initialisation.
I also ran into a problem where occasionally I couldn't
connect, which seems fixed with kicking the work items
after scanning.
Finally, also add some sanity checking code to verify
that we're not adding work items while an interface is
down -- that case could lead to something similar to
what Lennert was seeing.
There still seems to be a race condition that we're
trying to figure out separately.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Tested-by: Lennert Buytenhek <buytenh@marvell.com>
Tested-by: Kalle Valo <kalle.valo@iki.fi>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/work.c')
-rw-r--r-- | net/mac80211/work.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/net/mac80211/work.c b/net/mac80211/work.c index 5ba7599..7c5d95b 100644 --- a/net/mac80211/work.c +++ b/net/mac80211/work.c @@ -818,6 +818,7 @@ static void ieee80211_work_work(struct work_struct *work) wk->chan == local->tmp_channel && wk->chan_type == local->tmp_channel_type) { wk->started = true; + wk->timeout = jiffies; } if (!wk->started && !local->tmp_channel) { @@ -935,6 +936,9 @@ void ieee80211_add_work(struct ieee80211_work *wk) if (WARN_ON(!wk->done)) return; + if (WARN_ON(!ieee80211_sdata_running(wk->sdata))) + return; + wk->started = false; local = wk->sdata->local; |