-
Notifications
You must be signed in to change notification settings - Fork 325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Triband radio support #3357
base: main
Are you sure you want to change the base?
Triband radio support #3357
Conversation
I think the mac address problem is as of now unresolved? #1666 (comment) |
To be honest, I did not really understand the problem with the MAC yet I see that with 294b291 we add to the prior mac part if our counter overflows. |
Would love to see the triband support! Devices with potential triband support and Gluon support I'm aware of: Plasmacloud PA2200 and the LibreRouter. Any other devices people would look forward to for triband support? Also, I guess this was untested with 6GHz so far? (I'd be looking forward to the Banana Pi BPI-R4, for instance, seems to have a dedicated phy for 6GHz for the WiFi7 iPA NIC module) |
I think @Djfe gave a quite extensive list of affected devices in #1661 (comment) For now, this does not include 6GHz. I have some ideas to add counters to the 6g band as well, but this would create to much noise and untested code (and I am not to familiar with 6GHz frequencies) , so I think we should do this one at a time. From the list of open todos, I think I can handle the first two eventually, but would definitely need help for a complete overhaul (I think this way stopping the old PR mainly?). If there is someone willing to draft something where I can built on, improve and test, I would be very happy :) |
@T-X Wifi 6E is probably another can of worms but relies on resolving the tripple radio problem #2906 @maurerle The mac address problem is from my understanding that we need more then the current 8 but this has to be carefully considered to not cause collisions (#2061, #1983). Just statically assigning more might not be the best approach |
There are concerns here regarding the probability of collision due to the reservation of the additional MAC addresses.
Regarding the assignment of the interfaces: |
01ebc90
to
e45ef70
Compare
If the last mac-address byte was
One way to solve this is to zero more than 3 bits in See: af539fc I think one can even remove the |
@@ -13,13 +14,19 @@ local function is_disabled(config, name) | |||
return config.disabled(false) | |||
end | |||
|
|||
local has_client_radio = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be renamed to has_client_only_radio
as this checks for a radio which can not be used for mesh
At the last gluon meetup we discussed having a solution which suits the needs and is not to complex. A better approach would be to first count and evaluate all items and then decide what to do, as it is done for the multi band configuration:
Looking at the device list provided by @Djfe in #1661 (comment) I came to two possible operation modes: a - there is 1 radio which supports only a subset of the channels while the other radio supports all unsupported channels (or more)
This PR should work for the usage of a and b by default, while the user can configure different behavior. TODO:
for two 2.4GHz radios, this does not disable one band by default, so it meshes on all by default - I think this is fine |
af539fc
to
6d521a2
Compare
6d521a2
to
be2e63b
Compare
be2e63b
to
8c070c8
Compare
I'd like to reopen the changes from #1666 to make way to support #1661
This works quite well and exposes the three wifis correctly on the status-page:
radio0, 5GHz, 144 (client wifi)
radio1, 2.4GHz, 11 (client + mesh)
radio2, 5GHz, 44 (mesh)
and also the config mode shows the expected default config for such devices:
We might want to:
gluon/package/gluon-core/luasrc/lib/gluon/upgrade/200-wireless
Line 15 in fc25b49
gluon.wireless
in the future (far out of scope here)This is currently focusing on having two 5GHz bands, where at least one does not support all channels.