bridge: Fix enforcement of multicast hash_max limit
authorThomas Graf <tgraf@suug.ch>
Tue, 10 Jul 2012 22:29:19 +0000 (22:29 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 17 Jul 2012 05:59:30 +0000 (22:59 -0700)
The hash size is doubled when it needs to grow and compared against
hash_max. The >= comparison will limit the hash table size to half
of what is expected i.e. the default 512 hash_max will not allow
the hash table to grow larger than 256.

Also print the hash table limit instead of the desirable size when
the limit is reached.

Signed-off-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_multicast.c

index 2d9a0663b848f10df87f6a6be1b516c1877b0136..241743417f498bdc9c8c63fe2500a78f9a4d2126 100644 (file)
@@ -540,10 +540,11 @@ static struct net_bridge_mdb_entry *br_multicast_get_group(
 
        if (mdb->size >= max) {
                max *= 2;
-               if (unlikely(max >= br->hash_max)) {
-                       br_warn(br, "Multicast hash table maximum "
-                               "reached, disabling snooping: %s, %d\n",
-                               port ? port->dev->name : br->dev->name, max);
+               if (unlikely(max > br->hash_max)) {
+                       br_warn(br, "Multicast hash table maximum of %d "
+                               "reached, disabling snooping: %s\n",
+                               br->hash_max,
+                               port ? port->dev->name : br->dev->name);
                        err = -E2BIG;
 disable:
                        br->multicast_disabled = 1;