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?