WR841-v7: get routermac from phy0 instead of wlan0

Submitted by Tim Niemeyer on Jan. 21, 2018, 12:05 p.m.

Details

Message ID 20180121120553.12090-1-tim@tn-x.org
State Accepted
Headers show

Commit Message

Tim Niemeyer Jan. 21, 2018, 12:05 p.m.
Fixes: #71
Signed-off-by: Tim Niemeyer <tim@tn-x.org>
---

 src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7 b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
index 183e123..46fd57b 100644
--- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
+++ b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
@@ -5,4 +5,4 @@  WAN_PORTS=
 BATMAN_PORTS="3 4 0t"
 
 ETHMESHMAC=$(cat /sys/class/net/eth1/address)
-ROUTERMAC=$(cat /sys/class/net/wlan0/address)
+ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)

Comments

Christian Dresel Jan. 21, 2018, 1:36 p.m.
hi

ich kann nicht mit Sicherheit sagen ob es richtig ist aber bei
20170218-alpha fällt folgendes aus der Konsole:

root@Gartenrouter:~# cat /sys/class/ieee80211/phy0/macaddress
74:ea:3a:c1:66:cc
root@Gartenrouter:~# ifconfig br-mesh
br-mesh   Link encap:Ethernet  HWaddr 74:EA:3A:C1:66:CB
          inet6 addr: fdff::74ea:3ac1:66cb/64 Scope:Global
          inet6 addr: fe80::76ea:3aff:fec1:66cb/64 Scope:Link
          inet6 addr: fdff::76ea:3aff:fec1:66cb/64 Scope:Global
          inet6 addr: fdff::1/64 Scope:Global
          UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
          RX packets:1162832 errors:0 dropped:4 overruns:0 frame:0
          TX packets:441340 errors:0 dropped:0 overruns:0 carrier:0
          collisions:0 txqueuelen:1000
          RX bytes:62014065 (59.1 MiB)  TX bytes:213210555 (203.3 MiB)

ich bin mir relativ sicher, das 66:cb NICHT die MAC ist die auf dem
Aufkleber stand, ich glaube(!) man musste immer -1 vom Aufkleber rechnen
so KÖNNTE 74:ea:3a:c1:66:cc richtig sein (und mit -1 kam man dann auf
66:cb womit ich mich aktuell auch verbinden kann).

Mit diesem Hintergrundwissen ein vorsichtiges:

Reviewed-by: Christian Dresel <fff@chrisi01.de>

ohne es wirklich testen zu können.

mfg

Christian

On 21.01.2018 13:05, Tim Niemeyer wrote:
> Fixes: #71
> Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> ---
> 
>  src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7 b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
> index 183e123..46fd57b 100644
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
> +++ b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
> @@ -5,4 +5,4 @@ WAN_PORTS=
>  BATMAN_PORTS="3 4 0t"
>  
>  ETHMESHMAC=$(cat /sys/class/net/eth1/address)
> -ROUTERMAC=$(cat /sys/class/net/wlan0/address)
> +ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)
>
Tim Niemeyer Jan. 21, 2018, 1:40 p.m.
Hi

Danke fürs drauf gucken.

Am Sonntag, den 21.01.2018, 14:36 +0100 schrieb Christian Dresel:
> Mit diesem Hintergrundwissen ein vorsichtiges:
> 
> Reviewed-by: Christian Dresel <fff@chrisi01.de>
> 
> ohne es wirklich testen zu können.
Das können wir grad alle nicht und es hat sich ja seit dem 2017-11-03
auch keiner sonst gemeldet. Da durch einige Umbauten der Support kaputt
geht, wenn wir da kein gültiges Interface drin haben: lieber ne falsche
Mac als ganz kaputt.
Also entweder jemand meldet sich noch fix, der das testen kann oder wir
applien das einfach so auf gut Glück. Die bittere Alternative wäre den
Support für das Device ganz zu kippen.

Tim
Tim Niemeyer Jan. 21, 2018, 1:56 p.m.
Hi

