[PATCH] Remove batman option no_rebroadcast

[mail at adrianschmutzler.de]

Hallo robert und Rest,

ich habe das Ganze mal aufgerollt:

Hier ist die Diskussion dazu im Patchwork von Batman:

https://patchwork.open-mesh.org/patch/3434/

Man hat sich geeinigt, das nicht ins Batman reinzutun, und Gluon und diverse andere verwenden das seither als eigenen Batman-Patch.

Das hat aber zunächst nichts mit altem vs. neuem Batman zu tun.

Ich habe mal gestöbert, der Batman-Patch wurde im Dezember 2013 bei uns applied:
https://github.com/FreifunkFranken/firmware/commit/704c8b69c03291872dc91a6504b5958b68f5c9d3

Im Oktober 2015 wurde Batman nicht mehr aus dem OpenWRT-Routing bezogen, sondern von Gluon. Dadurch war der Patch inkludiert und wurde aus der Firmware gelöscht:
https://github.com/FreifunkFranken/firmware/commit/11aff9654deda34570011704b6b564b478b4b38c

Wir haben dann im Oktober 2017 auf das neue Batman gewechselt. Für das neue Batman wurde als Quelle wieder openwrt-routing gewählt, während batman-legacy auch zu diesem Zeitpunkt noch von gluon kam:

https://github.com/FreifunkFranken/firmware/commit/60fecf6ea52697c293a2563ffdbcb70ca2516fbb

Praktisch bedeutet das, dass wir in der V1-Firmware alle Gloun-Patches für Batman enthalten haben (inkl. no_rebroadcast) und in der V2-Firmware keinen davon, sondern nur das offizielle Batman. D.h. im V1-Netz ist die Option überall aktiv und im V2-Netz nirgends.

Ggf. sind zudem noch weitere Patches im Gluon, die wir dadurch auch "verloren" haben.

Wir haben demnach zwei Möglichkeiten:
1. Wir entfernen die Funktion in unserer Firmware, weil wir der Meinung sind, dass wir kein Split-Horizon brauchen (haben das in der Vergangenheit aber "immer" benutzt).
2. Wir bauen den Patch für Batman ein und aktivieren damit das Split-Horizon-Verhalten.

Beste Grüße

Adrian




> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of robert
> Sent: Sonntag, 29. Juli 2018 13:37
> To: franken-dev at freifunk.net
> Subject: Re: [PATCH] Remove batman option no_rebroadcast
> 
> Hi,
> 
> ich hab an der falschen Stelle gesucht. Das ist ein gluon Ding.
> 
> https://github.com/freifunk-gluon/batman-adv-
> legacy/commit/b0160f988c7ff386042b2feef4f056cb66602090
> 
> und war wohl nie im original batman drin. Etwas entsprechendes hab ich bei
> open-mesh.org nicht gefunden. Ob man das irgendwie nachbauen muss,
> wird sich zeigen. Die neuen Hoods sollen ja auch nicht mehr so groß werden.
> 
> Jedenfalls gibt es die Option aktuell nicht mehr.
> 
> Robert
> 
> 
> Am 29.07.2018 um 11:03 schrieb Tim Niemeyer:
> > Hi
> >
> > Am 29. Juli 2018 01:21:39 MESZ schrieb robert <rlanghammer at web.de>:
> >> Hi,
> >>
> >> ich hab auf die schnelle nur den Patch gefunden der es mal rein
> >> gebracht
> >> hat:
> >> https://lists.open-mesh.org/pipermail/b.a.t.m.a.n/2013-September/0105
> >> 86.html
> >>
> >>
> >> Und jetzt ist es im Code nicht mehr drin. Folglich hat man es
> >> irgendwann raus genommen. Im sysfs taucht es ja auch nicht mehr auf.
> > Das ist vermutlich mit dem upgrade auf das neue Batman raus geflogen.
> Die Option war damals sehr wichtig, weil es viel boradcast overhead entfernt
> hat. Das feature funktioniert aber nur in der Art und weise wie wir das
> Batman einsetzen (zentrale Server), deswegen war das nie mainline
> gekommen.
> >
> > Man sollte prüfen, ob ein ähnliches verfahren für das aktuelle Batman nötig
> ist.
> >
> > Tim
> >
> >
> >> Am 29.07.2018 um 00:30 schrieb mail at adrianschmutzler.de:
> >>> Hallo Robert,
> >>>
> >>>> -----Original Message-----
> >>>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> >> Behalf
> >>>> Of Robert Langhammer
> >>>> Sent: Samstag, 28. Juli 2018 23:58
> >>>> To: franken-dev at freifunk.net
> >>>> Subject: [PATCH] Remove batman option no_rebroadcast
> >>>>
> >>>> This sysfs option no longer exists.
> >>> steht das irgendwo, dass die entfernt wurde oder schließt du das nur
> >> aus der
> >>> Fehlermeldung/dem Vorhandensein im Dateisystem?
> >>>
> >>> Grüße
> >>>
> >>> Adrian
> >>>
> >>>> Signed-off-by: Robert Langhammer <rlanghammer at web.de>
> >>>> ---
> >>>>  src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> >>> | 1 -
> >>>>  .../fff/fff-tunneldigger/files/etc/tunneldigger/tunneldigger.hook
> >>> | 1 -
> >>>>  2 files changed, 2 deletions(-)
> >>>>
> >>>> diff --git
> >>> a/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> >>>> b/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> >>>> index 72970bd..d53eb43 100644
> >>>> --- a/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> >>>> +++ b/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> >>>> @@ -22,6 +22,5 @@ EOF
> >>>>  ln -s /tmp/fastd_fff_peers /etc/fastd/fff/peers  echo "#!/bin/sh" >
> >>>> /etc/fastd/fff/up.sh  echo "ip link set up dev fffVPN" >>
> >>> /etc/fastd/fff/up.sh
> >>>> -echo "echo enable >
> >>>> /sys/devices/virtual/net/fffVPN/batman_adv/no_rebroadcast" >>
> >>>> /etc/fastd/fff/up.sh  echo "batctl if add fffVPN" >>
> >> /etc/fastd/fff/up.sh
> >>>> chmod +x /etc/fastd/fff/up.sh diff --git a/src/packages/fff/fff-
> >>>> tunneldigger/files/etc/tunneldigger/tunneldigger.hook
> >>>> b/src/packages/fff/fff-
> >>>> tunneldigger/files/etc/tunneldigger/tunneldigger.hook
> >>>> index c4fea62..a96b42c 100755
> >>>> --- a/src/packages/fff/fff-
> >>>> tunneldigger/files/etc/tunneldigger/tunneldigger.hook
> >>>> +++
> >> b/src/packages/fff/fff-tunneldigger/files/etc/tunneldigger/tunneldig
> >>>> +++ ger.hook
> >>>> @@ -6,5 +6,4 @@ INTERFACE=$2
> >>>>  if [ "$HOOK" = session.up ]; then
> >>>>         ip l s dev $INTERFACE up
> >>>>         batctl if add $INTERFACE
> >>>> -       echo "enabled" >
> >>>> /sys/devices/virtual/net/$INTERFACE/batman_adv/no_rebroadcast
> >>>>  fi
> >>>> --
> >>>> 2.11.0
>