On Thu, Jul 23, 2009 at 3:59 PM, Johannes Berg<johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2009-07-24 at 00:58 +0200, Johannes Berg wrote: > >> > @@ -2187,7 +2187,7 @@ static void ieee80211_sta_bcn_mon_timer(unsigned long data) >> > (struct ieee80211_sub_if_data *) data; >> > struct ieee80211_local *local = sdata->local; >> > >> > - if (local->quiescing) >> > + if (local->quiescing || local->sw_scanning || local->hw_scanning) >> >> I'd rather have that check in ieee80211_mgd_probe_ap itself, so it's >> much closer to the source of the problem. > > And, incidentally, not racy then. And wouldn't it be even better to just disable the timers before scanning and re-enabling after scanning? Luis -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html