fff-batman-adv: Enable bridge loop avoidance

Submitted by Adrian Schmutzler on Nov. 27, 2018, 10:29 p.m.

Details

Message ID 20181127222926.1712-1-freifunk@adrianschmutzler.de
State Superseded
Headers show

Commit Message

Adrian Schmutzler Nov. 27, 2018, 10:29 p.m.
Although many people say it does not apply to our setup, it is
enabled by default and there are no known drawbacks.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 .../fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv b/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
index f312c491..b4515a27 100644
--- a/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
+++ b/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
@@ -5,7 +5,7 @@  uci batch <<EOF
   set batman-adv.bat0=mesh
   set batman-adv.bat0.gw_mode='client'
   set batman-adv.bat0.gw_sel_class='1'
-  set batman-adv.bat0.bridge_loop_avoidance='0'
+  set batman-adv.bat0.bridge_loop_avoidance='1'
   set batman-adv.bat0.network_coding='0'
   set batman-adv.bat0.aggregated_ogms='1'
   set batman-adv.bat0.ap_isolation='0'

Comments

Christian Dresel Nov. 28, 2018, 12:31 p.m.
Hi

So ich hab nochmal versucht dieses bla laut Batman Doku auf unseren Setup zu verstehen.

Wenn ich das nun richtig verstanden habe würde es in folgender Situation helfen:

2 Nodes sehen sich per WLAN (Mesh mit Batman drinnen) und werden per Ethernet-Kabel an den Client Ports(!! Nicht Batman! Also ansich total falsch) zusammen gesteckt. Dies würde ohne bla eine Loop ergeben. Mit bla kümmert sich Batman darum diese Loop zu lösen.

Dieser Fall ist durchaus schon mal aufgetreten und wenn der sich so leicht vermeiden lässt, sollten wir das aktivieren. Kann dies bitte so jemand bestätigen daß ich hier richtig liege?

Wenn aber default eh an ist, würde ich die Zeile lieber ganz raus werfen und auf Batman default vertrauen

Mfg

Christian

Am 27. November 2018 23:29:26 MEZ schrieb Adrian Schmutzler <freifunk@adrianschmutzler.de>:
>Although many people say it does not apply to our setup, it is
>enabled by default and there are no known drawbacks.
>
>Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>---
>.../fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv        
>| 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git
>a/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
>b/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
>index f312c491..b4515a27 100644
>---
>a/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
>+++
>b/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
>@@ -5,7 +5,7 @@ uci batch <<EOF
>   set batman-adv.bat0=mesh
>   set batman-adv.bat0.gw_mode='client'
>   set batman-adv.bat0.gw_sel_class='1'
>-  set batman-adv.bat0.bridge_loop_avoidance='0'
>+  set batman-adv.bat0.bridge_loop_avoidance='1'
>   set batman-adv.bat0.network_coding='0'
>   set batman-adv.bat0.aggregated_ogms='1'
>   set batman-adv.bat0.ap_isolation='0'
>-- 
>2.11.0
Florian Wiessner Nov. 28, 2018, 12:39 p.m.
Hi,


rein damit, hab das bei meinen Nodes schon lange manuell an. Schadet nicht,
hilft aber in gewissen Setups.


Am 28.11.2018 um 13:31 schrieb Christian Dresel:
> Hi
>
> So ich hab nochmal versucht dieses bla laut Batman Doku auf unseren Setup zu
> verstehen.
>
> Wenn ich das nun richtig verstanden habe würde es in folgender Situation helfen:
>
> 2 Nodes sehen sich per WLAN (Mesh mit Batman drinnen) und werden per
> Ethernet-Kabel an den Client Ports(!! Nicht Batman! Also ansich total falsch)
> zusammen gesteckt. Dies würde ohne bla eine Loop ergeben. Mit bla kümmert sich
> Batman darum diese Loop zu lösen.
>
> Dieser Fall ist durchaus schon mal aufgetreten und wenn der sich so leicht
> vermeiden lässt, sollten wir das aktivieren. Kann dies bitte so jemand
> bestätigen daß ich hier richtig liege?
>
> Wenn aber default eh an ist, würde ich die Zeile lieber ganz raus werfen und
> auf Batman default vertrauen
>
> Mfg
>
> Christian
>
> Am 27. November 2018 23:29:26 MEZ schrieb Adrian Schmutzler
> <freifunk@adrianschmutzler.de>:
>
>     Although many people say it does not apply to our setup, it is
>     enabled by default and there are no known drawbacks.
>
>     Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>     --------------------------------------------------------------------------------
>      .../fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv         | 2 +-
>      1 file changed, 1 insertion(+), 1 deletion(-)
>
>     diff --git a/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv b/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
>     index f312c491..b4515a27 100644
>     --- a/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
>     +++ b/src/packages/fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-adv
>     @@ -5,7 +5,7 @@ uci batch <<EOF
>        set batman-adv.bat0=mesh
>        set batman-adv.bat0.gw_mode='client'
>        set batman-adv.bat0.gw_sel_class='1'
>     -  set batman-adv.bat0.bridge_loop_avoidance='0'
>     +  set batman-adv.bat0.bridge_loop_avoidance='1'
>        set batman-adv.bat0.network_coding='0'
>        set batman-adv.bat0.aggregated_ogms='1'
>        set batman-adv.bat0.ap_isolation='0'
>
>
> -- 
> Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.