On Sat, 27 May 2017 10:14:36 -0400
Vladislav Yasevich <vyasev...@gmail.com> wrote:

>  
> +static const char *netdev_events[] = {"NONE",
> +                                   "REBOOT",
> +                                   "FEATURE CHANGE",
> +                                   "BONDING FAILOVER",
> +                                   "NOTIFY PEERS",
> +                                   "RESEND IGMP",
> +                                   "BONDING OPTION"};

Overall this looks fine, I will pickup the if_link.h from net-next.

One stylistic change.

Please add simple line break, and initialize by value:

static const char *netdev_events[] = {
        [IFLA_EVENT_NONE]       = "NONE",
...

Do you want some prefix or bounding around the event output?
Also a little concerned that the output format change may break some program
could the new output be at the end of the line?

Reply via email to