[PATCH v3 2/4] fff-hoods/fff-wireless: Reconfigure instead of delete and create

[Adrian Schmutzler]

Habe Patches 1-3 nun wie von Fabian angeboten applied.

 

Wir können gerne über die entsprechenden Modifikationen weiter diskutieren.

 

Den Patch 4/4 werden ich mal (ohne Version) neu schicken, den hat bisher eh noch keiner angekuckt.

 

Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of Adrian Schmutzler
Sent: Montag, 22. Juli 2019 21:23
To: 'Fabian Bläse' <fabian at blaese.de>; franken-dev at freifunk.net
Subject: RE: [PATCH v3 2/4] fff-hoods/fff-wireless: Reconfigure instead of delete and create

 

Hallo,

 

um die Übersicht etwas zu verbessern, und da keinerlei Widerspruch kam, werde ich heute abend oder morgen die wireless Patches und die tunneldigger-Serie applien.

 

Ich werde mir jetzt aber erstmal Übersicht verschaffen …

 

Grüße

 

Adrian

 

From: Fabian Bläse [mailto:fabian at blaese.de] 
Sent: Mittwoch, 3. Juli 2019 14:03
To: Adrian Schmutzler <freifunk at adrianschmutzler.de <mailto:freifunk at adrianschmutzler.de> >; franken-dev at freifunk.net <mailto:franken-dev at freifunk.net> 
Subject: Re: [PATCH v3 2/4] fff-hoods/fff-wireless: Reconfigure instead of delete and create

 

Hallo Adrian, 

mir gefällt hier weiterhin nicht, dass fff-hoods Dinge tut, die imho ins fff-wireless Modul gehören. 
Darunter fällt: 
- Anlegen eines wXsta Interfaces 
- Wechsel des Mesh-Modus (während die Mesh Interfaces sehr wohl von fff-wireless angelegt werden) 

wXsta würde ich einfach bedingungslos in fff-wireless (disabled) anlegen und somit immer bereitstellen. 

Für den Wechsel des Mesh-Modus würde ich für jeden Mesh-Modus eine Funktion bereitstellen, mit der der Modus entsprechend gesetzt werden kann.

Die Funktionen haben dann die passenden Parameter für den jeweiligen Mesh Modus. 

Vor allem letzteres ist mir relativ wichtig, da für den Wechsel bekannt sein muss, welche Modi es gibt und was alles an Resten gelöscht werden muss.

Dabei sollten die Funktionen jeweils weder commit noch reload machen, das soll dann dem Aufrufer überlassen werden. 
So lässt sich dann auch leicht der Test-Modus in der Gatewayfirmware implementieren. 

Aktuell wird das aber auch schon direkt in die uci wireless settings geschrieben. Daher würde ich vorschlagen, dass ich für die Funktionen einen eigenen Patch sende, über den man dann getrennt diskutieren kann.

Daher trotz der Anmerkungen: 
Reviewed-by: Fabian Bläse <fabian at blaese.de <mailto:fabian at blaese.de> > 

Gruß 
Fabian 

 
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190722/f695ff1b/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190722/f695ff1b/attachment.sig>