[PATCH 3/4] notify: Do not raise any signal from a P2P management interface
Tomasz Bursztyka
tomasz.bursztyka
Thu Oct 23 01:54:27 PDT 2014
Hi,
>> Such interface is not registered on DBus, thus there is no need to raise
>> any signal from it.
>
> Could this be done in one place instead of sprinkling these if
> statements?
I tried to find such centralization place with no luck. Maybe I missed it.
It follows what have been done already. If you take a look at dbus
signal handlers,
they will all verify that the dbus iface is up (so basically that the
DBusConnection is set).
Same for many object path on some parts etc...
Anyway, since the P2P mgmt interface is not really differentiated from
other interface,
though there are huge difference (it's not a netdev on first place),
it's instantiated as a
wpa_supplicant struct, which makes such notification verification hard
to centralize.
Afaik, unless a big rewrite, this is not possible.
Either we inhibit those notifications for the P2P dev from the code that
calls them,
the dbus handlers or the notification code itself. I choose the later one.
Tomasz
More information about the Hostap
mailing list