GL-AR150: Fix ETHMESHMAC

Submitted by Adrian Schmutzler on Jan. 3, 2018, 6:24 p.m.

Details

Message ID 1515003898-30118-1-git-send-email-freifunk@adrianschmutzler.de
State Accepted
Headers show

Commit Message

Adrian Schmutzler Jan. 3, 2018, 6:24 p.m.
The AR150 seems to have the same MAC address on eth0, eth1 and
phy0. Although it is not a ONEPORT, we can use the shifted bit
like for the ONEPORT devices to get a distinct ETHMESHMAC.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 src/packages/fff/fff-network/ar71xx/network.gl-ar150 | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-network/ar71xx/network.gl-ar150 b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
index 9e05f73..64049dd 100644
--- a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
+++ b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
@@ -1,3 +1,5 @@ 
+. /lib/functions/fff/network
+
 PORTORDER="1"
 
 WANDEV=eth0
@@ -6,8 +8,8 @@  WAN_PORTS=
 BATMAN_PORTS="0t"
 CLIENT_PORTS="0t"
 
-ETHMESHMAC=$(cat /sys/class/net/eth1/address)
 ROUTERMAC=$(cat /sys/class/net/eth0/address)
+ETHMESHMAC="$(macFlipLocalBit "$ROUTERMAC")"
 
 . /etc/network.mode
 

Comments

Tim Niemeyer Jan. 20, 2018, 1:53 p.m.
Hi

Am Mittwoch, den 03.01.2018, 19:24 +0100 schrieb Adrian Schmutzler:
> The AR150 seems to have the same MAC address on eth0, eth1 and
> phy0. Although it is not a ONEPORT, we can use the shifted bit
> like for the ONEPORT devices to get a distinct ETHMESHMAC.
Hm..

Das Gerät wurde damals von Mayosemmel eingepflegt. Wäre gut, wenn er
deine Änderung kurz bestätigen könnte.

Ansonsten von mir
Reviewed-by: Tim Niemeyer <tim@tn-x.org>

Tim

> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  src/packages/fff/fff-network/ar71xx/network.gl-ar150 | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> index 9e05f73..64049dd 100644
> --- a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> +++ b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> @@ -1,3 +1,5 @@
> +. /lib/functions/fff/network
> +
>  PORTORDER="1"
>  
>  WANDEV=eth0
> @@ -6,8 +8,8 @@ WAN_PORTS=
>  BATMAN_PORTS="0t"
>  CLIENT_PORTS="0t"
>  
> -ETHMESHMAC=$(cat /sys/class/net/eth1/address)
>  ROUTERMAC=$(cat /sys/class/net/eth0/address)
> +ETHMESHMAC="$(macFlipLocalBit "$ROUTERMAC")"
>  
>  . /etc/network.mode
>  
> -- 
> 2.7.4
>
Adrian Schmutzler Jan. 20, 2018, 2:02 p.m.
Hallo Tim/Liste,

mich wundert das auch. Kann durchaus auch sein, dass die zweite MAC mal im LEDE verloren gegangen ist. Auf allen Geräten, die ich im Monitoring gefunden habe, war's kaputt.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Samstag, 20. Januar 2018 14:54
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> dev@freifunk.net; Jan Kraus <mayosemmel@gmail.com>
> Subject: Re: [PATCH] GL-AR150: Fix ETHMESHMAC
> 
> Hi
> 
> Am Mittwoch, den 03.01.2018, 19:24 +0100 schrieb Adrian Schmutzler:
> > The AR150 seems to have the same MAC address on eth0, eth1 and phy0.
> > Although it is not a ONEPORT, we can use the shifted bit like for the
> > ONEPORT devices to get a distinct ETHMESHMAC.
> Hm..
> 
> Das Gerät wurde damals von Mayosemmel eingepflegt. Wäre gut, wenn er
> deine Änderung kurz bestätigen könnte.
> 
> Ansonsten von mir
> Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> 
> Tim
> 
> >
> > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > ---
> >  src/packages/fff/fff-network/ar71xx/network.gl-ar150 | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > index 9e05f73..64049dd 100644
> > --- a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > +++ b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > @@ -1,3 +1,5 @@
> > +. /lib/functions/fff/network
> > +
> >  PORTORDER="1"
> >
> >  WANDEV=eth0
> > @@ -6,8 +8,8 @@ WAN_PORTS=
> >  BATMAN_PORTS="0t"
> >  CLIENT_PORTS="0t"
> >
> > -ETHMESHMAC=$(cat /sys/class/net/eth1/address)
> >  ROUTERMAC=$(cat /sys/class/net/eth0/address)
> > +ETHMESHMAC="$(macFlipLocalBit "$ROUTERMAC")"
> >
> >  . /etc/network.mode
> >
> > --
> > 2.7.4
> >
Tim Niemeyer Jan. 20, 2018, 2:05 p.m.
Am Samstag, den 20.01.2018, 15:02 +0100 schrieb
mail@adrianschmutzler.de:
> Hallo Tim/Liste,
> 
> mich wundert das auch. Kann durchaus auch sein, dass die zweite MAC
> mal im LEDE verloren gegangen ist. Auf allen Geräten, die ich im
> Monitoring gefunden habe, war's kaputt.
Jo, soll Jan sich mal kurz zu äußern, dann passt das schon.

Er hat den Support dafür eingebaut, ich hoffe er hat noch so ein Teil
und er wollte sich ja eh wieder mehr mit dem Thema befassen. Das ist ja
jetzt ne prima Gelegenheit.

Tim

> Grüße
> 
> Adrian
> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> > Behalf
> > Of Tim Niemeyer
> > Sent: Samstag, 20. Januar 2018 14:54
> > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > dev@freifunk.net; Jan Kraus <mayosemmel@gmail.com>
> > Subject: Re: [PATCH] GL-AR150: Fix ETHMESHMAC
> > 
> > Hi
> > 
> > Am Mittwoch, den 03.01.2018, 19:24 +0100 schrieb Adrian Schmutzler:
> > > The AR150 seems to have the same MAC address on eth0, eth1 and
> > > phy0.
> > > Although it is not a ONEPORT, we can use the shifted bit like for
> > > the
> > > ONEPORT devices to get a distinct ETHMESHMAC.
> > 
> > Hm..
> > 
> > Das Gerät wurde damals von Mayosemmel eingepflegt. Wäre gut, wenn
> > er
> > deine Änderung kurz bestätigen könnte.
> > 
> > Ansonsten von mir
> > Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> > 
> > Tim
> > 
> > > 
> > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > ---
> > >  src/packages/fff/fff-network/ar71xx/network.gl-ar150 | 4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > index 9e05f73..64049dd 100644
> > > --- a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > +++ b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > @@ -1,3 +1,5 @@
> > > +. /lib/functions/fff/network
> > > +
> > >  PORTORDER="1"
> > > 
> > >  WANDEV=eth0
> > > @@ -6,8 +8,8 @@ WAN_PORTS=
> > >  BATMAN_PORTS="0t"
> > >  CLIENT_PORTS="0t"
> > > 
> > > -ETHMESHMAC=$(cat /sys/class/net/eth1/address)
> > >  ROUTERMAC=$(cat /sys/class/net/eth0/address)
> > > +ETHMESHMAC="$(macFlipLocalBit "$ROUTERMAC")"
> > > 
> > >  . /etc/network.mode
> > > 
> > > --
> > > 2.7.4
> > > 
> 
>
Jan Kraus Jan. 20, 2018, 3:57 p.m.
Ich hab so ne Kiste. Muss nachschauen wie aich das verhalten hat.

Grüße Jan

Am 20. Januar 2018 15:05:57 MEZ schrieb Tim Niemeyer <tim@tn-x.org>:
>Am Samstag, den 20.01.2018, 15:02 +0100 schrieb
>mail@adrianschmutzler.de:
>> Hallo Tim/Liste,
>> 
>> mich wundert das auch. Kann durchaus auch sein, dass die zweite MAC
>> mal im LEDE verloren gegangen ist. Auf allen Geräten, die ich im
>> Monitoring gefunden habe, war's kaputt.
>Jo, soll Jan sich mal kurz zu äußern, dann passt das schon.
>
>Er hat den Support dafür eingebaut, ich hoffe er hat noch so ein Teil
>und er wollte sich ja eh wieder mehr mit dem Thema befassen. Das ist ja
>jetzt ne prima Gelegenheit.
>
>Tim
>
>> Grüße
>> 
>> Adrian
>> 
>> > -----Original Message-----
>> > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
>> > Behalf
>> > Of Tim Niemeyer
>> > Sent: Samstag, 20. Januar 2018 14:54
>> > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
>> > dev@freifunk.net; Jan Kraus <mayosemmel@gmail.com>
>> > Subject: Re: [PATCH] GL-AR150: Fix ETHMESHMAC
>> > 
>> > Hi
>> > 
>> > Am Mittwoch, den 03.01.2018, 19:24 +0100 schrieb Adrian Schmutzler:
>> > > The AR150 seems to have the same MAC address on eth0, eth1 and
>> > > phy0.
>> > > Although it is not a ONEPORT, we can use the shifted bit like for
>> > > the
>> > > ONEPORT devices to get a distinct ETHMESHMAC.
>> > 
>> > Hm..
>> > 
>> > Das Gerät wurde damals von Mayosemmel eingepflegt. Wäre gut, wenn
>> > er
>> > deine Änderung kurz bestätigen könnte.
>> > 
>> > Ansonsten von mir
>> > Reviewed-by: Tim Niemeyer <tim@tn-x.org>
>> > 
>> > Tim
>> > 
>> > > 
>> > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>> > > ---
>> > >  src/packages/fff/fff-network/ar71xx/network.gl-ar150 | 4 +++-
>> > >  1 file changed, 3 insertions(+), 1 deletion(-)
>> > > 
>> > > diff --git a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
>> > > b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
>> > > index 9e05f73..64049dd 100644
>> > > --- a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
>> > > +++ b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
>> > > @@ -1,3 +1,5 @@
>> > > +. /lib/functions/fff/network
>> > > +
>> > >  PORTORDER="1"
>> > > 
>> > >  WANDEV=eth0
>> > > @@ -6,8 +8,8 @@ WAN_PORTS=
>> > >  BATMAN_PORTS="0t"
>> > >  CLIENT_PORTS="0t"
>> > > 
>> > > -ETHMESHMAC=$(cat /sys/class/net/eth1/address)
>> > >  ROUTERMAC=$(cat /sys/class/net/eth0/address)
>> > > +ETHMESHMAC="$(macFlipLocalBit "$ROUTERMAC")"
>> > > 
>> > >  . /etc/network.mode
>> > > 
>> > > --
>> > > 2.7.4
>> > > 
>> 
>>
Jan Kraus Jan. 21, 2018, 10:57 a.m.
Hi,

ich hab mir das grad mal angeschaut.
Die Mac Adresse ist tatsächlich auf beiden Interfaces identisch. Ich
bilde mir aber ein, das war damals als ich den eingepflegt hab noch
nicht so.
Seis drum: Reviewed-by: Jan Kraus <mayosemmel@gmail.com

Grüße Jan

Am Samstag, den 20.01.2018, 15:05 +0100 schrieb Tim Niemeyer:
> Am Samstag, den 20.01.2018, 15:02 +0100 schrieb
> mail@adrianschmutzler.de:
> > Hallo Tim/Liste,
> > 
> > mich wundert das auch. Kann durchaus auch sein, dass die zweite MAC
> > mal im LEDE verloren gegangen ist. Auf allen Geräten, die ich im
> > Monitoring gefunden habe, war's kaputt.
> 
> Jo, soll Jan sich mal kurz zu äußern, dann passt das schon.
> 
> Er hat den Support dafür eingebaut, ich hoffe er hat noch so ein Teil
> und er wollte sich ja eh wieder mehr mit dem Thema befassen. Das ist
> ja
> jetzt ne prima Gelegenheit.
> 
> Tim
> 
> > Grüße
> > 
> > Adrian
> > 
> > > -----Original Message-----
> > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> > > Behalf
> > > Of Tim Niemeyer
> > > Sent: Samstag, 20. Januar 2018 14:54
> > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > > dev@freifunk.net; Jan Kraus <mayosemmel@gmail.com>
> > > Subject: Re: [PATCH] GL-AR150: Fix ETHMESHMAC
> > > 
> > > Hi
> > > 
> > > Am Mittwoch, den 03.01.2018, 19:24 +0100 schrieb Adrian
> > > Schmutzler:
> > > > The AR150 seems to have the same MAC address on eth0, eth1 and
> > > > phy0.
> > > > Although it is not a ONEPORT, we can use the shifted bit like
> > > > for
> > > > the
> > > > ONEPORT devices to get a distinct ETHMESHMAC.
> > > 
> > > Hm..
> > > 
> > > Das Gerät wurde damals von Mayosemmel eingepflegt. Wäre gut, wenn
> > > er
> > > deine Änderung kurz bestätigen könnte.
> > > 
> > > Ansonsten von mir
> > > Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> > > 
> > > Tim
> > > 
> > > > 
> > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > > ---
> > > >  src/packages/fff/fff-network/ar71xx/network.gl-ar150 | 4 +++-
> > > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > > 
> > > > diff --git a/src/packages/fff/fff-network/ar71xx/network.gl-
> > > > ar150
> > > > b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > > index 9e05f73..64049dd 100644
> > > > --- a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > > +++ b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > > @@ -1,3 +1,5 @@
> > > > +. /lib/functions/fff/network
> > > > +
> > > >  PORTORDER="1"
> > > > 
> > > >  WANDEV=eth0
> > > > @@ -6,8 +8,8 @@ WAN_PORTS=
> > > >  BATMAN_PORTS="0t"
> > > >  CLIENT_PORTS="0t"
> > > > 
> > > > -ETHMESHMAC=$(cat /sys/class/net/eth1/address)
> > > >  ROUTERMAC=$(cat /sys/class/net/eth0/address)
> > > > +ETHMESHMAC="$(macFlipLocalBit "$ROUTERMAC")"
> > > > 
> > > >  . /etc/network.mode
> > > > 
> > > > --
> > > > 2.7.4
> > > >
Tim Niemeyer Jan. 21, 2018, 12:08 p.m.
Hi

