Commit c5c23260 authored by Herbert Xu's avatar Herbert Xu Committed by David S. Miller

bridge: Add multicast_querier toggle and disable queries by default

Sending general queries was implemented as an optimisation to speed
up convergence on start-up.  In order to prevent interference with
multicast routers a zero source address has to be used.

Unfortunately these packets appear to cause some multicast-aware
switches to misbehave, e.g., by disrupting multicast packets to us.

Since the multicast snooping feature still functions without sending
our own queries, this patch will change the default to not send
queries.

For those that need queries in order to speed up convergence on start-up,
a toggle is provided to restore the previous behaviour.
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c83b8fab
...@@ -783,6 +783,7 @@ static void br_multicast_send_query(struct net_bridge *br, ...@@ -783,6 +783,7 @@ static void br_multicast_send_query(struct net_bridge *br,
struct br_ip br_group; struct br_ip br_group;
if (!netif_running(br->dev) || br->multicast_disabled || if (!netif_running(br->dev) || br->multicast_disabled ||
!br->multicast_querier ||
timer_pending(&br->multicast_querier_timer)) timer_pending(&br->multicast_querier_timer))
return; return;
...@@ -1565,6 +1566,7 @@ void br_multicast_init(struct net_bridge *br) ...@@ -1565,6 +1566,7 @@ void br_multicast_init(struct net_bridge *br)
br->hash_max = 512; br->hash_max = 512;
br->multicast_router = 1; br->multicast_router = 1;
br->multicast_querier = 0;
br->multicast_last_member_count = 2; br->multicast_last_member_count = 2;
br->multicast_startup_query_count = 2; br->multicast_startup_query_count = 2;
...@@ -1760,6 +1762,24 @@ int br_multicast_toggle(struct net_bridge *br, unsigned long val) ...@@ -1760,6 +1762,24 @@ int br_multicast_toggle(struct net_bridge *br, unsigned long val)
return err; return err;
} }
int br_multicast_set_querier(struct net_bridge *br, unsigned long val)
{
val = !!val;
spin_lock_bh(&br->multicast_lock);
if (br->multicast_querier == val)
goto unlock;
br->multicast_querier = val;
if (val)
br_multicast_start_querier(br);
unlock:
spin_unlock_bh(&br->multicast_lock);
return 0;
}
int br_multicast_set_hash_max(struct net_bridge *br, unsigned long val) int br_multicast_set_hash_max(struct net_bridge *br, unsigned long val)
{ {
int err = -ENOENT; int err = -ENOENT;
......
...@@ -224,6 +224,7 @@ struct net_bridge ...@@ -224,6 +224,7 @@ struct net_bridge
unsigned char multicast_router; unsigned char multicast_router;
u8 multicast_disabled:1; u8 multicast_disabled:1;
u8 multicast_querier:1;
u32 hash_elasticity; u32 hash_elasticity;
u32 hash_max; u32 hash_max;
...@@ -417,6 +418,7 @@ extern int br_multicast_set_router(struct net_bridge *br, unsigned long val); ...@@ -417,6 +418,7 @@ extern int br_multicast_set_router(struct net_bridge *br, unsigned long val);
extern int br_multicast_set_port_router(struct net_bridge_port *p, extern int br_multicast_set_port_router(struct net_bridge_port *p,
unsigned long val); unsigned long val);
extern int br_multicast_toggle(struct net_bridge *br, unsigned long val); extern int br_multicast_toggle(struct net_bridge *br, unsigned long val);
extern int br_multicast_set_querier(struct net_bridge *br, unsigned long val);
extern int br_multicast_set_hash_max(struct net_bridge *br, unsigned long val); extern int br_multicast_set_hash_max(struct net_bridge *br, unsigned long val);
static inline bool br_multicast_is_router(struct net_bridge *br) static inline bool br_multicast_is_router(struct net_bridge *br)
......
...@@ -379,6 +379,23 @@ static ssize_t store_multicast_snooping(struct device *d, ...@@ -379,6 +379,23 @@ static ssize_t store_multicast_snooping(struct device *d,
static DEVICE_ATTR(multicast_snooping, S_IRUGO | S_IWUSR, static DEVICE_ATTR(multicast_snooping, S_IRUGO | S_IWUSR,
show_multicast_snooping, store_multicast_snooping); show_multicast_snooping, store_multicast_snooping);
static ssize_t show_multicast_querier(struct device *d,
struct device_attribute *attr,
char *buf)
{
struct net_bridge *br = to_bridge(d);
return sprintf(buf, "%d\n", br->multicast_querier);
}
static ssize_t store_multicast_querier(struct device *d,
struct device_attribute *attr,
const char *buf, size_t len)
{
return store_bridge_parm(d, buf, len, br_multicast_set_querier);
}
static DEVICE_ATTR(multicast_querier, S_IRUGO | S_IWUSR,
show_multicast_querier, store_multicast_querier);
static ssize_t show_hash_elasticity(struct device *d, static ssize_t show_hash_elasticity(struct device *d,
struct device_attribute *attr, char *buf) struct device_attribute *attr, char *buf)
{ {
...@@ -702,6 +719,7 @@ static struct attribute *bridge_attrs[] = { ...@@ -702,6 +719,7 @@ static struct attribute *bridge_attrs[] = {
#ifdef CONFIG_BRIDGE_IGMP_SNOOPING #ifdef CONFIG_BRIDGE_IGMP_SNOOPING
&dev_attr_multicast_router.attr, &dev_attr_multicast_router.attr,
&dev_attr_multicast_snooping.attr, &dev_attr_multicast_snooping.attr,
&dev_attr_multicast_querier.attr,
&dev_attr_hash_elasticity.attr, &dev_attr_hash_elasticity.attr,
&dev_attr_hash_max.attr, &dev_attr_hash_max.attr,
&dev_attr_multicast_last_member_count.attr, &dev_attr_multicast_last_member_count.attr,
......
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