From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=NYsundaaKCSD/R6+ClUpACemZpnvgYxl80qw1WR3thQ=; b=flkHso79VcnuWgypsakhdssUaMieXkiEFhQLto0zMK5TIyM7C5jAXh1Jnxp5msMiP/ 3cj6So7X+SokkS9TcnlkNoC8r/zFv47kIxdl7za81T3p1mA7HQW8RXoyMybzouCZbQGz z9snyRhtWj6huF7tT4Xfbip4MgV6esRHypd2zetZIQ+B+dmoYYY6Xt2Wv288iHouzhKi E29cCpeQAMBCME6tbpz+9y+5twKTyu4AIiAgE6yQ+ZasJ6MCOjs/TfOWrcZbftEiNR5m KxBq6f27qMXguZCFgj3iXdlnIHfAep5oCEf6yi9ovZVqifpdRnHDKCqhtCQlBWfA9jBY yjDg== Sender: "Luis R. Rodriguez" From: "Luis R. Rodriguez" Date: Wed, 12 Mar 2014 20:15:26 -0700 Message-Id: <1394680527-28970-3-git-send-email-mcgrof@do-not-panic.com> In-Reply-To: <1394680527-28970-1-git-send-email-mcgrof@do-not-panic.com> References: <1394680527-28970-1-git-send-email-mcgrof@do-not-panic.com> Subject: [Bridge] [PATCH 2/3] bridge: trigger a bridge calculation upon port changes List-Id: Linux Ethernet Bridging List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: netdev@vger.kernel.org Cc: kvm@vger.kernel.org, mcgrof@suse.com, bridge@lists.linux-foundation.org, linux-kernel@vger.kernel.org, Stephen Hemminger , xen-devel@lists.xenproject.org From: "Luis R. Rodriguez" If netlink is used to tune a port we currently don't trigger a new recalculation of the bridge id, ensure that happens just as if we're adding a new net_device onto the bridge. Cc: Stephen Hemminger Cc: bridge@lists.linux-foundation.org Cc: netdev@vger.kernel.org Cc: linux-kernel@vger.kernel.org Cc: xen-devel@lists.xenproject.org Cc: kvm@vger.kernel.org Signed-off-by: Luis R. Rodriguez --- net/bridge/br_netlink.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c index e74b6d53..6f1b26d 100644 --- a/net/bridge/br_netlink.c +++ b/net/bridge/br_netlink.c @@ -364,6 +364,7 @@ int br_setlink(struct net_device *dev, struct nlmsghdr *nlh) struct net_bridge_port *p; struct nlattr *tb[IFLA_BRPORT_MAX + 1]; int err = 0; + bool changed; protinfo = nlmsg_find_attr(nlh, sizeof(struct ifinfomsg), IFLA_PROTINFO); afspec = nlmsg_find_attr(nlh, sizeof(struct ifinfomsg), IFLA_AF_SPEC); @@ -386,7 +387,12 @@ int br_setlink(struct net_device *dev, struct nlmsghdr *nlh) spin_lock_bh(&p->br->lock); err = br_setport(p, tb); + changed = br_stp_recalculate_bridge_id(p->br); spin_unlock_bh(&p->br->lock); + if (changed) + call_netdevice_notifiers(NETDEV_CHANGEADDR, + p->br->dev); + netdev_update_features(p->br->dev); } else { /* Binary compatibility with old RSTP */ if (nla_len(protinfo) < sizeof(u8)) -- 1.8.5.3