From patchwork Wed Feb 26 15:18:25 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vlad Yasevich X-Patchwork-Id: 324476 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 8E9592C009E for ; Thu, 27 Feb 2014 02:19:05 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752559AbaBZPTA (ORCPT ); Wed, 26 Feb 2014 10:19:00 -0500 Received: from mx1.redhat.com ([209.132.183.28]:20710 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751114AbaBZPSz (ORCPT ); Wed, 26 Feb 2014 10:18:55 -0500 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s1QFIp5V020200 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Wed, 26 Feb 2014 10:18:52 -0500 Received: from vyasevic.redhat.com (ovpn-113-57.phx2.redhat.com [10.3.113.57]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id s1QFIcpc003617; Wed, 26 Feb 2014 10:18:50 -0500 From: Vlad Yasevich To: netdev@vger.kernel.org Cc: bridge@lists.linux-foundation.org, shemminger@vyatta.com, mst@redhat.com, jhs@mojatatu.com, john.r.fastabend@intel.com, Vlad Yasevich Subject: [PATCH 7/7] bridge: Support promisc management when all ports are non-flooding Date: Wed, 26 Feb 2014 10:18:25 -0500 Message-Id: <1393427905-6811-8-git-send-email-vyasevic@redhat.com> In-Reply-To: <1393427905-6811-1-git-send-email-vyasevic@redhat.com> References: <1393427905-6811-1-git-send-email-vyasevic@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Configuration where all ports are set to non-flooding is a slight special case. In this config, the user would have to manage all fdbs for all ports. In this special case, since we'll know all addresses, we can turn off promisc on all ports and program all ports with the same set of addresses. Signed-off-by: Vlad Yasevich --- include/linux/netdevice.h | 3 +++ net/bridge/br_fdb.c | 22 +++++++++++++++++----- net/bridge/br_if.c | 40 ++++++++++++++++++++++++++++++++++------ net/bridge/br_private.h | 2 +- net/core/dev_addr_lists.c | 7 ++++--- 5 files changed, 59 insertions(+), 15 deletions(-) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index e29cce1..79e97ee 100644 --- a/include/linux/netdevice.h +++ b/include/linux/netdevice.h @@ -2889,6 +2889,9 @@ int __hw_addr_del(struct netdev_hw_addr_list *list, unsigned char addr_type); int __hw_addr_sync(struct netdev_hw_addr_list *to_list, struct netdev_hw_addr_list *from_list, int addr_len); +int __hw_addr_sync_multiple(struct netdev_hw_addr_list *to_list, + struct netdev_hw_addr_list *from_list, + int addr_len); void __hw_addr_unsync(struct netdev_hw_addr_list *to_list, struct netdev_hw_addr_list *from_list, int addr_len); void __hw_addr_init(struct netdev_hw_addr_list *list); diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c index ef95e81..26ea4fe 100644 --- a/net/bridge/br_fdb.c +++ b/net/bridge/br_fdb.c @@ -911,19 +911,31 @@ void br_fdb_addrs_sync(struct net_bridge *br) if (br->n_flood_ports == 1) { p = br->c_flood_port; netif_addr_lock(p->dev); - err = __hw_addr_sync(&p->dev->uc, &br->conf_addrs, - p->dev->addr_len); + err = __hw_addr_sync_multiple(&p->dev->uc, &br->conf_addrs, + p->dev->addr_len); if (!err) __dev_set_rx_mode(p->dev); netif_addr_unlock(p->dev); + } else if (br->n_flood_ports == 0) { + list_for_each_entry(p, &br->port_list, list) { + /* skip over ports being deleted. */ + if (!br_port_exists(p->dev)) + continue; + netif_addr_lock_nested(p->dev); + err = __hw_addr_sync_multiple(&p->dev->uc, + &br->conf_addrs, + p->dev->addr_len); + if (!err) + __dev_set_rx_mode(p->dev); + netif_addr_unlock(p->dev); + } } + } -void br_fdb_addrs_unsync(struct net_bridge *br) +void br_fdb_addrs_unsync(struct net_bridge *br, struct net_bridge_port *p) { - struct net_bridge_port *p = br->c_flood_port; - if (!p) return; diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c index 55e4e28..4ba62ef 100644 --- a/net/bridge/br_if.c +++ b/net/bridge/br_if.c @@ -148,6 +148,8 @@ static void del_nbp(struct net_bridge_port *p) if (p->flags & BR_FLOOD) br_del_flood_port(p, br); + else + br_fdb_addrs_unsync(br, p); nbp_vlan_flush(p); br_fdb_delete_by_port(br, p, 1); @@ -530,8 +532,26 @@ static void br_add_flood_port(struct net_bridge_port *p, struct net_bridge *br) * only have 1 flooding port cache if for future use. */ br->n_flood_ports++; - if (br->n_flood_ports == 1) + if (br->n_flood_ports == 1) { + struct net_bridge_port *port; + + /* We are transitioning from 0 flood ports to 1. Remove + * the addresses from all the non-flood ports and turn on + * promisc on those ports. + */ + list_for_each_entry(port, &br->port_list, list) { + /* skip the current port we are changing */ + if (port == p) + continue; + + if (!(port->flags & BR_FLOOD)) { + br_port_set_promisc(port); + br_fdb_addrs_unsync(br, port); + } + } + br->c_flood_port = p; + } br_fdb_addrs_sync(br); br_manage_promisc(br); @@ -547,12 +567,20 @@ static void br_del_flood_port(struct net_bridge_port *p, struct net_bridge *br) * set it if it is not set. */ br->n_flood_ports--; - if (p == br->c_flood_port) { - br_fdb_addrs_unsync(br); - br->c_flood_port = NULL; - } + if (br->n_flood_ports == 0) { + /* We just dropped to 0 flood ports. If we + * are deleting this port, unsync addresses + * from it. + */ + if (!br_port_exists(p->dev)) + br_fdb_addrs_unsync(br, p); - if (br->n_flood_ports == 1) { + br->c_flood_port = NULL; + br_fdb_addrs_sync(br); + } else if (br->n_flood_ports == 1) { + /* We just dropped to 1 flood port. Find this one flood + * port and sync to it. + */ list_for_each_entry(port, &p->br->port_list, list) { if (br_port_exists(port->dev) && (port->flags & BR_FLOOD)) { diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h index 87dcc09..13840de 100644 --- a/net/bridge/br_private.h +++ b/net/bridge/br_private.h @@ -400,7 +400,7 @@ int br_fdb_add(struct ndmsg *nlh, struct nlattr *tb[], struct net_device *dev, int br_fdb_dump(struct sk_buff *skb, struct netlink_callback *cb, struct net_device *dev, int idx); void br_fdb_addrs_sync(struct net_bridge *br); -void br_fdb_addrs_unsync(struct net_bridge *br); +void br_fdb_addrs_unsync(struct net_bridge *br, struct net_bridge_port *p); /* br_forward.c */ void br_deliver(const struct net_bridge_port *to, struct sk_buff *skb); diff --git a/net/core/dev_addr_lists.c b/net/core/dev_addr_lists.c index 3de44a3..24da78f 100644 --- a/net/core/dev_addr_lists.c +++ b/net/core/dev_addr_lists.c @@ -171,9 +171,9 @@ static void __hw_addr_unsync_one(struct netdev_hw_addr_list *to_list, __hw_addr_del_entry(from_list, ha, false, false); } -static int __hw_addr_sync_multiple(struct netdev_hw_addr_list *to_list, - struct netdev_hw_addr_list *from_list, - int addr_len) +int __hw_addr_sync_multiple(struct netdev_hw_addr_list *to_list, + struct netdev_hw_addr_list *from_list, + int addr_len) { int err = 0; struct netdev_hw_addr *ha, *tmp; @@ -189,6 +189,7 @@ static int __hw_addr_sync_multiple(struct netdev_hw_addr_list *to_list, } return err; } +EXPORT_SYMBOL(__hw_addr_sync_multiple); /* This function only works where there is a strict 1-1 relationship * between source and destionation of they synch. If you ever need to