[RFC] Add support for TP-Link Archer C7 v5

Submitted by Adrian Schmutzler on April 3, 2019, 11:20 p.m.

Details

Message ID 005301d4ea73$d1a42080$74ec6180$@adrianschmutzler.de
State Not Applicable
Headers show

Commit Message

Adrian Schmutzler April 3, 2019, 11:20 p.m.
Hallo Fabian,

Test hat soweit geklappt, auch das Batman über w5mesh scheint zu gehen.

PORTORDER="1 2 3 4 5"

Ich habe das normalerweise als erste Zeile in die network.* und danach eine Leerzeile, falls du Wert auf Konsistenz legst.

Zur ETHMESHMAC:
Die "normale" Aufteilung (Stock-FW) beim C7v5 ist folgende:
5 GHz LANMAC-1
2.4 GHz LANMAC
LAN LANMAC
WAN LANMAC +1

Mir würde es daher als sinnvoll erscheinen, für die ETHMESHMAC die WANMAC zu nehmen, weil die quasi "noch frei" ist. Das könnte dann folgendermaßen aussehen:


Ich habe zumindest mal manuell getestet, dass die Adresse so auch richtig ausgerechnet wird.

Alle Tests bisher waren basierend auf meiner Firmware, wenn du ne v2 schickst kann ich die aber gerne zur Sicherheit nochmal mit der off. Firmware bauen. Ich sehe hier keine Probleme.

Beste Grüße

Adrian


> -----Original Message-----
> From: Fabian Bläse [mailto:fabian@blaese.de]
> Sent: Mittwoch, 3. April 2019 17:30
> To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-
> dev@freifunk.net
> Subject: Re: [RFC PATCH] Add support for TP-Link Archer C7 v5
> 
> Hallo Adrian,
> 
> On 03.04.19 17:21, Adrian Schmutzler wrote:
> >> diff --git a/src/packages/fff/fff-network/ar71xx/network.archer-c7-v5
> >> b/src/packages/fff/fff-network/ar71xx/network.archer-c7-v5
> >> new file mode 100644
> >> index 0000000..58a4cf0
> >> --- /dev/null
> >> +++ b/src/packages/fff/fff-network/ar71xx/network.archer-c7-v5
> >> @@ -0,0 +1,12 @@
> >
> > PORTORDER werde ich noch herausfinden, die sollten wir hier ergänzen.
> Gern. Ich selbst hab das nicht begriffen, die richtige Zeile darf mir gern
> jemand für eine v2 mitteilen :-)
> 
> >> +WANDEV=eth0
> >> +SWITCHDEV=eth0
> >> +CLIENT_PORTS="4 5 0t"
> >> +WAN_PORTS="1 0t"
> >> +BATMAN_PORTS="2 3 0t"
> >> +CPUPORT="0t"
> >
> > CPUPORT müsste zum jetzigen Stand eigtl. wieder raus.
> Verdammt, stimmt.
> Wird Zeit, dass die GW-Firmware fertig wird..
> 
> >
> >> +
> >> +# use mac address from phy1 with 'locally administered' bit set to '1'
> >> +# only possible, because wXmesh is created first and therefore gets
> >> +the
> >> 'universally administered address'
> >> +
> >> +ETHMESHMAC=$(macFlipLocalBit "$(cat
> >> /sys/class/ieee80211/phy1/macaddress)")
> >
> > Ich denke, dass es hier mehr Sinn macht, die eth1-Adresse zu nehmen
> (eth0 +1).
> > Werde mich dazu aber nach dem Test nochmal ausführlich äußern.
> Musst du an konkreter Hardware sehen. Ich glaube aber, dass das aus
> irgendeinem Grund nicht ging..
> 
> >
> >> +ROUTERMAC=$(cat /sys/class/net/eth0/address)
> >> --
> >> 2.21.0
> >
> > Ansonsten fehlt ATH79_MACH_ARCHER_C7_V5 in dem generic-zu-tiny
> OpenWrt-Patch:
> >
> https://github.com/openwrt/openwrt/commit/bf39d5594b3c8f9409e6d9408
> a1f
> > 370c9f18d0dd#diff-d3d7f2ed32814702d2aabcea4a942da8R55
> Fehlt nicht. Alle drei Object-Files werden in
> "CONFIG_ATH79_MACH_ARCHER_C7" eingebaut. Siehe genau die Stelle, die
> du verlinkt hast :-) Wir haben nur die Kernel-Config verändert.
> 
> Gruß
> Fabian

Patch hide | download patch | download mbox

--- /dev/null
+++ b/src/packages/fff/fff-network/ar71xx-generic/network.archer-c7-v5
@@ -0,0 +1,14 @@ 
+PORTORDER="1 2 3 4 5"
+
+WANDEV=eth0
+SWITCHDEV=eth0
+CLIENT_PORTS="4 5 0t"
+WAN_PORTS="1 0t"
+BATMAN_PORTS="2 3 0t"
+
+# Use WAN-MAC, which is LAN-MAC + 1
+. /lib/functions.sh
+. /lib/functions/system.sh
+
+ROUTERMAC=$(cat /sys/class/net/eth0/address)
+ETHMESHMAC=$(macaddr_add $ROUTERMAC 1)

Comments

Fabian Blaese May 31, 2019, 3:44 p.m.
Hallo Adrian.

On 04.04.19 01:20, mail@adrianschmutzler.de wrote:
> Hallo Fabian,
> 
> Test hat soweit geklappt, auch das Batman über w5mesh scheint zu gehen.
> 
> PORTORDER="1 2 3 4 5"
Danke.

