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

[Johannes Berg <johannes@sipsolutions.net>]

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