]> git.baikalelectronics.ru Git - kernel.git/commit
bridge: update mdb expiration timer upon reports.
authorVlad Yasevich <vyasevic@redhat.com>
Thu, 10 Oct 2013 19:57:59 +0000 (15:57 -0400)
committerDavid S. Miller <davem@davemloft.net>
Fri, 11 Oct 2013 04:56:47 +0000 (00:56 -0400)
commit56f1d3fc9ac8b2c38c74e963544ec88077f32eb9
treec6ed51aaaa66e55baeebd88f3c76f9c1552cc73b
parentaf301ffbb6588c7f0763d6402cf61547d75d0c5b
bridge: update mdb expiration timer upon reports.

commit 13ec12fbec01d5f0394136f6f514c01aa8a14d9c
bridge: only expire the mdb entry when query is received
changed the mdb expiration timer to be armed only when QUERY is
received.  Howerver, this causes issues in an environment where
the multicast server socket comes and goes very fast while a client
is trying to send traffic to it.

The root cause is a race where a sequence of LEAVE followed by REPORT
messages can race against QUERY messages generated in response to LEAVE.
The QUERY ends up starting the expiration timer, and that timer can
potentially expire after the new REPORT message has been received signaling
the new join operation.  This leads to a significant drop in multicast
traffic and possible complete stall.

The solution is to have REPORT messages update the expiration timer
on entries that already exist.

CC: Cong Wang <xiyou.wangcong@gmail.com>
CC: Herbert Xu <herbert@gondor.apana.org.au>
CC: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: Vlad Yasevich <vyasevic@redhat.com>
Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_multicast.c