netdev - Re: [PATCH v2 net-next 3/8] bridge: Add functionality to sync static fdb entries to hw

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140515184344.GD1699@redhat.com>
Date:	Thu, 15 May 2014 21:43:44 +0300
From:	"Michael S. Tsirkin" <mst@...hat.com>
To:	Vlad Yasevich <vyasevic@...hat.com>
Cc:	netdev@...r.kernel.org, bridge@...ts.linux-foundation.org,
	shemminger@...tta.com, john.r.fastabend@...el.com, jhs@...atatu.com
Subject: Re: [PATCH v2 net-next 3/8] bridge: Add functionality to sync static
 fdb entries to hw

On Thu, May 15, 2014 at 12:56:51PM -0400, Vlad Yasevich wrote:
> Add code that allows static fdb entires to be synced to the
> hw list for a specified port.  This will be used later to
> program ports that can function in non-promiscuous mode.
> 
> Signed-off-by: Vlad Yasevich <vyasevic@...hat.com>

Acked-by: Michael S. Tsirkin <mst@...hat.com>

> ---
>  net/bridge/br_fdb.c     | 56 +++++++++++++++++++++++++++++++++++++++++++++++++
>  net/bridge/br_private.h |  2 ++
>  2 files changed, 58 insertions(+)
> 
> diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
> index 9203d5a..fe124e5 100644
> --- a/net/bridge/br_fdb.c
> +++ b/net/bridge/br_fdb.c
> @@ -874,3 +874,59 @@ int br_fdb_delete(struct ndmsg *ndm, struct nlattr *tb[],
>  out:
>  	return err;
>  }
> +
> +int br_fdb_sync_static(struct net_bridge *br, struct net_bridge_port *p)
> +{
> +	struct net_bridge_fdb_entry *fdb, *tmp;
> +	int i;
> +	int err;
> +
> +	ASSERT_RTNL();
> +
> +	for (i = 0; i < BR_HASH_SIZE; i++) {
> +		hlist_for_each_entry(fdb, &br->hash[i], hlist) {
> +			/* We only care for static entries */
> +			if (!fdb->is_static)
> +				continue;
> +
> +			err = dev_uc_add(p->dev, fdb->addr.addr);
> +			if (err)
> +				goto rollback;
> +		}
> +	}
> +	return 0;
> +
> +rollback:
> +	for (i = 0; i < BR_HASH_SIZE; i++) {
> +		hlist_for_each_entry(tmp, &br->hash[i], hlist) {
> +			/* If we reached the fdb that failed, we can stop */
> +			if (tmp == fdb)
> +				break;
> +
> +			/* We only care for static entries */
> +			if (!tmp->is_static)
> +				continue;
> +
> +			dev_uc_del(p->dev, tmp->addr.addr);
> +		}
> +	}
> +	return err;
> +}
> +
> +void br_fdb_unsync_static(struct net_bridge *br, struct net_bridge_port *p)
> +{
> +	struct net_bridge_fdb_entry *fdb;
> +	int i;
> +
> +	ASSERT_RTNL();
> +
> +	for (i = 0; i < BR_HASH_SIZE; i++) {
> +		hlist_for_each_entry_rcu(fdb, &br->hash[i], hlist) {
> +			/* We only care for static entries */
> +			if (!fdb->is_static)
> +				continue;
> +
> +			dev_uc_del(p->dev, fdb->addr.addr);
> +		}
> +	}
> +}
> diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h
> index 5ce3191..c0a804b 100644
> --- a/net/bridge/br_private.h
> +++ b/net/bridge/br_private.h
> @@ -399,6 +399,8 @@ int br_fdb_add(struct ndmsg *nlh, struct nlattr *tb[], struct net_device *dev,
>  	       const unsigned char *addr, u16 nlh_flags);
>  int br_fdb_dump(struct sk_buff *skb, struct netlink_callback *cb,
>  		struct net_device *dev, int idx);
> +int br_fdb_sync_static(struct net_bridge *br, struct net_bridge_port *p);
> +void br_fdb_unsync_static(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);
> -- 
> 1.9.0
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html