On Wed, 2010-09-29 at 09:58 -0700, Ben Greear wrote: > >> + if (!(wk->chan == local->scan_channel || > >> + (wk->chan == local->oper_channel&& > > > > I don't think work items will ever run while scanning? And also, this > > might need to check the channel type as well, which isn't quite > > necessary right now (since no works requires HT channels) but would seem > > cleaner. > > Just in case work ever does happen when scanning, I think we should > leave this check in. I think we'll have bigger problems if that happens, since scanning will switch away from the channel again quickly, likely before the work finishes. I'd rather add an assertion that it _doesn't_ happen, I think. 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
