[PATCH] fff-network/fff-hoods: Calculate fdff IPs based on uci

Adrian Schmutzler mail at adrianschmutzler.de
Fr Dez 1 11:48:06 CET 2017


Hallo Fabian,

zunächst folgendes:

Fakt ist, dass die alte Lösung mit /sys/class/ auf den ONE_PORT Geräten NICHT funktioniert, während die uci-Lösung in 100% der Testfälle funktioniert.

Der Grund dafür könnte sein, dass die ONE_PORT-Geräte langsamer sind und daher einfach einen längeren Sleep brauchen, um die Datei lesen zu können. Vll. macht auch der ETH0MAC-Block, der nur bei den ONE-PORTS ausgeführt wird, irgendetwas böses.

Wenn im uci nichts steht, ist vorher ohnehin etwas schief gegangen. Im Moment ist dies aber aufgrund der Tests die robustere Variante.

Die Idee mit der ROUTERMAC ist grundsätzlich äquivalent, allerdings würden wir so wieder eine Lösung bauen, die von einer von uns erfundenen Variable abhängt. Im Hinblick auf die Zukunft gefällt mir also die uci-Lösung eigtl am besten.

Randnotiz:
Ich habe auch schon bei der alten Firmware Probleme auf bestimmten one-port Geräten beobachtet. Ist evtl. derselbe Grund gewesen:
https://mantis.freifunk-franken.de/view.php?id=49

Grüße

Adrian

> -----Original Message-----
> From: Fabian Bläse [mailto:fabian at blaese.de]
> Sent: Freitag, 1. Dezember 2017 10:43
> To: Adrian Schmutzler <mail at adrianschmutzler.de>; franken-
> dev at freifunk.net
> Subject: Re: [PATCH] fff-network/fff-hoods: Calculate fdff IPs based on uci
> 
> Irgendwie gefällt mir diese Änderung nicht so richtig. Das Auslesen aus UCI
> funktioniert nur, wenn die ROUTERMAC gesetzt wurde. Das ist zwar aktuell
> der Fall, besonders hübsch finde ich das aber nicht, davon abzuhängen. In
> /sys/class/… sollte genau die gleiche MAC stehen. Tut sie das nicht, sollte das
> Problem vielleicht erstmal dort gesucht werden, denn dann scheint
> irgendwas kaputt zu sein.
> 
> Was mir schon eher gefallen würde: Die Mac aus ROUTERMAC verwenden,
> falls gesetzt, andernfalls mit br-mesh aus /sys/class/... versuchen.
> 
> Fabian



Mehr Informationen über die Mailingliste franken-dev