Re: [PATCH wireless-2.6] iwlwifi: do not perferm force reset while doing scan

["Guy, Wey-Yi" <wey-yi.w.guy@intel.com>]

On Wed, 2010-09-22 at 08:23 -0700, Stanislaw Gruszka wrote:
> On Wed, 22 Sep 2010 17:10:57 +0200
> Johannes Berg <johannes@sipsolutions.net> 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