On Mon, Aug 09, 2010 at 07:06:53PM +0200, Johannes Berg wrote: > On Mon, 2010-08-09 at 18:02 +0200, Stanislaw Gruszka wrote: > > > iwl_scan_cancel_timeout() will not success if we are performing it > > from work handler, because workqueue can not fire new work when > > still other one is performed. To prevent such situation > > schedule priv->abort_scan to generic workqueue. Also increase > > sleep time to have better change abort_scan work complete. > > I don't like this much, but you're right. However, I think we can get > around this by having iwl_scan_cancel_timeout() not call > iwl_scan_cancel() but rather do everything that iwl_bg_abort_scan() > would do directly, since _timeout() may sleep, while scan_cancel() may > not. What do you think? Make sense to me. I will do this as separate patch. Stanislaw -- 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