Am Sonntag, den 21.01.2018, 14:40 +0100 schrieb Tim Niemeyer:
> Hi
> 
> Danke fürs drauf gucken.
> 
> Am Sonntag, den 21.01.2018, 14:36 +0100 schrieb Christian Dresel:
> > Mit diesem Hintergrundwissen ein vorsichtiges:
> > 
> > Reviewed-by: Christian Dresel <fff@chrisi01.de>
> > 
> > ohne es wirklich testen zu können.
> 
> Das können wir grad alle nicht und es hat sich ja seit dem 2017-11-03
> auch keiner sonst gemeldet. Da durch einige Umbauten der Support
> kaputt
> geht, wenn wir da kein gültiges Interface drin haben: lieber ne
> falsche
> Mac als ganz kaputt.
> Also entweder jemand meldet sich noch fix, der das testen kann oder
> wir
> applien das einfach so auf gut Glück. 
Sry, ich habs mir anders überlegt. Wenn jemand noch testen will, kann
er das ja auch noch während der nächsten beta/alpha etc machen.

Applied.

Tim

> Die bittere Alternative wäre den
> Support für das Device ganz zu kippen.
> 
> Tim
Adrian Schmutzler Jan. 21, 2018, 11:18 p.m.
Hallo,

anhand der jetzigen Situation im Monitoring müssten wir eth0 für die Router-MAC nehmen:

https://monitoring.freifunk-franken.de/routers/4395
https://monitoring.freifunk-franken.de/routers/5149

Dies wäre konsistent im Vergleich zur alten Firmware (20170218-alpha und 20170918).

Der Wert ist tatsächlich der vom w2mesh minus 1.

Solange wir nicht wissen, was auf dem Aufkleber steht, schlage ich daher vor, die Kontinuität zu bevorzugen und 

ROUTERMAC=$(cat /sys/class/net/eth0/address)

zu verwenden.

Ich mach auch selber einen Patch.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Christian Dresel
> Sent: Sonntag, 21. Januar 2018 14:36
> To: Tim Niemeyer <tim@tn-x.org>; franken-dev@freifunk.net
> Subject: Re: [PATCH] WR841-v7: get routermac from phy0 instead of wlan0
> 
> hi
> 
> ich kann nicht mit Sicherheit sagen ob es richtig ist aber bei 20170218-alpha
> fällt folgendes aus der Konsole:
> 
> root@Gartenrouter:~# cat /sys/class/ieee80211/phy0/macaddress
> 74:ea:3a:c1:66:cc
> root@Gartenrouter:~# ifconfig br-mesh
> br-mesh   Link encap:Ethernet  HWaddr 74:EA:3A:C1:66:CB
>           inet6 addr: fdff::74ea:3ac1:66cb/64 Scope:Global
>           inet6 addr: fe80::76ea:3aff:fec1:66cb/64 Scope:Link
>           inet6 addr: fdff::76ea:3aff:fec1:66cb/64 Scope:Global
>           inet6 addr: fdff::1/64 Scope:Global
>           UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
>           RX packets:1162832 errors:0 dropped:4 overruns:0 frame:0
>           TX packets:441340 errors:0 dropped:0 overruns:0 carrier:0
>           collisions:0 txqueuelen:1000
>           RX bytes:62014065 (59.1 MiB)  TX bytes:213210555 (203.3 MiB)
> 
> ich bin mir relativ sicher, das 66:cb NICHT die MAC ist die auf dem Aufkleber
> stand, ich glaube(!) man musste immer -1 vom Aufkleber rechnen so
> KÖNNTE 74:ea:3a:c1:66:cc richtig sein (und mit -1 kam man dann auf 66:cb
> womit ich mich aktuell auch verbinden kann).
> 
> Mit diesem Hintergrundwissen ein vorsichtiges:
> 
> Reviewed-by: Christian Dresel <fff@chrisi01.de>
> 
> ohne es wirklich testen zu können.
> 
> mfg
> 
> Christian
> 
> On 21.01.2018 13:05, Tim Niemeyer wrote:
> > Fixes: #71
> > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > ---
> >
> >  src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7 | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
> > b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
> > index 183e123..46fd57b 100644
> > --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
> > +++ b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
> > @@ -5,4 +5,4 @@ WAN_PORTS=
> >  BATMAN_PORTS="3 4 0t"
> >
> >  ETHMESHMAC=$(cat /sys/class/net/eth1/address) -ROUTERMAC=$(cat
> > /sys/class/net/wlan0/address)
> > +ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)
> >
Christian Dresel Jan. 22, 2018, 11:01 a.m.
Hallo Adrian

