CPE510: Fix ROUTERMAC

Submitted by Fabian Blaese on Nov. 3, 2017, 2:37 p.m.

Details

Message ID 20171103143731.8066-1-fabian@blaese.de
State Superseded
Headers show

Commit Message

Fabian Blaese Nov. 3, 2017, 2:37 p.m.
This adds missing ROUTERMAC variable for CPE510

Signed-off-by: Fabian Bläse <fabian@blaese.de>
---
 src/packages/fff/fff-network/ar71xx/network.cpe510 | 1 +
 1 file changed, 1 insertion(+)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-network/ar71xx/network.cpe510 b/src/packages/fff/fff-network/ar71xx/network.cpe510
index 6764c64..97269e5 100644
--- a/src/packages/fff/fff-network/ar71xx/network.cpe510
+++ b/src/packages/fff/fff-network/ar71xx/network.cpe510
@@ -4,6 +4,7 @@  CLIENT_PORTS="0t"
 WAN_PORTS="0t"
 BATMAN_PORTS="0t"
 
+ROUTERMAC="eth0"
 ETHMESHMAC="w2ap"
 
 . /etc/network.mode

Comments

Tim Niemeyer Nov. 3, 2017, 2:40 p.m.
Hi

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

Tim

Am Freitag, den 03.11.2017, 15:37 +0100 schrieb Fabian Bläse:
> This adds missing ROUTERMAC variable for CPE510
> 
> Signed-off-by: Fabian Bläse <fabian@blaese.de>
> ---
>  src/packages/fff/fff-network/ar71xx/network.cpe510 | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/packages/fff/fff-network/ar71xx/network.cpe510
> b/src/packages/fff/fff-network/ar71xx/network.cpe510
> index 6764c64..97269e5 100644
> --- a/src/packages/fff/fff-network/ar71xx/network.cpe510
> +++ b/src/packages/fff/fff-network/ar71xx/network.cpe510
> @@ -4,6 +4,7 @@ CLIENT_PORTS="0t"
>  WAN_PORTS="0t"
>  BATMAN_PORTS="0t"
>  
> +ROUTERMAC="eth0"
>  ETHMESHMAC="w2ap"
>  
>  . /etc/network.mode
> -- 
> 2.14.2
>
Adrian Schmutzler Nov. 3, 2017, 2:45 p.m.
Mag bei der Gelegenheit mal jemand kommentieren, ob das w2ap ganz unten Sinn
macht?

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Fabian Bläse
> Sent: Freitag, 3. November 2017 15:38
> To: franken-dev@freifunk.net
> Subject: [PATCH] CPE510: Fix ROUTERMAC
> 
> This adds missing ROUTERMAC variable for CPE510
> 
> Signed-off-by: Fabian Bläse <fabian@blaese.de>
> ---
>  src/packages/fff/fff-network/ar71xx/network.cpe510 | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/packages/fff/fff-network/ar71xx/network.cpe510
> b/src/packages/fff/fff-network/ar71xx/network.cpe510
> index 6764c64..97269e5 100644
> --- a/src/packages/fff/fff-network/ar71xx/network.cpe510
> +++ b/src/packages/fff/fff-network/ar71xx/network.cpe510
> @@ -4,6 +4,7 @@ CLIENT_PORTS="0t"
>  WAN_PORTS="0t"
>  BATMAN_PORTS="0t"
> 
> +ROUTERMAC="eth0"
>  ETHMESHMAC="w2ap"
> 
>  . /etc/network.mode
> --
> 2.14.2
> 
> --
> franken-dev mailing list
> franken-dev@freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Fabian Blaese Nov. 3, 2017, 2:50 p.m.
Ja, aktuell tut es das. w2ap wird von LEDE als zweites oder drittes Device angelegt und erhält somit die Mac Adresse des phy0 mit gekipptem “locally administered” bit (und ggf. angepasster OID). Diese unterscheidet sich daher von der eth0, was für einwandfreies funktionieren von batman nötig ist. (ACHTUNG: eth0mac == phy0mac bei cpe210/510, daher ist das setzen zwingend notwendig!)

Hier sollte man sich in Zukunft möglichst nicht auf das Wifi Device (w2ap) verlassen, sondern aus der phy0 lesen und selbst das bit kippen.

