Re: [PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-26 Thread Arend van Spriel
On 4/26/2017 11:08 AM, Johannes Berg wrote: On Wed, 2017-04-26 at 11:05 +0200, Arend van Spriel wrote: I guess it could be done, but I'm tending towards doing that less again, I have no strong preference in this case. If it was something that seems likely to get abused, perhaps, but here...

Re: [PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-26 Thread Arend van Spriel
On 4/26/2017 10:49 AM, Johannes Berg wrote: FWIW: https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211-next.git/commit/?id=978812b3f43dda86e20588cfe0a4a244c4fab141 On Wed, 2017-04-26 at 10:46 +0200, Arend van Spriel wrote: Sure. In the worker we are already iterating so I guess

Re: [PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-26 Thread Johannes Berg
On Wed, 2017-04-26 at 11:05 +0200, Arend van Spriel wrote: > I guess it could be done, but I'm tending towards doing that less > > again, I have no strong preference in this case. If it was > > something > > that seems likely to get abused, perhaps, but here... > > Ok. As long as drivers don't

Re: [PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-26 Thread Johannes Berg
FWIW: https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211-next.git/commit/?id=978812b3f43dda86e20588cfe0a4a244c4fab141 On Wed, 2017-04-26 at 10:46 +0200, Arend van Spriel wrote: > Sure. In the worker we are already iterating so I guess we could. > The number of internal fields in

Re: [PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-26 Thread Arend van Spriel
On 4/25/2017 9:49 PM, Johannes Berg wrote: if (schedule_destroy_work) { - struct cfg80211_iface_destroy *destroy; + struct cfg80211_nlport_release *destroy; destroy = kzalloc(sizeof(*destroy), GFP_ATOMIC); I was never

Re: [PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-26 Thread Johannes Berg
On Tue, 2017-04-25 at 21:49 +0200, Johannes Berg wrote: > >   if (schedule_destroy_work) { > > - struct cfg80211_iface_destroy *destroy; > > + struct cfg80211_nlport_release *destroy; > >   > >   destroy = kzalloc(sizeof(*destroy), >

Re: [PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-25 Thread Johannes Berg
>   if (schedule_destroy_work) { > - struct cfg80211_iface_destroy *destroy; > + struct cfg80211_nlport_release *destroy; >   >   destroy = kzalloc(sizeof(*destroy), > GFP_ATOMIC); I was never really happy with this

[PATCH V3 1/9] nl80211: allow multiple active scheduled scan requests

2017-04-21 Thread Arend van Spriel
This patch implements the idea to have multiple scheduled scan requests running concurrently. It mainly illustrates how to deal with the incoming request from user-space in terms of backward compatibility. In order to use multiple scheduled scans user-space needs to provide a flag attribute