Search Linux Wireless

Re: [PATCH w-t] iwlwifi: rewrite iwl-scan.c to avoid race conditions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2010-09-01 at 17:24 +0200, Stanislaw Gruszka wrote:

> > Ok ... Not sure I understand. Why do we care about abort_timeout work
> > coming after it? We'd cancel it anyway, when we kill the scan from
> > bg_restart, no?
> 
> I wanted abort_timeout finish to avoid warning I putted in
> iwl_scan_cancel_sleep, but I could just remove the warning ...

Ok.

> Cancel at the end of iwl_cancel_scan_deferred_work() is for case
> when we still get notification from f/w and clear STATUS_SCAN_HW bit in 
> iwl_rx_scan_complete_notif. This can happen when firmware is in some
> sane state but we do restart (for example: when debugfs force_reset
> file was used).
> 
> For the same reason we need have to iwl_bg_scan_completed allow to run
> in parallel with iwl_bg_restart. To complete scanning when performing
> restart, but f/w is responsible.

I think you mean "responsive"? Otherwise what you're saying doesn't make
much sense to me?

But they all take the mutex, so they can't run in parallel anyway. So it
only becomes an ordering issue, no?

johannes

--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux