| Message ID | 1510656289-2215-1-git-send-email-freifunk@adrianschmutzler.de |
|---|---|
| State | Accepted |
| Headers | show |
diff --git a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork index 1571c05..98e2066 100755 --- a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork +++ b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork @@ -170,7 +170,7 @@ if [[ -n "$ETHMESHMAC" ]]; then uci commit ifconfig $SWITCHDEV.3 down - ifconfig $SWITCHDEV.3 hw ether $NEW_MACADDR + ifconfig $SWITCHDEV.3 hw ether $ETHMESHMAC ifconfig $SWITCHDEV.3 up /etc/init.d/network restart fi @@ -188,7 +188,7 @@ if [[ -n "$ROUTERMAC" ]]; then uci commit ifconfig br-mesh down - ifconfig br-mesh hw ether $NEW_MACADDR + ifconfig br-mesh hw ether $ROUTERMAC ifconfig br-mesh up /etc/init.d/network restart fi
Hallo Adrian, wahrscheinlich, weil es, wenn ich nichts übersehen habe, kein Problem darstellt, da direkt danach init.d/network komplett restartet wird, was dazu führt, dass das Interface die MAC aus network.mesh.macaddr bekommt. Network nicht mehrfach komplett zu restarten würde den ersten boot vermutlich etwas beschleunigen. Entweder noch mit einbauen oder ich mach n neuen Patch dafür, mir egal. Reviewed-by: Fabian Bläse <fabian@blaese.de> Fabian > On 14. Nov 2017, at 11:44, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote: > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > Wie konnten wir alle das übersehen?! > --- > src/packages/fff/fff-network/files/usr/sbin/configurenetwork | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > index 1571c05..98e2066 100755 > --- a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > +++ b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > @@ -170,7 +170,7 @@ if [[ -n "$ETHMESHMAC" ]]; then > uci commit > > ifconfig $SWITCHDEV.3 down > - ifconfig $SWITCHDEV.3 hw ether $NEW_MACADDR > + ifconfig $SWITCHDEV.3 hw ether $ETHMESHMAC > ifconfig $SWITCHDEV.3 up > /etc/init.d/network restart > fi > @@ -188,7 +188,7 @@ if [[ -n "$ROUTERMAC" ]]; then > uci commit > > ifconfig br-mesh down > - ifconfig br-mesh hw ether $NEW_MACADDR > + ifconfig br-mesh hw ether $ROUTERMAC > ifconfig br-mesh up > /etc/init.d/network restart > fi > -- > 2.7.4 > > -- > franken-dev mailing list > franken-dev@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Bitte deinen Vorschlag separat patchen. > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Fabian Bläse > Sent: Dienstag, 14. November 2017 20:25 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > dev@freifunk.net > Subject: Re: [PATCH] fff-network: Fix second occurrence of ROUTERMAC and > ETHMESHMAC > > Hallo Adrian, > > wahrscheinlich, weil es, wenn ich nichts übersehen habe, kein Problem > darstellt, da direkt danach init.d/network komplett restartet wird, was dazu > führt, dass das Interface die MAC aus network.mesh.macaddr bekommt. > Network nicht mehrfach komplett zu restarten würde den ersten boot > vermutlich etwas beschleunigen. Entweder noch mit einbauen oder ich mach > n neuen Patch dafür, mir egal. > > Reviewed-by: Fabian Bläse <fabian@blaese.de> > > Fabian > > > On 14. Nov 2017, at 11:44, Adrian Schmutzler > <freifunk@adrianschmutzler.de> wrote: > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > > > --- > > > > Wie konnten wir alle das übersehen?! > > --- > > src/packages/fff/fff-network/files/usr/sbin/configurenetwork | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > > index 1571c05..98e2066 100755 > > --- a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > > +++ b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > > @@ -170,7 +170,7 @@ if [[ -n "$ETHMESHMAC" ]]; then > > uci commit > > > > ifconfig $SWITCHDEV.3 down > > - ifconfig $SWITCHDEV.3 hw ether $NEW_MACADDR > > + ifconfig $SWITCHDEV.3 hw ether $ETHMESHMAC > > ifconfig $SWITCHDEV.3 up > > /etc/init.d/network restart > > fi > > @@ -188,7 +188,7 @@ if [[ -n "$ROUTERMAC" ]]; then > > uci commit > > > > ifconfig br-mesh down > > - ifconfig br-mesh hw ether $NEW_MACADDR > > + ifconfig br-mesh hw ether $ROUTERMAC > > ifconfig br-mesh up > > /etc/init.d/network restart > > fi > > -- > > 2.7.4 > > > > -- > > franken-dev mailing list > > franken-dev@freifunk.net > > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Adrian Schmutzler > Sent: Dienstag, 14. November 2017 11:45 > To: franken-dev@freifunk.net > Subject: [PATCH] fff-network: Fix second occurrence of ROUTERMAC and > ETHMESHMAC > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > Wie konnten wir alle das übersehen?! > --- > src/packages/fff/fff-network/files/usr/sbin/configurenetwork | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > index 1571c05..98e2066 100755 > --- a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > +++ b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > @@ -170,7 +170,7 @@ if [[ -n "$ETHMESHMAC" ]]; then > uci commit > > ifconfig $SWITCHDEV.3 down > - ifconfig $SWITCHDEV.3 hw ether $NEW_MACADDR > + ifconfig $SWITCHDEV.3 hw ether $ETHMESHMAC > ifconfig $SWITCHDEV.3 up > /etc/init.d/network restart > fi > @@ -188,7 +188,7 @@ if [[ -n "$ROUTERMAC" ]]; then > uci commit > > ifconfig br-mesh down > - ifconfig br-mesh hw ether $NEW_MACADDR > + ifconfig br-mesh hw ether $ROUTERMAC > ifconfig br-mesh up > /etc/init.d/network restart > fi > -- > 2.7.4 > > -- > franken-dev mailing list > franken-dev@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Hallo, ich sehe das auch so, dass hier doppelt restartet wird. Ich wuerde vorschlagen, das hier erst mal rein und dann kann man die network restart raus nehemen. Reviewed-by: Robert Langhammer <rlanghammer@web.de> Am 14.11.2017 um 20:24 schrieb Fabian Bläse: > Hallo Adrian, > > wahrscheinlich, weil es, wenn ich nichts übersehen habe, kein Problem darstellt, da direkt danach init.d/network komplett restartet wird, was dazu führt, dass das Interface die MAC aus network.mesh.macaddr bekommt. > Network nicht mehrfach komplett zu restarten würde den ersten boot vermutlich etwas beschleunigen. Entweder noch mit einbauen oder ich mach n neuen Patch dafür, mir egal. > > Reviewed-by: Fabian Bläse <fabian@blaese.de> > > Fabian > >> On 14. Nov 2017, at 11:44, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote: >> >> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> >> >> --- >> >> Wie konnten wir alle das übersehen?! >> --- >> src/packages/fff/fff-network/files/usr/sbin/configurenetwork | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork >> index 1571c05..98e2066 100755 >> --- a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork >> +++ b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork >> @@ -170,7 +170,7 @@ if [[ -n "$ETHMESHMAC" ]]; then >> uci commit >> >> ifconfig $SWITCHDEV.3 down >> - ifconfig $SWITCHDEV.3 hw ether $NEW_MACADDR >> + ifconfig $SWITCHDEV.3 hw ether $ETHMESHMAC >> ifconfig $SWITCHDEV.3 up >> /etc/init.d/network restart >> fi >> @@ -188,7 +188,7 @@ if [[ -n "$ROUTERMAC" ]]; then >> uci commit >> >> ifconfig br-mesh down >> - ifconfig br-mesh hw ether $NEW_MACADDR >> + ifconfig br-mesh hw ether $ROUTERMAC >> ifconfig br-mesh up >> /etc/init.d/network restart >> fi >> -- >> 2.7.4 >> >> -- >> franken-dev mailing list >> franken-dev@freifunk.net >> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net > >
Hi Jetzt ist das eine mal gerade gezogen. -> Applied. Der restart war damals aus irgendeinem Grund nötig. Möglicherweise war (oder ist sogar) ein Bug im UCI System.. Keine Ahnung. Tim Am Dienstag, den 14.11.2017, 11:44 +0100 schrieb Adrian Schmutzler: > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > Wie konnten wir alle das übersehen?! > --- > src/packages/fff/fff-network/files/usr/sbin/configurenetwork | 4 ++- > - > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/packages/fff/fff- > network/files/usr/sbin/configurenetwork b/src/packages/fff/fff- > network/files/usr/sbin/configurenetwork > index 1571c05..98e2066 100755 > --- a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > +++ b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork > @@ -170,7 +170,7 @@ if [[ -n "$ETHMESHMAC" ]]; then > uci commit > > ifconfig $SWITCHDEV.3 down > - ifconfig $SWITCHDEV.3 hw ether $NEW_MACADDR > + ifconfig $SWITCHDEV.3 hw ether $ETHMESHMAC > ifconfig $SWITCHDEV.3 up > /etc/init.d/network restart > fi > @@ -188,7 +188,7 @@ if [[ -n "$ROUTERMAC" ]]; then > uci commit > > ifconfig br-mesh down > - ifconfig br-mesh hw ether $NEW_MACADDR > + ifconfig br-mesh hw ether $ROUTERMAC > ifconfig br-mesh up > /etc/init.d/network restart > fi > -- > 2.7.4 >
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- Wie konnten wir alle das übersehen?! --- src/packages/fff/fff-network/files/usr/sbin/configurenetwork | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)