From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [PATCH net 0/3] bonding: void calling rtmsg_ifinfo for netlink notifications Date: Tue, 17 Oct 2017 11:59:43 +0200 Message-ID: <20171017095943.GE2112@nanopsycho> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: network dev , davem@davemloft.net To: Xin Long Return-path: Received: from mail-wr0-f193.google.com ([209.85.128.193]:48419 "EHLO mail-wr0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933334AbdJQJ7p (ORCPT ); Tue, 17 Oct 2017 05:59:45 -0400 Received: by mail-wr0-f193.google.com with SMTP id u5so1024508wrc.5 for ; Tue, 17 Oct 2017 02:59:44 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Tue, Oct 17, 2017 at 11:39:38AM CEST, lucien.xin@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. > >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 >