fff-network: fix support for CPE and provide two-port support #143

adschm · 2021-06-09T12:48:02+02:00

adschm commented

2021-06-09 12:48:02 +02:00

Neben kleineren Änderungen erlaubt dies den Support nativer Two-Ports und ermöglicht damit, die CPE v1 Geräte wieder zu unterstützen.

Dies beinhaltet den Aufräumpatch aus #104 .

Außer shell-check ist der Code komplett ungetestet. Ich besitze selbst keines der Geräte. Mittelfristig werde ich ggf. die Logik mit einem GL-AR150 oder der Fritzbox 4040 testen.

Im Code habe ich bewusst auf die Lesbarkeit geachtet und nicht jede Doppelung wegoptimiert. Auch der Support-Level ist so begrenzt, dass die Änderung zunächt recht einfach zu realisieren ist (z.B. nur ein Batman-Port erlaubt).

Mittelfristig könnte man auch weitere Geräte auf das neue Setup migrieren:

GL-AR150
Nanostation
Fritzbox 4040 könnte jetzt supported werden

Zu beachten ist dabei jedoch, dass Two-Ports (wie bisher auch die CPEs) nicht im Web-UI unterstützt werden. Für die ersten beiden genannten Geräte würde man daher diese Konfigurationsmöglichkeit verlieren.

Neben kleineren Änderungen erlaubt dies den Support nativer Two-Ports und ermöglicht damit, die CPE v1 Geräte wieder zu unterstützen. Dies beinhaltet den Aufräumpatch aus #104 . Außer shell-check ist der Code komplett ungetestet. Ich besitze selbst keines der Geräte. Mittelfristig werde ich ggf. die Logik mit einem GL-AR150 oder der Fritzbox 4040 testen. Im Code habe ich bewusst auf die Lesbarkeit geachtet und nicht jede Doppelung wegoptimiert. Auch der Support-Level ist so begrenzt, dass die Änderung zunächt recht einfach zu realisieren ist (z.B. nur ein Batman-Port erlaubt). Mittelfristig könnte man auch weitere Geräte auf das neue Setup migrieren: - GL-AR150 - Nanostation - Fritzbox 4040 könnte jetzt supported werden Zu beachten ist dabei jedoch, dass Two-Ports (wie bisher auch die CPEs) nicht im Web-UI unterstützt werden. Für die ersten beiden genannten Geräte würde man daher diese Konfigurationsmöglichkeit verlieren.

adschm added the

packages/fff

bsp

labels 2021-06-09 12:48:02 +02:00

adschm commented

2021-06-09 12:51:18 +02:00

Achja, und Migration fehlt auch noch komplett.

Da die CPEs aber ja ohnehin LAN0MODE und LAN1MODE verwendet haben, ist wahrscheinlich gar keine Migration nötig.

Achja, und Migration fehlt auch noch komplett. Da die CPEs aber ja ohnehin LAN0MODE und LAN1MODE verwendet haben, ist wahrscheinlich gar keine Migration nötig.

adschm force-pushed twoport from 527e44e74e to 0bfc3a656d

2021-06-20 23:09:44 +02:00

Compare

adschm commented

2021-06-20 23:12:01 +02:00

Update:

Ich verwende jetzt WANDEV und SWITCHDEV, sodass man die richtige Port-Reihenfolge korrekt konfigurieren kann.
Ein paar Bugs wurden noch behoben.
Die Portzuordnung der CPEs sollte jetzt korrekt sein (habe nicht getestet, aber mit OpenWrt rekonstruiert).

Update: * Ich verwende jetzt WANDEV und SWITCHDEV, sodass man die richtige Port-Reihenfolge korrekt konfigurieren kann. * Ein paar Bugs wurden noch behoben. * Die Portzuordnung der CPEs sollte jetzt korrekt sein (habe nicht getestet, aber mit OpenWrt rekonstruiert).

fbl commented

2021-07-08 21:21:18 +02:00

Getestet auf CPE210 v1.
Tested-by: Fabian Bläse <fabian@blaese.de>

Getestet auf CPE210 v1. `Tested-by: Fabian Bläse <fabian@blaese.de>`

fbl approved these changes 2021-07-08 21:44:02 +02:00

fbl left a comment

Sieht gut aus. Danke!
Reviewed-by: Fabian Bläse <fabian@blaese.de>

Sieht gut aus. Danke! `Reviewed-by: Fabian Bläse <fabian@blaese.de>`

fbl commented

2021-07-08 22:42:14 +02:00

Migration ist an sich keine nötig, wir sollten aber vermutlich die *_PORTS Variablen in /etc/network.config aufräumen. Da stehen ja aus der swconfig Konfiguration noch Sachen drin.

Allerdings hatte ich sowieso überlegt, dieses Feature für die Node-Firmware komplett zu entfernen.

Migration ist an sich keine nötig, wir sollten aber vermutlich die `*_PORTS` Variablen in `/etc/network.config` aufräumen. Da stehen ja aus der swconfig Konfiguration noch Sachen drin. Allerdings hatte ich sowieso überlegt, dieses Feature für die Node-Firmware komplett zu entfernen.

fbl referenced this pull request

2021-07-09 09:58:58 +02:00

fff-network: Add support for DSA switch configuration #144

fbl commented

2021-07-09 10:23:56 +02:00

applied.

fbl closed this pull request

2021-07-09 10:23:56 +02:00