> Zur ETHMESHMAC:
> Die "normale" Aufteilung (Stock-FW) beim C7v5 ist folgende:
> 5 GHz LANMAC-1
> 2.4 GHz LANMAC
> LAN LANMAC
> WAN LANMAC +1
> 
> Mir würde es daher als sinnvoll erscheinen, für die ETHMESHMAC die WANMAC zu nehmen, weil die quasi "noch frei" ist. Das könnte dann folgendermaßen aussehen:
Beim MAC Adressen hin und her rechnen hab ich ein sehr ungutes Gefühl (siehe auch deine letzte Mail zu dem Thema).
Spricht etwas gegen die MAC, die ich verwendet hab? Ansonsten würde ich es dabei belassen.

Gruß
Fabian
Adrian Schmutzler May 31, 2019, 8:37 p.m.
Hallo Fabian,

 

OpenWrt macht bei TP-Link auch nichts anderes als eine Mac-Adresse auszulesen und dann die anderen mit +/-1 auszurechnen, je nachdem was die Original-FW benutzt. Wir kriegen das halt nur nicht mit, weil wir die dann vergebenen Adressen wieder von eth0/eth1/phy0 usw. runter holen. Die WAN-MAC werfen wir nur mit der aktuellen network-config-Strategie quasi weg.

 

Ansonsten ist es wahrscheinlich ziemlich egal, ob man die WAN-MAC oder das Local-Bit nimmt. Ich versuche halt immer möglichst Adressen mit local-bit zu vermeiden.

Auf der anderen Seite hätte das local-bit den Vorteil, dass man das irgendwann sogar automatisieren könnte: Für das ethmesh einfach die bestehende MAC mit allen phys vergleichen, wenn Übereinstimmung dann Bit klappen. So könnte man ggf. die komplette ETHMESHMAC-Definition loswerden. Aber ich schweife ab …

Mach wie du möchtest, ist wohl insgesamt relativ wurscht.

 

Ansonsten würde ich noch überlegen, entweder CT-Treiber UND Firmware zu nehmen oder beides Non-CT. Ich wäre für letzteres.

 

Grüße

 

Adrian

 

From: Fabian Bläse [mailto:fabian@blaese.de] 
Sent: Freitag, 31. Mai 2019 17:44
To: mail@adrianschmutzler.de; franken-dev@freifunk.net
Subject: Re: [RFC PATCH] Add support for TP-Link Archer C7 v5

 

Hallo Adrian. 

On 04.04.19 01:20, mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de>  wrote: 
> Hallo Fabian, 
> 
> Test hat soweit geklappt, auch das Batman über w5mesh scheint zu gehen. 
> 
> PORTORDER="1 2 3 4 5" 
Danke. 

> Zur ETHMESHMAC: 
> Die "normale" Aufteilung (Stock-FW) beim C7v5 ist folgende: 
> 5 GHz LANMAC-1 
> 2.4 GHz LANMAC 
> LAN LANMAC 
> WAN LANMAC +1 
> 
> Mir würde es daher als sinnvoll erscheinen, für die ETHMESHMAC die WANMAC zu nehmen, weil die quasi "noch frei" ist. Das könnte dann folgendermaßen aussehen:

Beim MAC Adressen hin und her rechnen hab ich ein sehr ungutes Gefühl (siehe auch deine letzte Mail zu dem Thema). 
Spricht etwas gegen die MAC, die ich verwendet hab? Ansonsten würde ich es dabei belassen. 

Gruß 
Fabian
Fabian Blaese June 2, 2019, 11:02 a.m.
Hallo Adrian,

On 31.05.19 22:37, mail@adrianschmutzler.de wrote:
> OpenWrt macht bei TP-Link auch nichts anderes als eine Mac-Adresse auszulesen und dann die anderen mit +/-1 auszurechnen, je nachdem was die Original-FW benutzt. Wir kriegen das halt nur nicht mit, weil wir die dann vergebenen Adressen wieder von eth0/eth1/phy0 usw. runter holen. Die WAN-MAC werfen wir nur mit der aktuellen network-config-Strategie quasi weg.
Jo, aber OpenWRT liest die Mac Adresse aus dem Flash und kann sich (vermutlich) darauf verlassen, dass da die "richtige" für die Rechnung drin steht.
Wir lesen dagegen aus einem Interface raus.. Ich hätte dabei ein ungutes Gefühl, wenn es nicht unbedingt nötig ist.

> Auf der anderen Seite hätte das local-bit den Vorteil, dass man das irgendwann sogar automatisieren könnte: Für das ethmesh einfach die bestehende MAC mit allen phys vergleichen, wenn Übereinstimmung dann Bit klappen. So könnte man ggf. die komplette ETHMESHMAC-Definition loswerden. Aber ich schweife ab …
Über so eine automatisierung habe ich schon vor Ewigkeiten mal nachgedacht, die Idee aber wieder verworfen, da ich es zu undurchsichtig fand. Aber man kann über so etwas natürlich mal nachdenken.

> Mach wie du möchtest, ist wohl insgesamt relativ wurscht.
Ich würde meine Lösung bevorzugen und das demnächst als nicht-RFC Patch senden.

> Ansonsten würde ich noch überlegen, entweder CT-Treiber UND Firmware zu nehmen oder beides Non-CT. Ich wäre für letzteres.
Ich hatte das einfach analog zum C7v2 umgesetzt.
Für diese Diskussion gibt es einen eigenen Thread, das Ergebnis davon warte ich mal noch ab. :-)

Gruß
Fabian