Message ID | 20181110221645.19497-1-fabian@blaese.de |
---|---|
State | Superseded |
Headers | show |
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
Hi Am Samstag, den 10.11.2018, 23:16 +0100 schrieb Fabian Bläse: > The no_rebroadcast option has been introduced by > a gluon patch in batman-adv-legacy. Switching to > upstream batman with fff-batman-adv has made this > option obsolete, so it can safely be removed. Oh.. a) Heißt das, die Option gibt es nicht mehr? b) Heißt das auch, dass es kein adäquates äquivalent zu dem Patch gibt? Wenn a): Reviewed-by: Tim Niemeyer <tim@tn-x.org> Wenn b): Dann muss da eine Lösung her, weil es sonst einfach eskalieren wird. Tim > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd | 1 - > 1 file changed, 1 deletion(-) > > 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
Hey, Christian hat gerade entdeckt, dass es da schonmal einen Patch und Diskussion gab: "[Patch] Remove batman option no_rebroadcast" https://pw.freifunk-franken.de/patch/900/ a) Die Option wurde nie bei batman-adv upstream genommen. Beim gluon-batman gibt es einen passenden Patch. Diesen haben wir auch früher applied. b) Ich glaube nicht, dass es vollkommen eskaliert, aber es würde bei dem zentralen Kram bestimmt einiges an Rauschen sparen. Man müsste den Patch eigentlich auch auf das aktuelle BatmanAdv übernehmen können. Gibt es auch für compat15 ein gluon BatmanAdv repo? Fabian On 10.11.18 23:55, Tim Niemeyer wrote: > Hi > > Am Samstag, den 10.11.2018, 23:16 +0100 schrieb Fabian Bläse: >> The no_rebroadcast option has been introduced by >> a gluon patch in batman-adv-legacy. Switching to >> upstream batman with fff-batman-adv has made this >> option obsolete, so it can safely be removed. > Oh.. > > a) Heißt das, die Option gibt es nicht mehr? > > b) Heißt das auch, dass es kein adäquates äquivalent zu dem Patch gibt? > > Wenn a): Reviewed-by: Tim Niemeyer <tim@tn-x.org> > Wenn b): Dann muss da eine Lösung her, weil es sonst einfach eskalieren > wird. > > Tim > >> >> Signed-off-by: Fabian Bläse <fabian@blaese.de> >> --- >> src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd | 1 - >> 1 file changed, 1 deletion(-) >> >> 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
Hi hab mich mal auf dumm gestellt und bei Gluon umgehört: https://github.com/freifunk-gluon/gluon/commit/2adf67edad86 https://git.open-mesh.org/batman-adv.git/commit/a00797d8fa8fd1471e8be1ac23d506f76d866aaa Wenn ich das richtig verstehe, ist das im neuen Batman-adv also schon mit drinnen. Reviewed-by: Christian Dresel <fff@chrisi01.de> mfg Christian Am 11.11.18 um 00:08 schrieb Fabian Bläse: > Hey, > > Christian hat gerade entdeckt, dass es da schonmal einen Patch und Diskussion gab: > "[Patch] Remove batman option no_rebroadcast" > https://pw.freifunk-franken.de/patch/900/ > > a) Die Option wurde nie bei batman-adv upstream genommen. Beim gluon-batman gibt es einen passenden Patch. Diesen haben wir auch früher applied. > b) Ich glaube nicht, dass es vollkommen eskaliert, aber es würde bei dem zentralen Kram bestimmt einiges an Rauschen sparen. > > Man müsste den Patch eigentlich auch auf das aktuelle BatmanAdv übernehmen können. > Gibt es auch für compat15 ein gluon BatmanAdv repo? > > Fabian > > On 10.11.18 23:55, Tim Niemeyer wrote: >> Hi >> >> Am Samstag, den 10.11.2018, 23:16 +0100 schrieb Fabian Bläse: >>> The no_rebroadcast option has been introduced by >>> a gluon patch in batman-adv-legacy. Switching to >>> upstream batman with fff-batman-adv has made this >>> option obsolete, so it can safely be removed. >> Oh.. >> >> a) Heißt das, die Option gibt es nicht mehr? >> >> b) Heißt das auch, dass es kein adäquates äquivalent zu dem Patch gibt? >> >> Wenn a): Reviewed-by: Tim Niemeyer <tim@tn-x.org> >> Wenn b): Dann muss da eine Lösung her, weil es sonst einfach eskalieren >> wird. >> >> Tim >> >>> >>> Signed-off-by: Fabian Bläse <fabian@blaese.de> >>> --- >>> src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd | 1 - >>> 1 file changed, 1 deletion(-) >>> >>> 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 >
Hallo, schön, dass ihr euch nochmal damit auseinandergesetzt habt. Robert hat denselben Patch ja schon vor einiger Zeit geschickt, zudem umfasst sein Patch auch den Tunneldigger. https://pw.freifunk-franken.de/patch/900/ Ich würde mir daher wünschen, dass ihr Roberts-Patch entsprechend reviewt und wir dann den applien (1. er war erster und 2. der Tunneldigger ist mit drin). Weiterhin würde ich mir wünschen, dass jemand den Sachstand aus der Diskussion (alte option aus gluon repo, fällt durch neues batman weg, danach integriert ...) nochmal in zwei bis drei Absätzen als Commit Description zusammenfasst. Das reicht per Mail, ich würde das dann beim Applien von Hand mit einfügen. Beste Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Fabian Bläse > Sent: Samstag, 10. November 2018 23:17 > To: franken-dev@freifunk.net > Subject: [PATCH] fastd: Don't set obsolete batman-adv option > > The no_rebroadcast option has been introduced by a gluon patch in batman- > adv-legacy. Switching to upstream batman with fff-batman-adv has made > this option obsolete, so it can safely be removed. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd | 1 - > 1 file changed, 1 deletion(-) > > 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 > -- > 2.19.1
Hallo Adrian, On 11.11.18 12:30, mail@adrianschmutzler.de wrote: > Hallo, > > schön, dass ihr euch nochmal damit auseinandergesetzt habt. > > Robert hat denselben Patch ja schon vor einiger Zeit geschickt, zudem umfasst sein Patch auch den Tunneldigger. > > https://pw.freifunk-franken.de/patch/900/ > > Ich würde mir daher wünschen, dass ihr Roberts-Patch entsprechend reviewt und wir dann den applien (1. er war erster und 2. der Tunneldigger ist mit drin). Sehe ich genauso, dazu hatte ich ja auch schon mal ne Mail geschrieben. Ich werde das demnächst tun. Fabian
Habe den Patch von Robert applied, damit ist der hinfällig … From: Fabian Bläse [mailto:fabian@blaese.de] Sent: Sonntag, 11. November 2018 12:36 To: mail@adrianschmutzler.de; franken-dev@freifunk.net Subject: Re: [PATCH] fastd: Don't set obsolete batman-adv option Hallo Adrian, On 11.11.18 12:30, mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> wrote: > Hallo, > > schön, dass ihr euch nochmal damit auseinandergesetzt habt. > > Robert hat denselben Patch ja schon vor einiger Zeit geschickt, zudem umfasst sein Patch auch den Tunneldigger. > > https://pw.freifunk-franken.de/patch/900/ > > Ich würde mir daher wünschen, dass ihr Roberts-Patch entsprechend reviewt und wir dann den applien (1. er war erster und 2. der Tunneldigger ist mit drin). Sehe ich genauso, dazu hatte ich ja auch schon mal ne Mail geschrieben. Ich werde das demnächst tun. Fabian
The no_rebroadcast option has been introduced by a gluon patch in batman-adv-legacy. Switching to upstream batman with fff-batman-adv has made this option obsolete, so it can safely be removed. Signed-off-by: Fabian Bläse <fabian@blaese.de> --- src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd | 1 - 1 file changed, 1 deletion(-)