Fabian

> On 3. Nov 2017, at 15:45, Adrian Schmutzler <mail@adrianschmutzler.de> wrote:
> 
> Mag bei der Gelegenheit mal jemand kommentieren, ob das w2ap ganz unten Sinn
> macht?
> 
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
>> Of Fabian Bläse
>> Sent: Freitag, 3. November 2017 15:38
>> To: franken-dev@freifunk.net
>> Subject: [PATCH] CPE510: Fix ROUTERMAC
>> 
>> This adds missing ROUTERMAC variable for CPE510
>> 
>> Signed-off-by: Fabian Bläse <fabian@blaese.de>
>> ---
>> src/packages/fff/fff-network/ar71xx/network.cpe510 | 1 +
>> 1 file changed, 1 insertion(+)
>> 
>> diff --git a/src/packages/fff/fff-network/ar71xx/network.cpe510
>> b/src/packages/fff/fff-network/ar71xx/network.cpe510
>> index 6764c64..97269e5 100644
>> --- a/src/packages/fff/fff-network/ar71xx/network.cpe510
>> +++ b/src/packages/fff/fff-network/ar71xx/network.cpe510
>> @@ -4,6 +4,7 @@ CLIENT_PORTS="0t"
>> WAN_PORTS="0t"
>> BATMAN_PORTS="0t"
>> 
>> +ROUTERMAC="eth0"
>> ETHMESHMAC="w2ap"
>> 
>> . /etc/network.mode
>> --
>> 2.14.2
>> 
>> --
>> franken-dev mailing list
>> franken-dev@freifunk.net
>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
>
Adrian Schmutzler Nov. 3, 2017, 2:55 p.m.
Auch wenn es um ein reines 5 GHz Device geht, das gar kein w2ap hat?

> -----Original Message-----
> From: Fabian Bläse [mailto:fabian@blaese.de]
> Sent: Freitag, 3. November 2017 15:50
> To: Adrian Schmutzler <mail@adrianschmutzler.de>
> Cc: franken-dev@freifunk.net
> Subject: Re: [PATCH] CPE510: Fix ROUTERMAC
> 
> Ja, aktuell tut es das. w2ap wird von LEDE als zweites oder drittes Device
> angelegt und erhält somit die Mac Adresse des phy0 mit gekipptem “locally
> administered” bit (und ggf. angepasster OID). Diese unterscheidet sich daher
> von der eth0, was für einwandfreies funktionieren von batman nötig ist.
> (ACHTUNG: eth0mac == phy0mac bei cpe210/510, daher ist das setzen
> zwingend notwendig!)
> 
> Hier sollte man sich in Zukunft möglichst nicht auf das Wifi Device (w2ap)
> verlassen, sondern aus der phy0 lesen und selbst das bit kippen.
> 
> Fabian
> 
> > On 3. Nov 2017, at 15:45, Adrian Schmutzler <mail@adrianschmutzler.de>
> wrote:
> >
> > Mag bei der Gelegenheit mal jemand kommentieren, ob das w2ap ganz
> > unten Sinn macht?
> >
> >> -----Original Message-----
> >> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> Behalf
> >> Of Fabian Bläse
> >> Sent: Freitag, 3. November 2017 15:38
> >> To: franken-dev@freifunk.net
> >> Subject: [PATCH] CPE510: Fix ROUTERMAC
> >>
> >> This adds missing ROUTERMAC variable for CPE510
> >>
> >> Signed-off-by: Fabian Bläse <fabian@blaese.de>
> >> ---
> >> src/packages/fff/fff-network/ar71xx/network.cpe510 | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/src/packages/fff/fff-network/ar71xx/network.cpe510
> >> b/src/packages/fff/fff-network/ar71xx/network.cpe510
> >> index 6764c64..97269e5 100644
> >> --- a/src/packages/fff/fff-network/ar71xx/network.cpe510
> >> +++ b/src/packages/fff/fff-network/ar71xx/network.cpe510
> >> @@ -4,6 +4,7 @@ CLIENT_PORTS="0t"
> >> WAN_PORTS="0t"
> >> BATMAN_PORTS="0t"
> >>
> >> +ROUTERMAC="eth0"
> >> ETHMESHMAC="w2ap"
> >>
> >> . /etc/network.mode
> >> --
> >> 2.14.2
> >>
> >> --
> >> franken-dev mailing list
> >> franken-dev@freifunk.net
> >> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
> >
Fabian Blaese Nov. 3, 2017, 2:57 p.m.
Das ist vollkommen richtig, guter Punkt. Ist mir gar nicht aufgefallen.
Hier müsste es natürlich w5ap heißen.

