Tue, Oct 17, 2017 at 12:28:45PM CEST, lucien....@gmail.com wrote: >On Tue, Oct 17, 2017 at 5:59 PM, Jiri Pirko <j...@resnulli.us> wrote: >> Tue, Oct 17, 2017 at 11:39:38AM CEST, lucien....@gmail.com wrote: >>>It's better to send notifications to userspace by the events >>>in rtnetlink_event, instead of calling rtmsg_ifinfo directly. >>> >>>This patcheset is to remove rtmsg_ifinfo called in bonding, >>>the notifications can be handled by NETDEV_CHANGEUPPER and >>>NETDEV_CHANGELOWERSTATE events in rtnetlink_event. >>> >>>It could also fix some redundant notifications from bonding. >> >> This should go to net-next. > >NETDEV_CHANGEUPPER is not yet in rtnetlink_event in net-next tree. >patches can only work on net tree by now. > >Hi, David, you want me to hold them until the patches for NETDEV_CHANGEUPPER >are copied to net-next, or you would apply them to net ?
This patchset is not fix right? I see no "Fixes" line. You should wait until the dependency is merged to net-next, rebase, post for net-next. > >> >> >>> >>>Xin Long (3): >>> bonding: remove rtmsg_ifinfo called in bond_master_upper_dev_link >>> rtnetlink: bring NETDEV_CHANGELOWERSTATE event process back to >>> rtnetlink_event >>> bonding: remove rtmsg_ifinfo called after bond_lower_state_changed >>> >>> drivers/net/bonding/bond_main.c | 11 +++-------- >>> include/net/bonding.h | 4 ---- >>> net/core/rtnetlink.c | 2 +- >>> 3 files changed, 4 insertions(+), 13 deletions(-) >>> >>>-- >>>2.1.0 >>>
