{"id":814390,"url":"http://patchwork.ozlabs.org/api/patches/814390/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/patch/20170915193808.22338-1-vincent@bernat.im/","project":{"id":7,"url":"http://patchwork.ozlabs.org/api/projects/7/?format=json","name":"Linux network development","link_name":"netdev","list_id":"netdev.vger.kernel.org","list_email":"netdev@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170915193808.22338-1-vincent@bernat.im>","list_archive_url":null,"date":"2017-09-15T19:38:08","name":"[net-next,v1] bridge: also trigger RTM_NEWLINK when interface is released from bridge","commit_ref":null,"pull_url":null,"state":"superseded","archived":true,"hash":"4b7f1f893810753ba8ba701b1f022bbaa1752cd1","submitter":{"id":64027,"url":"http://patchwork.ozlabs.org/api/people/64027/?format=json","name":"Vincent Bernat","email":"vincent@bernat.im"},"delegate":{"id":34,"url":"http://patchwork.ozlabs.org/api/users/34/?format=json","username":"davem","first_name":"David","last_name":"Miller","email":"davem@davemloft.net"},"mbox":"http://patchwork.ozlabs.org/project/netdev/patch/20170915193808.22338-1-vincent@bernat.im/mbox/","series":[{"id":3375,"url":"http://patchwork.ozlabs.org/api/series/3375/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/list/?series=3375","date":"2017-09-15T19:38:08","name":"[net-next,v1] bridge: also trigger RTM_NEWLINK when interface is released from bridge","version":1,"mbox":"http://patchwork.ozlabs.org/series/3375/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/814390/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/814390/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=)","ozlabs.org; dkim=pass (1024-bit key;\n\tunprotected) header.d=bernat.im header.i=@bernat.im\n\theader.b=\"SppSkT4r\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xv5X05s9Hz9s7c\n\tfor ;\n\tSat, 16 Sep 2017 05:47:12 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751545AbdIOTrK (ORCPT );\n\tFri, 15 Sep 2017 15:47:10 -0400","from bart.luffy.cx ([78.47.78.131]:52850 \"EHLO bart.luffy.cx\"\n\trhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP\n\tid S1751259AbdIOTrJ (ORCPT );\n\tFri, 15 Sep 2017 15:47:09 -0400","from bart.luffy.cx (localhost [127.0.0.1])\n\tby bart.luffy.cx (Postfix) with ESMTP id 5BA72141C1;\n\tFri, 15 Sep 2017 21:38:16 +0200 (CEST)","from neo.luffy.cx (184.36.105.92.dynamic.wline.res.cust.swisscom.ch\n\t[92.105.36.184])\n\tby bart.luffy.cx (Postfix) with ESMTPS id 2EA941419C;\n\tFri, 15 Sep 2017 21:38:16 +0200 (CEST)","by neo.luffy.cx (Postfix, from userid 500)\n\tid B498EA62; Fri, 15 Sep 2017 21:38:15 +0200 (CEST)"],"X-Greylist":"delayed 531 seconds by postgrey-1.27 at vger.kernel.org;\n\tFri, 15 Sep 2017 15:47:08 EDT","DKIM-Signature":"v=1; a=rsa-sha1; c=relaxed; d=bernat.im; h=from:to:cc\n\t:subject:date:message-id; s=postfix; bh=2oonJjpyaies3zycCLAbM06l\n\tNTo=; b=SppSkT4rgmEGCqZxVr4RBP3LsCt48w2WZJSW1oABa2xdA+H/GLheoDhQ\n\ty7vopao1SJlPRFtaurAeYsZcpySNv4d1KuoChmlas/UIMIRJUFw0Ze0fatlI22Gb\n\tXBScFBjj+91LlwzC/UexSs6UmbV1kJz3gskOhT/XgWNgAmbbaMM=","DomainKey-Signature":"a=rsa-sha1; c=nofws; d=bernat.im; h=from:to:cc\n\t:subject:date:message-id; q=dns; s=postfix; b=YWvx/9HL5L1pPILJpY\n\t9Ew34nQalk6GnmxOVldiIex3RBhRR3zo7xjWHlR1qFFCrnNWubVMwouregKBt+Ck\n\thzZ3X3gF4qOV3CK95S+oGeMhjsPq+h5qJEcK+CmJMI00cidIJpH/2rLhbvcZhKyE\n\tdnVZ5hH34lzZ/sdgmTBlX4QhA=","From":"Vincent Bernat ","To":"Stephen Hemminger ,\n\t\"David S. Miller\" ,\n\tbridge@lists.linux-foundation.org, netdev@vger.kernel.org","Cc":"Vincent Bernat ","Subject":"[PATCH net-next v1] bridge: also trigger RTM_NEWLINK when interface\n\tis released from bridge","Date":"Fri, 15 Sep 2017 21:38:08 +0200","Message-Id":"<20170915193808.22338-1-vincent@bernat.im>","X-Mailer":"git-send-email 2.14.1","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"netdev@vger.kernel.org"},"content":"Currently, when an interface is released from a bridge, we get a\nRTM_DELLINK event through netlink:\n\nDeleted 2: dummy0: mtu 1500 master bridge0 state UNKNOWN\n link/ether 6e:23:c2:54:3a:b3\n\nUserspace has to interpret that as a removal from the bridge, not as a\ncomplete removal of the interface. When an bridged interface is\ncompletely removed, we get two events:\n\nDeleted 2: dummy0: mtu 1500 master bridge0 state DOWN\n link/ether 6e:23:c2:54:3a:b3\nDeleted 2: dummy0: mtu 1500 qdisc noop state DOWN group default\n link/ether 6e:23:c2:54:3a:b3 brd ff:ff:ff:ff:ff:ff\n\nIn constrast, when an interface is released from a bond, we get a\nRTM_NEWLINK with only the new characteristics (no master):\n\n3: dummy1: mtu 1500 qdisc noqueue master bond0 state UNKNOWN group default\n link/ether ae:dc:7a:8c:9a:3c brd ff:ff:ff:ff:ff:ff\n3: dummy1: mtu 1500 qdisc noqueue state UNKNOWN group default\n link/ether ae:dc:7a:8c:9a:3c brd ff:ff:ff:ff:ff:ff\n4: bond0: mtu 1500 qdisc noqueue state UP group default\n link/ether ae:dc:7a:8c:9a:3c brd ff:ff:ff:ff:ff:ff\n3: dummy1: mtu 1500 qdisc noqueue state DOWN group default\n link/ether ae:dc:7a:8c:9a:3c brd ff:ff:ff:ff:ff:ff\n3: dummy1: mtu 1500 qdisc noqueue state DOWN group default\n link/ether ca:c8:7b:66:f8:25 brd ff:ff:ff:ff:ff:ff\n4: bond0: mtu 1500 qdisc noqueue state UP group default\n link/ether ae:dc:7a:8c:9a:3c brd ff:ff:ff:ff:ff:ff\n\nUserland may be confused by the fact we say a link is deleted while\nits characteristics are only modified. A first solution would have\nbeen to turn the RTM_DELLINK event in del_nbp() into a RTM_NEWLINK\nevent. However, maybe some piece of userland is relying on this\nRTM_DELLINK to detect when a bridged interface is released. Instead,\nwe also emit a RTM_NEWLINK event once the interface is\nreleased (without master info).\n\nDeleted 2: dummy0: mtu 1500 master bridge0 state UNKNOWN\n link/ether 8a:bb:e7:94:b1:f8\n2: dummy0: mtu 1500 qdisc noqueue state UNKNOWN group default\n link/ether 8a:bb:e7:94:b1:f8 brd ff:ff:ff:ff:ff:ff\n\nSigned-off-by: Vincent Bernat \n---\n net/bridge/br_if.c | 2 ++\n 1 file changed, 2 insertions(+)","diff":"diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c\nindex f3aef22931ab..636e0a842f8a 100644\n--- a/net/bridge/br_if.c\n+++ b/net/bridge/br_if.c\n@@ -288,6 +288,8 @@ static void del_nbp(struct net_bridge_port *p)\n \n \tdev->priv_flags &= ~IFF_BRIDGE_PORT;\n \n+\trtmsg_ifinfo(RTM_NEWLINK, dev, IFF_MASTER, GFP_KERNEL);\n+\n \tnetdev_rx_handler_unregister(dev);\n \n \tbr_multicast_del_port(p);\n","prefixes":["net-next","v1"]}