Fabian

> On 3. Nov 2017, at 15:55, Adrian Schmutzler <mail@adrianschmutzler.de> wrote:
> 
> Auch wenn es um ein reines 5 GHz Device geht, das gar kein w2ap hat?
> 
>> -----Original Message-----
>> From: Fabian Bläse [mailto:fabian@blaese.de]
>> Sent: Freitag, 3. November 2017 15:50
>> To: Adrian Schmutzler <mail@adrianschmutzler.de>
>> Cc: franken-dev@freifunk.net
>> Subject: Re: [PATCH] CPE510: Fix ROUTERMAC
>> 
>> Ja, aktuell tut es das. w2ap wird von LEDE als zweites oder drittes Device
>> angelegt und erhält somit die Mac Adresse des phy0 mit gekipptem “locally
>> administered” bit (und ggf. angepasster OID). Diese unterscheidet sich daher
>> von der eth0, was für einwandfreies funktionieren von batman nötig ist.
>> (ACHTUNG: eth0mac == phy0mac bei cpe210/510, daher ist das setzen
>> zwingend notwendig!)
>> 
>> Hier sollte man sich in Zukunft möglichst nicht auf das Wifi Device (w2ap)
>> verlassen, sondern aus der phy0 lesen und selbst das bit kippen.
>> 
>> Fabian
>> 
>>> On 3. Nov 2017, at 15:45, Adrian Schmutzler <mail@adrianschmutzler.de>
>> wrote:
>>> 
>>> Mag bei der Gelegenheit mal jemand kommentieren, ob das w2ap ganz
>>> unten Sinn macht?
>>> 
>>>> -----Original Message-----
>>>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
>> Behalf
>>>> Of Fabian Bläse
>>>> Sent: Freitag, 3. November 2017 15:38
>>>> To: franken-dev@freifunk.net
>>>> Subject: [PATCH] CPE510: Fix ROUTERMAC
>>>> 
>>>> This adds missing ROUTERMAC variable for CPE510
>>>> 
>>>> Signed-off-by: Fabian Bläse <fabian@blaese.de>
>>>> ---
>>>> src/packages/fff/fff-network/ar71xx/network.cpe510 | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>> 
>>>> diff --git a/src/packages/fff/fff-network/ar71xx/network.cpe510
>>>> b/src/packages/fff/fff-network/ar71xx/network.cpe510
>>>> index 6764c64..97269e5 100644
>>>> --- a/src/packages/fff/fff-network/ar71xx/network.cpe510
>>>> +++ b/src/packages/fff/fff-network/ar71xx/network.cpe510
>>>> @@ -4,6 +4,7 @@ CLIENT_PORTS="0t"
>>>> WAN_PORTS="0t"
>>>> BATMAN_PORTS="0t"
>>>> 
>>>> +ROUTERMAC="eth0"
>>>> ETHMESHMAC="w2ap"
>>>> 
>>>> . /etc/network.mode
>>>> --
>>>> 2.14.2
>>>> 
>>>> --
>>>> franken-dev mailing list
>>>> franken-dev@freifunk.net
>>>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
>>> 
> 
>
Tim Niemeyer Nov. 3, 2017, 2:59 p.m.
Hi

Am Freitag, den 03.11.2017, 15:57 +0100 schrieb Fabian Bläse:
> Das ist vollkommen richtig, guter Punkt. Ist mir gar nicht
> aufgefallen.
> Hier müsste es natürlich w5ap heißen.
Lass kaputt und mach n Ticket. Das war schon immer kaputt.

Wir können das gern mit dem Thema mit fixen:
""Hier sollte man sich in Zukunft möglichst nicht auf das Wifi Device
(w2ap) verlassen, sondern aus der phy0 lesen und selbst das bit
kippen.""

