[PATCH v2 6/6] nl80211: Use global netlink objects.

Jouni Malinen j
Sat Oct 22 10:57:57 PDT 2011


On Thu, Oct 20, 2011 at 01:21:24PM -0700, greearb at candelatech.com wrote:
> Netlink sockets can be shared among all driver instances,
> saving lots of sockets, spurious log messages, memory,
> and CPU usage when using multiple interfaces in a single
> process.

I split this into pieces to make it easier to review and address
conflict with other pending work. The patches are here:
http://w1.fi/p/ben/
(the patch numbers below refer to these six patches and not the six that
happened to be in this series you sent to the list)

I applied 1/6 and 2/6 now, i.e., global nl_cb and nl/nl80211 for
commands.

5/6 is clearly in conflict with what Johannes is working on, so I don't
want to push that in at this point. And anyway, its effect should be
pretty minimal since Probe Request reporting is currently used only for
P2P Listen state and there is unlikely to be more than one of those
running (unless you start simulating large number of P2P devices, I'd
guess).

4/6 is somewhat less clear. I think we may end up having to split
process_event() into parts since it is now used for events that could be
global and for some cases like Action frames where it would likely
benefit from being per drv (or bss). I hope that Johannes can address
this in whatever way fits with his patch set. 3/6 is no-op preparation
for 4/6, so that is in the same category.

-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list