diff options
author | Thomas Graf <tgraf@suug.ch> | 2012-07-10 18:29:19 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-07-17 01:59:30 -0400 |
commit | 036be6dbcfbe3da14be9b3463b94a5f79d8a2d71 (patch) | |
tree | 6de042db9c434574e4d23a3a440001ab95df2b27 /net/bridge | |
parent | 9c64508af2c171f57a198729c59a7a6f6998b3a1 (diff) |
bridge: Fix enforcement of multicast hash_max limit
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>
Diffstat (limited to 'net/bridge')
-rw-r--r-- | net/bridge/br_multicast.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/net/bridge/br_multicast.c b/net/bridge/br_multicast.c index 2d9a0663b848..241743417f49 100644 --- a/net/bridge/br_multicast.c +++ b/net/bridge/br_multicast.c | |||
@@ -540,10 +540,11 @@ static struct net_bridge_mdb_entry *br_multicast_get_group( | |||
540 | 540 | ||
541 | if (mdb->size >= max) { | 541 | if (mdb->size >= max) { |
542 | max *= 2; | 542 | max *= 2; |
543 | if (unlikely(max >= br->hash_max)) { | 543 | if (unlikely(max > br->hash_max)) { |
544 | br_warn(br, "Multicast hash table maximum " | 544 | br_warn(br, "Multicast hash table maximum of %d " |
545 | "reached, disabling snooping: %s, %d\n", | 545 | "reached, disabling snooping: %s\n", |
546 | port ? port->dev->name : br->dev->name, max); | 546 | br->hash_max, |
547 | port ? port->dev->name : br->dev->name); | ||
547 | err = -E2BIG; | 548 | err = -E2BIG; |
548 | disable: | 549 | disable: |
549 | br->multicast_disabled = 1; | 550 | br->multicast_disabled = 1; |