Am Sonntag, den 21.01.2018, 11:57 +0100 schrieb Jan Kraus:
> Hi,
> 
> ich hab mir das grad mal angeschaut.
> Die Mac Adresse ist tatsächlich auf beiden Interfaces identisch. Ich
> bilde mir aber ein, das war damals als ich den eingepflegt hab noch
> nicht so.
Danke fürs nachgucken.

> Seis drum: Reviewed-by: Jan Kraus <mayosemmel@gmail.com
Achtung, das Patchwork erkennt das scheinbar nur, wenn es am
Zeilenanfang steht.

Patch ist applied.

Tim

> 
> Grüße Jan
> 
> Am Samstag, den 20.01.2018, 15:05 +0100 schrieb Tim Niemeyer:
> > Am Samstag, den 20.01.2018, 15:02 +0100 schrieb
> > mail@adrianschmutzler.de:
> > > Hallo Tim/Liste,
> > > 
> > > mich wundert das auch. Kann durchaus auch sein, dass die zweite
> > > MAC
> > > mal im LEDE verloren gegangen ist. Auf allen Geräten, die ich im
> > > Monitoring gefunden habe, war's kaputt.
> > 
> > Jo, soll Jan sich mal kurz zu äußern, dann passt das schon.
> > 
> > Er hat den Support dafür eingebaut, ich hoffe er hat noch so ein
> > Teil
> > und er wollte sich ja eh wieder mehr mit dem Thema befassen. Das
> > ist
> > ja
> > jetzt ne prima Gelegenheit.
> > 
> > Tim
> > 
> > > Grüße
> > > 
> > > Adrian
> > > 
> > > > -----Original Message-----
> > > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> > > > Behalf
> > > > Of Tim Niemeyer
> > > > Sent: Samstag, 20. Januar 2018 14:54
> > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > > > dev@freifunk.net; Jan Kraus <mayosemmel@gmail.com>
> > > > Subject: Re: [PATCH] GL-AR150: Fix ETHMESHMAC
> > > > 
> > > > Hi
> > > > 
> > > > Am Mittwoch, den 03.01.2018, 19:24 +0100 schrieb Adrian
> > > > Schmutzler:
> > > > > The AR150 seems to have the same MAC address on eth0, eth1
> > > > > and
> > > > > phy0.
> > > > > Although it is not a ONEPORT, we can use the shifted bit like
> > > > > for
> > > > > the
> > > > > ONEPORT devices to get a distinct ETHMESHMAC.
> > > > 
> > > > Hm..
> > > > 
> > > > Das Gerät wurde damals von Mayosemmel eingepflegt. Wäre gut,
> > > > wenn
> > > > er
> > > > deine Änderung kurz bestätigen könnte.
> > > > 
> > > > Ansonsten von mir
> > > > Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> > > > 
> > > > Tim
> > > > 
> > > > > 
> > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.d
> > > > > e>
> > > > > ---
> > > > >  src/packages/fff/fff-network/ar71xx/network.gl-ar150 | 4
> > > > > +++-
> > > > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > > > 
> > > > > diff --git a/src/packages/fff/fff-network/ar71xx/network.gl-
> > > > > ar150
> > > > > b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > > > index 9e05f73..64049dd 100644
> > > > > --- a/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > > > +++ b/src/packages/fff/fff-network/ar71xx/network.gl-ar150
> > > > > @@ -1,3 +1,5 @@
> > > > > +. /lib/functions/fff/network
> > > > > +
> > > > >  PORTORDER="1"
> > > > > 
> > > > >  WANDEV=eth0
> > > > > @@ -6,8 +8,8 @@ WAN_PORTS=
> > > > >  BATMAN_PORTS="0t"
> > > > >  CLIENT_PORTS="0t"
> > > > > 
> > > > > -ETHMESHMAC=$(cat /sys/class/net/eth1/address)
> > > > >  ROUTERMAC=$(cat /sys/class/net/eth0/address)
> > > > > +ETHMESHMAC="$(macFlipLocalBit "$ROUTERMAC")"
> > > > > 
> > > > >  . /etc/network.mode
> > > > > 
> > > > > --
> > > > > 2.7.4