On 22.01.2018 00:18, mail@adrianschmutzler.de wrote:
> Hallo,
> 
> anhand der jetzigen Situation im Monitoring müssten wir eth0 für die Router-MAC nehmen:
> 
> https://monitoring.freifunk-franken.de/routers/4395
> https://monitoring.freifunk-franken.de/routers/5149
> 
> Dies wäre konsistent im Vergleich zur alten Firmware (20170218-alpha und 20170918).
> 
> Der Wert ist tatsächlich der vom w2mesh minus 1.
> 
> Solange wir nicht wissen, was auf dem Aufkleber steht, schlage ich daher vor, die Kontinuität zu bevorzugen und 

ich kann es nicht 100%ig bestätigen aber ich bin mir recht sicher das
die aktuelle MAC auf br-mesh falsch ist.

https://monitoring.freifunk-franken.de/routers/4395

ich bin mir recht sicher, das ich immer -1 vom Aufkleber rechnen musste,
damit ich auf den Router drauf komme. D.h. Auf den Aufkleber wird
vermutlich :CC stehen (ich kann aber leider nicht nachgucken...) und das
aktuell :CB ist falsch würde aber mit der -1 vom Aufkleber Rechnung dann
"richtig" werden damit ich den Router erreiche. Demnach bin ich der
Meinung wir sollten RedDog's Lösung nehmen (ich komm per remote drauf
und aus dem phy0 fällt die :CC raus).

mfg

Christian

> 
> ROUTERMAC=$(cat /sys/class/net/eth0/address)
> 
> zu verwenden.
> 
> Ich mach auch selber einen Patch.
> 
> Grüße
> 
> Adrian
> 
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
>> Of Christian Dresel
>> Sent: Sonntag, 21. Januar 2018 14:36
>> To: Tim Niemeyer <tim@tn-x.org>; franken-dev@freifunk.net
>> Subject: Re: [PATCH] WR841-v7: get routermac from phy0 instead of wlan0
>>
>> hi
>>
>> ich kann nicht mit Sicherheit sagen ob es richtig ist aber bei 20170218-alpha
>> fällt folgendes aus der Konsole:
>>
>> root@Gartenrouter:~# cat /sys/class/ieee80211/phy0/macaddress
>> 74:ea:3a:c1:66:cc
>> root@Gartenrouter:~# ifconfig br-mesh
>> br-mesh   Link encap:Ethernet  HWaddr 74:EA:3A:C1:66:CB
>>           inet6 addr: fdff::74ea:3ac1:66cb/64 Scope:Global
>>           inet6 addr: fe80::76ea:3aff:fec1:66cb/64 Scope:Link
>>           inet6 addr: fdff::76ea:3aff:fec1:66cb/64 Scope:Global
>>           inet6 addr: fdff::1/64 Scope:Global
>>           UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
>>           RX packets:1162832 errors:0 dropped:4 overruns:0 frame:0
>>           TX packets:441340 errors:0 dropped:0 overruns:0 carrier:0
>>           collisions:0 txqueuelen:1000
>>           RX bytes:62014065 (59.1 MiB)  TX bytes:213210555 (203.3 MiB)
>>
>> ich bin mir relativ sicher, das 66:cb NICHT die MAC ist die auf dem Aufkleber
>> stand, ich glaube(!) man musste immer -1 vom Aufkleber rechnen so
>> KÖNNTE 74:ea:3a:c1:66:cc richtig sein (und mit -1 kam man dann auf 66:cb
>> womit ich mich aktuell auch verbinden kann).
>>
>> Mit diesem Hintergrundwissen ein vorsichtiges:
>>
>> Reviewed-by: Christian Dresel <fff@chrisi01.de>
>>
>> ohne es wirklich testen zu können.
>>
>> mfg
>>
>> Christian
>>
>> On 21.01.2018 13:05, Tim Niemeyer wrote:
>>> Fixes: #71
>>> Signed-off-by: Tim Niemeyer <tim@tn-x.org>
>>> ---
>>>
>>>  src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7 | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>>> b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>>> index 183e123..46fd57b 100644
>>> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>>> +++ b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>>> @@ -5,4 +5,4 @@ WAN_PORTS=
>>>  BATMAN_PORTS="3 4 0t"
>>>
>>>  ETHMESHMAC=$(cat /sys/class/net/eth1/address) -ROUTERMAC=$(cat
>>> /sys/class/net/wlan0/address)
>>> +ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)
>>>
> 
>
Tim Niemeyer Jan. 22, 2018, 11:08 a.m.
Hi

