aboutsummaryrefslogtreecommitdiffstats
path: root/target/linux/generic/patches-3.18/051-02-bridge-allow-setting-hash_max-multicast_router-if-in.patch
diff options
context:
space:
mode:
authorFelix Fietkau <nbd@openwrt.org>2015-05-27 15:20:04 +0000
committerFelix Fietkau <nbd@openwrt.org>2015-05-27 15:20:04 +0000
commitb810acd44b7b458ae62a3cbf4cf418a7db78e1e7 (patch)
treecf3968b0e9505da05b0a8bde94188bf5215706fa /target/linux/generic/patches-3.18/051-02-bridge-allow-setting-hash_max-multicast_router-if-in.patch
parent0c4d4b2013da0d5f997f78a911f5c3466f6dcf1a (diff)
downloadmaster-187ad058-b810acd44b7b458ae62a3cbf4cf418a7db78e1e7.tar.gz
master-187ad058-b810acd44b7b458ae62a3cbf4cf418a7db78e1e7.tar.bz2
master-187ad058-b810acd44b7b458ae62a3cbf4cf418a7db78e1e7.zip
kernel: bridge: backport two snooping related patches
With this patch the following two patches are backported: * bridge: fix parsing of MLDv2 reports * bridge: allow setting hash_max + multicast_router if interface is down The former one is an important fix which got just applied to the net-tree and is queued for stable. The latter is a patch which is needed to make the hash_max and multicast_router attributes configurable through netifd. Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue> git-svn-id: svn://svn.openwrt.org/openwrt/trunk@45783 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target/linux/generic/patches-3.18/051-02-bridge-allow-setting-hash_max-multicast_router-if-in.patch')
-rw-r--r--target/linux/generic/patches-3.18/051-02-bridge-allow-setting-hash_max-multicast_router-if-in.patch99
1 files changed, 99 insertions, 0 deletions
diff --git a/target/linux/generic/patches-3.18/051-02-bridge-allow-setting-hash_max-multicast_router-if-in.patch b/target/linux/generic/patches-3.18/051-02-bridge-allow-setting-hash_max-multicast_router-if-in.patch
new file mode 100644
index 0000000000..c899c10cff
--- /dev/null
+++ b/target/linux/generic/patches-3.18/051-02-bridge-allow-setting-hash_max-multicast_router-if-in.patch
@@ -0,0 +1,99 @@
+From 6ae4ae8e512bd229f806c22f8a2cd751e4f987c2 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Linus=20L=C3=BCssing?= <linus.luessing@c0d3.blue>
+Date: Sat, 23 May 2015 03:12:34 +0200
+Subject: [PATCH] bridge: allow setting hash_max + multicast_router if
+ interface is down
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Network managers like netifd (used in OpenWRT for instance) try to
+configure interface options after creation but before setting the
+interface up.
+
+Unfortunately the sysfs / bridge currently only allows to configure the
+hash_max and multicast_router options when the bridge interface is up.
+But since br_multicast_init() doesn't start any timers and only sets
+default values and initializes timers it should be save to reconfigure
+the default values after that, before things actually get active after
+the bridge is set up.
+
+Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
+Signed-off-by: David S. Miller <davem@davemloft.net>
+---
+ net/bridge/br_multicast.c | 24 +++---------------------
+ 1 file changed, 3 insertions(+), 21 deletions(-)
+
+--- a/net/bridge/br_multicast.c
++++ b/net/bridge/br_multicast.c
+@@ -1949,11 +1949,9 @@ out:
+
+ int br_multicast_set_router(struct net_bridge *br, unsigned long val)
+ {
+- int err = -ENOENT;
++ int err = -EINVAL;
+
+ spin_lock_bh(&br->multicast_lock);
+- if (!netif_running(br->dev))
+- goto unlock;
+
+ switch (val) {
+ case 0:
+@@ -1964,13 +1962,8 @@ int br_multicast_set_router(struct net_b
+ br->multicast_router = val;
+ err = 0;
+ break;
+-
+- default:
+- err = -EINVAL;
+- break;
+ }
+
+-unlock:
+ spin_unlock_bh(&br->multicast_lock);
+
+ return err;
+@@ -1979,11 +1972,9 @@ unlock:
+ int br_multicast_set_port_router(struct net_bridge_port *p, unsigned long val)
+ {
+ struct net_bridge *br = p->br;
+- int err = -ENOENT;
++ int err = -EINVAL;
+
+ spin_lock(&br->multicast_lock);
+- if (!netif_running(br->dev) || p->state == BR_STATE_DISABLED)
+- goto unlock;
+
+ switch (val) {
+ case 0:
+@@ -2005,13 +1996,8 @@ int br_multicast_set_port_router(struct
+
+ br_multicast_add_router(br, p);
+ break;
+-
+- default:
+- err = -EINVAL;
+- break;
+ }
+
+-unlock:
+ spin_unlock(&br->multicast_lock);
+
+ return err;
+@@ -2116,15 +2102,11 @@ unlock:
+
+ int br_multicast_set_hash_max(struct net_bridge *br, unsigned long val)
+ {
+- int err = -ENOENT;
++ int err = -EINVAL;
+ u32 old;
+ struct net_bridge_mdb_htable *mdb;
+
+ spin_lock_bh(&br->multicast_lock);
+- if (!netif_running(br->dev))
+- goto unlock;
+-
+- err = -EINVAL;
+ if (!is_power_of_2(val))
+ goto unlock;
+