[PATCH v2] supplicant: Print human readable event names.
j at w1.fi
Wed Nov 9 09:26:37 EST 2011
On Mon, Nov 07, 2011 at 02:21:02PM -0800, greearb at candelatech.com wrote:
> This makes it easier to understand the event related
I can understand this part..
> Also, print re-entrancy depth in the event handler.
> I'm suspicious that it could be called in a re-entrant
> manner if netlink events are received & processed as
> part of another event.
This sounds strange. I would recommend splitting this part to a separate
patch if you want to get the human readable part in.. I would need to
hear quite a bit more convincing explanation on how you think this
function can be used in re-entrant manner in a single process
implementation with eloop.
> diff --git a/src/common/wpa_common.c b/src/common/wpa_common.c
> +#include "drivers/driver.h"
This wpa_common.c file should not really use driver interface.
> +const char* event_to_string(enum wpa_event_type event)
And this does not belong here either.. wpa_common.c is a set of common
helper functions for WPA EAPOL-Key processing. Driver events have
nothing to do with it. If you are using this only in
wpa_supplicant/events.c, it would be better to add the function there.
If there is a plan to have this shared with hostapd, a new C file in
src/drivers could be the best approach.
Jouni Malinen PGP id EFC895FA
More information about the HostAP