Am 22. Januar 2018 00:18:50 MEZ schrieb mail@adrianschmutzler.de:
>Hallo,
>
>anhand der jetzigen Situation im Monitoring müssten wir eth0 für die
>Router-MAC nehmen:
>
>https://monitoring.freifunk-franken.de/routers/4395
>https://monitoring.freifunk-franken.de/routers/5149
>
>Dies wäre konsistent im Vergleich zur alten Firmware (20170218-alpha
>und 20170918).
>
>Der Wert ist tatsächlich der vom w2mesh minus 1.
>
>Solange wir nicht wissen, was auf dem Aufkleber steht, schlage ich
>daher vor, die Kontinuität zu bevorzugen und 
>
>ROUTERMAC=$(cat /sys/class/net/eth0/address)
>
>zu verwenden.
>
>Ich mach auch selber einen Patch.

Lass ma. Die Kuh ist vom Eis. Das Gerät wird funktionieren und das war das Ziel von dem Ticket.

Der Patch ist bereits applied und das Ticket zu. Sry, dass es dann doch so schnell ging.

Das jetzt auf eth0 nochmal zu ändern sollten wir erst nach Feedback vom Geräteaufkleber machen.

Tim


>Grüße
>
>Adrian
>
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
>> Of Christian Dresel
>> Sent: Sonntag, 21. Januar 2018 14:36
>> To: Tim Niemeyer <tim@tn-x.org>; franken-dev@freifunk.net
>> Subject: Re: [PATCH] WR841-v7: get routermac from phy0 instead of
>wlan0
>> 
>> hi
>> 
>> ich kann nicht mit Sicherheit sagen ob es richtig ist aber bei
>20170218-alpha
>> fällt folgendes aus der Konsole:
>> 
>> root@Gartenrouter:~# cat /sys/class/ieee80211/phy0/macaddress
>> 74:ea:3a:c1:66:cc
>> root@Gartenrouter:~# ifconfig br-mesh
>> br-mesh   Link encap:Ethernet  HWaddr 74:EA:3A:C1:66:CB
>>           inet6 addr: fdff::74ea:3ac1:66cb/64 Scope:Global
>>           inet6 addr: fe80::76ea:3aff:fec1:66cb/64 Scope:Link
>>           inet6 addr: fdff::76ea:3aff:fec1:66cb/64 Scope:Global
>>           inet6 addr: fdff::1/64 Scope:Global
>>           UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
>>           RX packets:1162832 errors:0 dropped:4 overruns:0 frame:0
>>           TX packets:441340 errors:0 dropped:0 overruns:0 carrier:0
>>           collisions:0 txqueuelen:1000
>>           RX bytes:62014065 (59.1 MiB)  TX bytes:213210555 (203.3
>MiB)
>> 
>> ich bin mir relativ sicher, das 66:cb NICHT die MAC ist die auf dem
>Aufkleber
>> stand, ich glaube(!) man musste immer -1 vom Aufkleber rechnen so
>> KÖNNTE 74:ea:3a:c1:66:cc richtig sein (und mit -1 kam man dann auf
>66:cb
>> womit ich mich aktuell auch verbinden kann).
>> 
>> Mit diesem Hintergrundwissen ein vorsichtiges:
>> 
>> Reviewed-by: Christian Dresel <fff@chrisi01.de>
>> 
>> ohne es wirklich testen zu können.
>> 
>> mfg
>> 
>> Christian
>> 
>> On 21.01.2018 13:05, Tim Niemeyer wrote:
>> > Fixes: #71
>> > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
>> > ---
>> >
>> >  src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7 | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git
>a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>> > b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>> > index 183e123..46fd57b 100644
>> > --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>> > +++ b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v7
>> > @@ -5,4 +5,4 @@ WAN_PORTS=
>> >  BATMAN_PORTS="3 4 0t"
>> >
>> >  ETHMESHMAC=$(cat /sys/class/net/eth1/address) -ROUTERMAC=$(cat
>> > /sys/class/net/wlan0/address)
>> > +ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)
>> >