Re: [RFC 3/5] nl80211/cfg80211: adding intermediate scan result event.

[Johannes Berg <johannes@sipsolutions.net>]

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