Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/814390/?format=api
{ "id": 814390, "url": "http://patchwork.ozlabs.org/api/patches/814390/?format=api", "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=api", "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=api", "name": "Vincent Bernat", "email": "vincent@bernat.im" }, "delegate": { "id": 34, "url": "http://patchwork.ozlabs.org/api/users/34/?format=api", "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=api", "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": "<netdev-owner@vger.kernel.org>", "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=<UNKNOWN>)", "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 <patchwork-incoming@ozlabs.org>;\n\tSat, 16 Sep 2017 05:47:12 +1000 (AEST)", "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751545AbdIOTrK (ORCPT <rfc822;patchwork-incoming@ozlabs.org>);\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 <rfc822;netdev@vger.kernel.org>);\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 <vincent@bernat.im>", "To": "Stephen Hemminger <stephen@networkplumber.org>,\n\t\"David S. Miller\" <davem@davemloft.net>,\n\tbridge@lists.linux-foundation.org, netdev@vger.kernel.org", "Cc": "Vincent Bernat <vincent@bernat.im>", "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": "<netdev.vger.kernel.org>", "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: <BROADCAST,NOARP,UP,LOWER_UP> 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: <BROADCAST,NOARP> mtu 1500 master bridge0 state DOWN\n link/ether 6e:23:c2:54:3a:b3\nDeleted 2: dummy0: <BROADCAST,NOARP> 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: <BROADCAST,NOARP,SLAVE,UP,LOWER_UP> 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: <BROADCAST,NOARP,UP,LOWER_UP> 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: <BROADCAST,MULTICAST,MASTER,UP,LOWER_UP> 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: <BROADCAST,NOARP> 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: <BROADCAST,NOARP> 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: <BROADCAST,MULTICAST,MASTER,UP,LOWER_UP> 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: <BROADCAST,NOARP,UP,LOWER_UP> mtu 1500 master bridge0 state UNKNOWN\n link/ether 8a:bb:e7:94:b1:f8\n2: dummy0: <BROADCAST,NOARP,UP,LOWER_UP> 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 <vincent@bernat.im>\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" ] }