On Mon, Mar 16, 2009 at 02:35:30PM +0200, Kalle Valo wrote: > > so this should not matter much, but the comment could be made more > > clear about the different needs for nullfunc frames (please also > > s/null frames/nullfunc frames/) and probe request frames. The former > > are sent only on the operational channel in the beginning and end of > > scan while the latter are sent on the channels to be scanned during > > an active scan. > > Should the description be in ieee80211_start_scan() in scan.c? I think > it would make more sense to have it there instead of tx.c. I can then > add a reference to the comment above. Either way works for me as long as there is something giving me enough information (or pointer to that) next to the place that allows nullfunc frames go through. Anyway, Johannes is correct about the proper longer term fix being better mechanism to stop the queue so that we don't get into this code in the first place and in that sense, this patch is more of a workaround for the time being. -- Jouni Malinen PGP id EFC895FA -- 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
