mac80211: flush workqueue before calling driver ->stop() method
authorLennert Buytenhek <buytenh@wantstofly.org>
Sun, 10 Jan 2010 13:07:53 +0000 (14:07 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 12 Jan 2010 19:20:56 +0000 (14:20 -0500)
commit678f415fdc534c0a806fce992e4c62df0eff19d2
treeb3313257eca6f22d0ffdb65949c5b22e9c19fbe6
parent5e124bd5e00fcf54df555b368c2dafe6886f1df2
mac80211: flush workqueue before calling driver ->stop() method

Since commit "mwl8k: handle station database update for AP's sta entry
via ->sta_notify()", mwl8k every now and then gets a command timeout
when ifconfig'ing a STA interface down.  This turns out to be due to
mwl8k_stop() being called while the work queue item that was scheduled
by mwl8k_sta_notify() to remove the STA entry for the associated AP is
still queued, and the former disables interrupts so that when the
latter eventually runs, a command completion interrupt is never seen.

Fix this by changing ieee80211_stop_device() so that the workqueue is
flushed before drv_stop() is called, instead of doing it the other way
around as is done now.  (As ->stop() is allowed to sleep, there isn't
any reason for drivers to queue work from within it.)

Signed-off-by: Lennert Buytenhek <buytenh@marvell.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/util.c