Thu, Apr 27, 2017 at 09:59:38PM CEST, d...@cumulusnetworks.com wrote: >On 4/27/17 1:43 PM, Vlad Yasevich wrote: >>> For example, NETDEV_CHANGEINFODATA is only for bonds though nothing >>> about the name suggests it is a bonding notification. This one was added >>> specifically to notify userspace (d4261e5650004), yet seems to happen >>> only during a changelink and that already generates a RTM_NEWLINK >>> message via do_setlink. Since the rtnetlink_event message does not >>> contain anything "NETDEV_CHANGEINFODATA" related what purpose does it >>> really serve besides duplicating netlink messages to userspace. >>> >> >> I am not sure about this one, but if you have an app trying to monitor >> for this event, it can't really since there is no info in the netlink >> message. > >I cc'ed Jiri on this thread hoping he would explain the intent. > >I propose it gets removed.
Hmm, I don't really recall. But looking at it now, I agree it is redundant.