On Fri, 2012-03-16 at 14:13 +0100, Stanislaw Gruszka wrote: > On Fri, Mar 16, 2012 at 01:52:58PM +0100, Johannes Berg wrote: > > On Fri, 2012-03-16 at 13:02 +0100, Stanislaw Gruszka wrote: > > > Before we send probes in connection monitoring we check if scan is not > > > pending. But we do that check without locking. Fix that and also do not > > > start scan if connection monitoring is in progress. > > > > Wouldn't it be easier to just abort the check instead of deferring the > > scan? > > It would be easier. However we already defer scan on > !list_empty(&local->work_list) case. Also random failures of > scan request can not be well seen from user-space perspective. Ah, I didn't mean aborting the scan request, but rather the connection monitor check. But yeah, I guess you're right about already deferring. 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