diff options
| author | Thomas Graf <tgraf@suug.ch> | 2012-07-10 22:29:19 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-07-16 22:59:30 -0700 |
| commit | 036be6dbcfbe3da14be9b3463b94a5f79d8a2d71 (patch) | |
| tree | 6de042db9c434574e4d23a3a440001ab95df2b27 /net/bridge/br_multicast.c | |
| parent | 9c64508af2c171f57a198729c59a7a6f6998b3a1 (diff) | |
| download | op-kernel-dev-036be6dbcfbe3da14be9b3463b94a5f79d8a2d71.zip op-kernel-dev-036be6dbcfbe3da14be9b3463b94a5f79d8a2d71.tar.gz | |
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/br_multicast.c')
| -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 2d9a066..2417434 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( 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; |
