[RFC,2/2] batman-adv: Don't re-broadcast packets on non-wifi devices

Message ID 1459340372-6971-2-git-send-email-sven@narfation.org (mailing list archive)
State RFC, archived
Headers

Commit Message

Sven Eckelmann March 30, 2016, 12:19 p.m. UTC
  It is not necessary to re-broadcast a received broadcast packet on the
rx-device when it is a standard ethernet device. The link medium already
takes care of transporting it to all participants in the broadcast domain.

The re-broadcast on other devices is still necessary to allow the broadcast
packet to be received by devices which are using a different link medium.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
 net/batman-adv/main.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Matthias Schiffer March 31, 2016, 8:41 a.m. UTC | #1
On 03/30/2016 02:19 PM, Sven Eckelmann wrote:
> It is not necessary to re-broadcast a received broadcast packet on the
> rx-device when it is a standard ethernet device. The link medium already
> takes care of transporting it to all participants in the broadcast domain.
> 
> The re-broadcast on other devices is still necessary to allow the broadcast
> packet to be received by devices which are using a different link medium.
> 
> Signed-off-by: Sven Eckelmann <sven@narfation.org>
> ---

I don't think completely disabling the rebroadcast here without a way to
enable it again is a viable option.

At least when using batman-adv over VPN tunnels, making batman-adv take
care of establishing transitive connectivity may make sense, as batman-adv
does this very well and without much configuration. This is actually the
recommended setup for non-full-mesh setups using the VPN tool fastd (which
is developed by me.)

Regards,
Matthias
  
Sven Eckelmann March 31, 2016, 9:07 a.m. UTC | #2
On Thursday 31 March 2016 10:41:04 Matthias Schiffer wrote:
[...]
> I don't think completely disabling the rebroadcast here without a way to
> enable it again is a viable option.
> 
> At least when using batman-adv over VPN tunnels, making batman-adv take
> care of establishing transitive connectivity may make sense, as batman-adv
> does this very well and without much configuration. This is actually the
> recommended setup for non-full-mesh setups using the VPN tool fastd (which
> is developed by me.)

Thanks for the feedback. There was also a small questionnaire answered by
Adrian Reyer about fastd:
https://lists.open-mesh.org/pipermail/b.a.t.m.a.n/2016-March/014803.html

Kind regards,
	Sven
  

Patch

diff --git a/net/batman-adv/main.h b/net/batman-adv/main.h
index aad4a3a..ddad99d 100644
--- a/net/batman-adv/main.h
+++ b/net/batman-adv/main.h
@@ -96,7 +96,7 @@ 
 #define BATADV_LOG_BUF_LEN 8192	  /* has to be a power of 2 */
 
 /* number of packets to send for broadcasts on different interface types */
-#define BATADV_NUM_BCASTS_SAMEIF 1
+#define BATADV_NUM_BCASTS_SAMEIF 0
 #define BATADV_NUM_BCASTS_OTHERIF 1
 #define BATADV_NUM_BCASTS_WIRELESS 3
 #define BATADV_NUM_BCASTS_MAX 3