On Wed, 2011-09-21 at 19:28 +0300, Victor Goldenshtein wrote: > On 21/09/2011 18:45, Luciano Coelho wrote: > > On Wed, 2011-09-21 at 18:31 +0300, Jouni Malinen wrote: > >> On Thu, Sep 08, 2011 at 09:31:01AM +0300, Victor Goldenshtein wrote: > >> That brings up a question: Do we need to have this new function or would it > >> be feasible for cfg80211 to generate the intermediate reports > >> automatically based on cfg80211_inform_bss{,_frame}() calls during the > >> scan? > > > > Very good point! Thinking of it, there is not need to add this new > > function and have the drivers call it. cfg80211 can find out when to > > send the results by itself, whenever someone calls > > cfg80211_inform_bss_frame(), by checking whether intermediate results > > are enabled or not. > > > > This will simplify the code. :) Thanks for your comments! > > > > This could be a little bit problematic, since we need the > rdev->event_work, for the queue_work(), and wdev->event_list to save a > copy of the BSS, or just the mac which will be used to fetch the BSS > before the transmission (still not sure about it ..), and it seems like > these two can be obtained form sdata->dev, so it's look like that we > need this function and it should be in ieee80211_scan_rx. It is not an issue to send a genl message with GFP_ATOMIC. 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