bridge: allow setting hash_max value
authorLinus Lüssing <linus.luessing@c0d3.blue>
Tue, 26 May 2015 19:49:11 +0000 (21:49 +0200)
committerFelix Fietkau <nbd@openwrt.org>
Wed, 27 May 2015 15:02:41 +0000 (17:02 +0200)
If the number of entries in the MDB exceeds hash_max then the
multicast snooping capabilities of the bridge are disabled
automatically.

The default value for hash_max is 512 which is already exceeded by some
wireless community mesh networks. They need to be able to set a higher
value.

Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
bridge.c
system-linux.c
system.h

index fd553dcfac63f5127821ac94164d9386386350aa..9da531464f0bf16fb9f6d3bb27458d73d0542e69 100644 (file)
--- a/bridge.c
+++ b/bridge.c
@@ -33,6 +33,7 @@ enum {
        BRIDGE_ATTR_MAX_AGE,
        BRIDGE_ATTR_BRIDGE_EMPTY,
        BRIDGE_ATTR_MULTICAST_QUERIER,
+       BRIDGE_ATTR_HASH_MAX,
        __BRIDGE_ATTR_MAX
 };
 
@@ -47,6 +48,7 @@ static const struct blobmsg_policy bridge_attrs[__BRIDGE_ATTR_MAX] = {
        [BRIDGE_ATTR_IGMP_SNOOP] = { "igmp_snooping", BLOBMSG_TYPE_BOOL },
        [BRIDGE_ATTR_BRIDGE_EMPTY] = { "bridge_empty", BLOBMSG_TYPE_BOOL },
        [BRIDGE_ATTR_MULTICAST_QUERIER] = { "multicast_querier", BLOBMSG_TYPE_BOOL },
+       [BRIDGE_ATTR_HASH_MAX] = { "hash_max", BLOBMSG_TYPE_INT32 },
 };
 
 static const struct uci_blob_param_info bridge_attr_info[__BRIDGE_ATTR_MAX] = {
@@ -551,6 +553,7 @@ bridge_apply_settings(struct bridge_state *bst, struct blob_attr **tb)
        cfg->forward_delay = 2;
        cfg->igmp_snoop = true;
        cfg->multicast_querier = true;
+       cfg->hash_max = 512;
        cfg->bridge_empty = false;
        cfg->priority = 0x7FFF;
 
@@ -569,6 +572,9 @@ bridge_apply_settings(struct bridge_state *bst, struct blob_attr **tb)
        if ((cur = tb[BRIDGE_ATTR_MULTICAST_QUERIER]))
                cfg->multicast_querier = blobmsg_get_bool(cur);
 
+       if ((cur = tb[BRIDGE_ATTR_HASH_MAX]))
+               cfg->hash_max = blobmsg_get_u32(cur);
+
        if ((cur = tb[BRIDGE_ATTR_AGEING_TIME])) {
                cfg->ageing_time = blobmsg_get_u32(cur);
                cfg->flags |= BRIDGE_OPT_AGEING_TIME;
index 0ba729e826fc584a652bbb55a81eafbe5d8297ef..6dc9acd82db704573a54a4ca53cd43ee722e7a1b 100644 (file)
@@ -791,6 +791,7 @@ sec_to_jiffies(int val)
 
 int system_bridge_addbr(struct device *bridge, struct bridge_config *cfg)
 {
+       char buf[64];
        unsigned long args[4] = {};
 
        if (ioctl(sock_ioctl, SIOCBRADDBR, bridge->ifname) < 0)
@@ -810,6 +811,10 @@ int system_bridge_addbr(struct device *bridge, struct bridge_config *cfg)
        system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_querier",
                bridge->ifname, cfg->multicast_querier ? "1" : "0");
 
+       snprintf(buf, sizeof(buf), "%i", cfg->hash_max);
+       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/hash_max",
+               bridge->ifname, buf);
+
        args[0] = BRCTL_SET_BRIDGE_PRIORITY;
        args[1] = cfg->priority;
        system_bridge_if(bridge->ifname, NULL, SIOCDEVPRIVATE, &args);
index 382c907d64d630e54ba24001f135e394b7e2e1d5..486fe1423bedbf27bd8ef72d2161647258591901 100644 (file)
--- a/system.h
+++ b/system.h
@@ -58,6 +58,7 @@ struct bridge_config {
        int ageing_time;
        int hello_time;
        int max_age;
+       int hash_max;
 };
 
 enum macvlan_opt {