[PATCH 1/2] fff-network: Only use one Switch Port on Archer C7v2

[Fabian Bläse]

On 03.12.19 15:47, Adrian Schmutzler wrote:
> Hallo,
> 
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of
>> Fabian Bläse
>> Sent: Dienstag, 3. Dezember 2019 15:23
>> To: franken-dev at freifunk.net
>> Subject: [PATCH 1/2] fff-network: Only use one Switch Port on Archer C7v2
>>
>> Previously the TP-Link Archer C7v2 was configured to use
>> two Switch CPU Interfaces. One for Trunk (Client + Batman), one
>> for WAN.
>>
>> As this setup is very uncommon it is converted to a single trunk port
>> setup just like most of the other routers with integrated managed switches.
> 
> Also "very uncommon" ist es nicht, ich würde nicht mal behaupten, dass es "uncommon" ist. Streng genommen wäre der 1043 v4/v5 genauso, den haben wir bloß schon immer falsch aufgesetzt. Dementsprechend würde ich hier nicht den Eindruck erwecken, dass das so sei, sondern einfach feststellen, dass es für unsere Firmware (im Moment) einfacher ist, alles an eth0 zu hängen.

Tatsache. Ich hab vorhin irgendeinen Mist gegrept. Da gibts noch einige Geräte die das ebenfalls so machen.
Das ist natürlich unglücklich. Ich wäre ja fast dafür, alle Geräte dahingehend anzupassen, dass sie nur den einen CPU Port benutzen. Dann muss man nicht so viel zwischen tagged/untagged, vlan-interface/interface etc. unterscheiden.

Für die Layer3 Variante ist das aktuell auf jeden Fall ein Problem, weil der CPU-Port dort ja automatisch gesetzt wird.
Man ist dann viel weniger flexibel, auf welchen Port man jetzt WAN haben möchte.

>>
>> As eth0 is now used as the switchport, this change would require to setup
>> a different ETHMESHMAC if mac addresses would be shared between interfaces.
>>
>> The device does seems to have 4 discrete mac addresses however, so the
>> explicit ETHMESHMAC setting is completely removed instead.
>>
>> Signed-off-by: Fabian Bläse <fabian at blaese.de>
>> ---
>>  src/packages/fff/fff-network/ar71xx/network.archer-c7-v2 | 5 ++---
>>  1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/src/packages/fff/fff-network/ar71xx/network.archer-c7-v2
>> b/src/packages/fff/fff-network/ar71xx/network.archer-c7-v2
>> index 593bc2a..aa7251a 100644
>> --- a/src/packages/fff/fff-network/ar71xx/network.archer-c7-v2
>> +++ b/src/packages/fff/fff-network/ar71xx/network.archer-c7-v2
>> @@ -1,8 +1,7 @@
>>  WANDEV=eth0
>> -SWITCHDEV=eth1
>> +SWITCHDEV=eth0
>>  CLIENT_PORTS="4 5 0t"
>> -WAN_PORTS="1 6"
>> +WAN_PORTS="1 0t"
>>  BATMAN_PORTS="2 3 0t"
> 
> Funktioniert das wirklich? Eigentlich müsste doch eth0=6t sein, oder?
Stimmt. Test steht noch aus, hab so ein Gerät leider nicht selbst.

> 
> Also entweder man nimmt 0t und eth1 für alles oder 6t und eth0 für alles...
> 
> https://github.com/openwrt/openwrt/blob/master/target/linux/ar71xx/base-files/etc/board.d/02_network#L240
> 
> Vergiss nicht zu prüfen ob sich der CPU-Port in dem anderen Skript ändert.
Mit dem 0t und eth0 würde es ja aktuell passen, da das aber vermutlich falsch ist, müsste man das noch entsprechend ändern..

Gruß
Fabian

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20191203/ec702cd8/attachment.sig>