[B.A.T.M.A.N.] [PATCH] batman-adv: enable MCAST by default at compile time

[B.A.T.M.A.N.] [PATCH] batman-adv: enable MCAST by default at compile time

[Linus Lüssing]

Thanks to rigorous testing in wireless community mesh networks several
issues with multicast entries in the translation table were found and
fixed in the last 1.5 years. Now we see the first larger networks
(a few hundred nodes) with a batman-adv version with multicast
optimizations enabled arising, with no TT / multicast optimization
related issues so far.

Therefore it seems safe to enable multicast optimizations by default not
only in kernel module builds but also for in-kernel builds.

Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
---
 net/batman-adv/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/batman-adv/Kconfig b/net/batman-adv/Kconfig
index f75816f5..b70b96cd 100644
--- a/net/batman-adv/Kconfig
+++ b/net/batman-adv/Kconfig
@@ -82,6 +82,7 @@ config BATMAN_ADV_NC
 config BATMAN_ADV_MCAST
 	bool "Multicast optimisation"
 	depends on BATMAN_ADV && INET && !(BRIDGE=m && BATMAN_ADV=y)
+	default y
 	help
 	  This option enables the multicast optimisation which aims to
 	  reduce the air overhead while improving the reliability of
-- 
2.11.0

Re: [B.A.T.M.A.N.] [PATCH] batman-adv: enable MCAST by default at compile time

[Sven Eckelmann]

[-- Attachment #1: Type: text/plain, Size: 1124 bytes --]

On Dienstag, 6. November 2018 10:01:50 CET Linus Lüssing wrote:
> Thanks to rigorous testing in wireless community mesh networks several
> issues with multicast entries in the translation table were found and
> fixed in the last 1.5 years. Now we see the first larger networks
> (a few hundred nodes) with a batman-adv version with multicast
> optimizations enabled arising, with no TT / multicast optimization
> related issues so far.
> 
> Therefore it seems safe to enable multicast optimizations by default not
> only in kernel module builds but also for in-kernel builds.
> 
> Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
> ---
>  net/batman-adv/Kconfig | 1 +
>  1 file changed, 1 insertion(+)

I have now queued it up for the upcoming net-next PR. This patch should be 
merged after we released batman-adv 2018.4. But my original patch to disable 
the MCAST optimization by default [1] for 2018.4 was already reverted. The 
2018.4 release will therefore still have MCAST enabled by default for the out-
of-tree module.

Thanks,
	Sven

[1] https://patchwork.open-mesh.org/patch/17435/

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]