Commit e028e4b8 authored by Vlad Yasevich's avatar Vlad Yasevich Committed by David S. Miller

bridge: Keep track of ports capable of automatic discovery.

By default, ports on the bridge are capable of automatic
discovery of nodes located behind the port.  This is accomplished
via flooding of unknown traffic (BR_FLOOD) and learning the
mac addresses from these packets (BR_LEARNING).
If the above functionality is disabled by turning off these
flags, the port requires static configuration in the form
of static FDB entries to function properly.

This patch adds functionality to keep track of all ports
capable of automatic discovery.  This will later be used
to control promiscuity settings.
Acked-by: default avatarMichael S. Tsirkin <mst@redhat.com>
Signed-off-by: default avatarVlad Yasevich <vyasevic@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 63c3a622
...@@ -85,6 +85,18 @@ void br_port_carrier_check(struct net_bridge_port *p) ...@@ -85,6 +85,18 @@ void br_port_carrier_check(struct net_bridge_port *p)
spin_unlock_bh(&br->lock); spin_unlock_bh(&br->lock);
} }
static void nbp_update_port_count(struct net_bridge *br)
{
struct net_bridge_port *p;
u32 cnt = 0;
list_for_each_entry(p, &br->port_list, list) {
if (br_auto_port(p))
cnt++;
}
br->auto_cnt = cnt;
}
static void release_nbp(struct kobject *kobj) static void release_nbp(struct kobject *kobj)
{ {
struct net_bridge_port *p struct net_bridge_port *p
...@@ -146,6 +158,8 @@ static void del_nbp(struct net_bridge_port *p) ...@@ -146,6 +158,8 @@ static void del_nbp(struct net_bridge_port *p)
list_del_rcu(&p->list); list_del_rcu(&p->list);
nbp_update_port_count(br);
dev->priv_flags &= ~IFF_BRIDGE_PORT; dev->priv_flags &= ~IFF_BRIDGE_PORT;
netdev_rx_handler_unregister(dev); netdev_rx_handler_unregister(dev);
...@@ -384,6 +398,8 @@ int br_add_if(struct net_bridge *br, struct net_device *dev) ...@@ -384,6 +398,8 @@ int br_add_if(struct net_bridge *br, struct net_device *dev)
list_add_rcu(&p->list, &br->port_list); list_add_rcu(&p->list, &br->port_list);
nbp_update_port_count(br);
netdev_update_features(br->dev); netdev_update_features(br->dev);
if (br->dev->needed_headroom < dev->needed_headroom) if (br->dev->needed_headroom < dev->needed_headroom)
...@@ -455,3 +471,11 @@ int br_del_if(struct net_bridge *br, struct net_device *dev) ...@@ -455,3 +471,11 @@ int br_del_if(struct net_bridge *br, struct net_device *dev)
return 0; return 0;
} }
void br_port_flags_change(struct net_bridge_port *p, unsigned long mask)
{
struct net_bridge *br = p->br;
if (mask & BR_AUTO_MASK)
nbp_update_port_count(br);
}
...@@ -328,6 +328,7 @@ static void br_set_port_flag(struct net_bridge_port *p, struct nlattr *tb[], ...@@ -328,6 +328,7 @@ static void br_set_port_flag(struct net_bridge_port *p, struct nlattr *tb[],
static int br_setport(struct net_bridge_port *p, struct nlattr *tb[]) static int br_setport(struct net_bridge_port *p, struct nlattr *tb[])
{ {
int err; int err;
unsigned long old_flags = p->flags;
br_set_port_flag(p, tb, IFLA_BRPORT_MODE, BR_HAIRPIN_MODE); br_set_port_flag(p, tb, IFLA_BRPORT_MODE, BR_HAIRPIN_MODE);
br_set_port_flag(p, tb, IFLA_BRPORT_GUARD, BR_BPDU_GUARD); br_set_port_flag(p, tb, IFLA_BRPORT_GUARD, BR_BPDU_GUARD);
...@@ -353,6 +354,8 @@ static int br_setport(struct net_bridge_port *p, struct nlattr *tb[]) ...@@ -353,6 +354,8 @@ static int br_setport(struct net_bridge_port *p, struct nlattr *tb[])
if (err) if (err)
return err; return err;
} }
br_port_flags_change(p, old_flags ^ p->flags);
return 0; return 0;
} }
......
...@@ -174,6 +174,7 @@ struct net_bridge_port ...@@ -174,6 +174,7 @@ struct net_bridge_port
#define BR_ADMIN_COST 0x00000010 #define BR_ADMIN_COST 0x00000010
#define BR_LEARNING 0x00000020 #define BR_LEARNING 0x00000020
#define BR_FLOOD 0x00000040 #define BR_FLOOD 0x00000040
#define BR_AUTO_MASK (BR_FLOOD | BR_LEARNING)
#ifdef CONFIG_BRIDGE_IGMP_SNOOPING #ifdef CONFIG_BRIDGE_IGMP_SNOOPING
struct bridge_mcast_query ip4_query; struct bridge_mcast_query ip4_query;
...@@ -198,6 +199,8 @@ struct net_bridge_port ...@@ -198,6 +199,8 @@ struct net_bridge_port
#endif #endif
}; };
#define br_auto_port(p) ((p)->flags & BR_AUTO_MASK)
#define br_port_exists(dev) (dev->priv_flags & IFF_BRIDGE_PORT) #define br_port_exists(dev) (dev->priv_flags & IFF_BRIDGE_PORT)
static inline struct net_bridge_port *br_port_get_rcu(const struct net_device *dev) static inline struct net_bridge_port *br_port_get_rcu(const struct net_device *dev)
...@@ -290,6 +293,7 @@ struct net_bridge ...@@ -290,6 +293,7 @@ struct net_bridge
struct timer_list topology_change_timer; struct timer_list topology_change_timer;
struct timer_list gc_timer; struct timer_list gc_timer;
struct kobject *ifobj; struct kobject *ifobj;
u32 auto_cnt;
#ifdef CONFIG_BRIDGE_VLAN_FILTERING #ifdef CONFIG_BRIDGE_VLAN_FILTERING
u8 vlan_enabled; u8 vlan_enabled;
struct net_port_vlans __rcu *vlan_info; struct net_port_vlans __rcu *vlan_info;
...@@ -415,6 +419,7 @@ int br_del_if(struct net_bridge *br, struct net_device *dev); ...@@ -415,6 +419,7 @@ int br_del_if(struct net_bridge *br, struct net_device *dev);
int br_min_mtu(const struct net_bridge *br); int br_min_mtu(const struct net_bridge *br);
netdev_features_t br_features_recompute(struct net_bridge *br, netdev_features_t br_features_recompute(struct net_bridge *br,
netdev_features_t features); netdev_features_t features);
void br_port_flags_change(struct net_bridge_port *port, unsigned long mask);
/* br_input.c */ /* br_input.c */
int br_handle_frame_finish(struct sk_buff *skb); int br_handle_frame_finish(struct sk_buff *skb);
......
...@@ -49,7 +49,9 @@ static BRPORT_ATTR(_name, S_IRUGO | S_IWUSR, \ ...@@ -49,7 +49,9 @@ static BRPORT_ATTR(_name, S_IRUGO | S_IWUSR, \
static int store_flag(struct net_bridge_port *p, unsigned long v, static int store_flag(struct net_bridge_port *p, unsigned long v,
unsigned long mask) unsigned long mask)
{ {
unsigned long flags = p->flags; unsigned long flags;
flags = p->flags;
if (v) if (v)
flags |= mask; flags |= mask;
...@@ -58,6 +60,7 @@ static int store_flag(struct net_bridge_port *p, unsigned long v, ...@@ -58,6 +60,7 @@ static int store_flag(struct net_bridge_port *p, unsigned long v,
if (flags != p->flags) { if (flags != p->flags) {
p->flags = flags; p->flags = flags;
br_port_flags_change(p, mask);
br_ifinfo_notify(RTM_NEWLINK, p); br_ifinfo_notify(RTM_NEWLINK, p);
} }
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment