[Bridge] [PATCHv2] bridge: disable snooping if there is no querier
linux at baker-net.org.uk
Fri Jul 26 22:19:00 UTC 2013
On 25/07/13 14:56, Linus Lüssing wrote:
> If there is no querier on a link then we won't get periodic reports and
> therefore won't be able to learn about multicast listeners behind ports,
> potentially leading to lost multicast packets, especially for multicast
> listeners that joined before the creation of the bridge.
> These lost multicast packets can appear since c5c23260594
> ("bridge: Add multicast_querier toggle and disable queries by default")
> in particular.
> With this patch we are flooding multicast packets if our querier is
> disabled and if we didn't detect any other querier.
> A grace period of the Maximum Response Delay of the querier is added to
> give multicast responses enough time to arrive and to be learned from
> before disabling the flooding behaviour again.
> Signed-off-by: Linus Lüssing<linus.luessing at web.de>
If the lack of queries if there is no other querier is unacceptable to
the majority of users (and I believe it is) then surely the sensible
option is to have the multicast querier toggle enabled by default.
The toggle was added in the first place because the queries were
reported to be generating issues with certain other equipment. This may
have been because the queries by default have an invalid IP address
(although I have been unable to identify what equipment they caused
problems with so can't verify this).
If the only reason to turn the querier off is because it interferes with
other equipment then the solution to it being off by default isn't to
generate queries in some instances even if it is off but rather to turn
it on by default and only turn it off if it causes problems. If
multicast_query_use_ifaddr was also enabled by default the the
likelihood of the querier causing problems elsewhere should be reduced.
More information about the Bridge