On Wed, 2010-09-22 at 08:23 -0700, Stanislaw Gruszka wrote: > On Wed, 22 Sep 2010 17:10:57 +0200 > Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > > > On Wed, 2010-09-22 at 16:51 +0200, Stanislaw Gruszka wrote: > > > > > Yes, bg_restart() force scan abort, but not if no scan is pending. It looks > > > like that: > > > > > > cpu0 cpu1 > > > > > > iwl_bg_restart() > > > __iwl_down() > > > iwl_scan_cancel_timeout > > > (do nothing since no pending scan) > > > > > > __ieee80211_start_scan > > > __set_bit(SCAN_HW_SCANNING, &local->scanning); > > > > Ah, ok, makes sense. I think some mutex trickery could possibly help > > here though? But I'm not sure it's worth it... > > I showed possible patch earlier in this thread, I will rethink about it and > eventually post. Perhaps applying the patch is good idea, since we will stop > seeing the warning it that harmless case, and be sure something is really > wrong when see it again. > I will prefer not seeing "warning" even it is harmless in this case Thanks Wey -- 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