Skip to content

net: remove rtmsg_ifinfo used in bridge and bonding

Xin Long netdev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v3 Latest4 patches 1 replyIn review

Changes in v3 from the cover letter

- add patch 1/4 to remove rtmsg_ifinfo called in add_del_if

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4bridge: remove rtmsg_ifinfo called in add_del_ifnet/bridge/br_ioctl.c+0 -30 replies
2/4bonding: remove rtmsg_ifinfo called in bond_master_upper_dev_linkdrivers/net/bonding/bond_main.c+3 -80 replies
3/4rtnetlink: bring NETDEV_CHANGELOWERSTATE event process back to rtnetlink_eventnet/core/rtnetlink.c+1 -00 replies
4/4bonding: remove rtmsg_ifinfo called after bond_lower_state_changedinclude/net/bonding.h net/core/rtnetlink.c+0 -50 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v3 yet.

Read the v3 thread

Cover letter [PATCHv3 net-next 0/4] net: remove rtmsg_ifinfo used in bridge and bonding

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 and bridge, the notifications can be handled by NETDEV_CHANGEUPPER and NETDEV_CHANGELOWERSTATE events in rtnetlink_event.

It could also fix some redundant notifications from bonding and bridge.

v1->v2:
- post to net-next.git instead of net.git, for it's more like an improvement for bonding
v2->v3:
- add patch 1/4 to remove rtmsg_ifinfo called in add_del_if

Xin Long (4):
bridge: remove rtmsg_ifinfo called in add_del_if
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/bridge/br_ioctl.c           |  3 ---
 net/core/rtnetlink.c            |  2 +-
 4 files changed, 4 insertions(+), 16 deletions(-)
Signature · 1 line
2.1.0