Da jetzt extra n Patch zu kippen, für ein Gerät, was nur einmal im
Einsatz ist.. Nee.. Gibt grad wichtigeres.

Tim

> 
> Fabian
> 
> > On 3. Nov 2017, at 15:55, Adrian Schmutzler <mail@adrianschmutzler.
> > de> wrote:
> > 
> > Auch wenn es um ein reines 5 GHz Device geht, das gar kein w2ap
> > hat?
> > 
> > > -----Original Message-----
> > > From: Fabian Bläse [mailto:fabian@blaese.de]
> > > Sent: Freitag, 3. November 2017 15:50
> > > To: Adrian Schmutzler <mail@adrianschmutzler.de>
> > > Cc: franken-dev@freifunk.net
> > > Subject: Re: [PATCH] CPE510: Fix ROUTERMAC
> > > 
> > > Ja, aktuell tut es das. w2ap wird von LEDE als zweites oder
> > > drittes Device
> > > angelegt und erhält somit die Mac Adresse des phy0 mit gekipptem
> > > “locally
> > > administered” bit (und ggf. angepasster OID). Diese unterscheidet
> > > sich daher
> > > von der eth0, was für einwandfreies funktionieren von batman
> > > nötig ist.
> > > (ACHTUNG: eth0mac == phy0mac bei cpe210/510, daher ist das setzen
> > > zwingend notwendig!)
> > > 
> > > Hier sollte man sich in Zukunft möglichst nicht auf das Wifi
> > > Device (w2ap)
> > > verlassen, sondern aus der phy0 lesen und selbst das bit kippen.
> > > 
> > > Fabian
> > > 
> > > > On 3. Nov 2017, at 15:45, Adrian Schmutzler <mail@adrianschmutz
> > > > ler.de>
> > > 
> > > wrote:
> > > > 
> > > > Mag bei der Gelegenheit mal jemand kommentieren, ob das w2ap
> > > > ganz
> > > > unten Sinn macht?
> > > > 
> > > > > -----Original Message-----
> > > > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net]
> > > > > On
> > > 
> > > Behalf
> > > > > Of Fabian Bläse
> > > > > Sent: Freitag, 3. November 2017 15:38
> > > > > To: franken-dev@freifunk.net
> > > > > Subject: [PATCH] CPE510: Fix ROUTERMAC
> > > > > 
> > > > > This adds missing ROUTERMAC variable for CPE510
> > > > > 
> > > > > Signed-off-by: Fabian Bläse <fabian@blaese.de>
> > > > > ---
> > > > > src/packages/fff/fff-network/ar71xx/network.cpe510 | 1 +
> > > > > 1 file changed, 1 insertion(+)
> > > > > 
> > > > > diff --git a/src/packages/fff/fff-
> > > > > network/ar71xx/network.cpe510
> > > > > b/src/packages/fff/fff-network/ar71xx/network.cpe510
> > > > > index 6764c64..97269e5 100644
> > > > > --- a/src/packages/fff/fff-network/ar71xx/network.cpe510
> > > > > +++ b/src/packages/fff/fff-network/ar71xx/network.cpe510
> > > > > @@ -4,6 +4,7 @@ CLIENT_PORTS="0t"
> > > > > WAN_PORTS="0t"
> > > > > BATMAN_PORTS="0t"
> > > > > 
> > > > > +ROUTERMAC="eth0"
> > > > > ETHMESHMAC="w2ap"
> > > > > 
> > > > > . /etc/network.mode
> > > > > --
> > > > > 2.14.2
> > > > > 
> > > > > --
> > > > > franken-dev mailing list
> > > > > franken-dev@freifunk.net
> > > > > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifu
> > > > > nk.net
> > 
> > 
> 
>
Fabian Blaese Nov. 3, 2017, 3:06 p.m.
Hallo Tim

> Lass kaputt und mach n Ticket. Das war schon immer kaputt.

Bin da absolut deiner Meinung. Wir sollten dann auch diesen Patch [erstmal] ignorieren. Es gibt einige Geräte, die da Überarbeitung brauchen (z.B. der 841v7).

Fabian