Skip to content

Optimizations for band steering and config defaults / bug fixes to aggressive roaming and socket handling#18

Open
NilsRo wants to merge 6 commits into
openwrt:masterfrom
NilsRo:usteer_src_pr_2
Open

Optimizations for band steering and config defaults / bug fixes to aggressive roaming and socket handling#18
NilsRo wants to merge 6 commits into
openwrt:masterfrom
NilsRo:usteer_src_pr_2

Conversation

@NilsRo
Copy link
Copy Markdown
Contributor

@NilsRo NilsRo commented May 24, 2026

This PR continues the work started in #10 and ports the last changes, improvements, and bug fixes from usteer‑ng back to the primary usteer project. The goal is to ensure that recent developments in usteer‑ng are available to all users of usteer.

Optimizations

  • Added default values for missing configuration items
  • Harmonized the calculation of default values
  • Added a new section to the README to help first‑time users set up usteer

Bugs

  • Stations without 5 GHz support sometimes attempted to steer to 5 GHz indefinitely. This is now prevented by requiring the station to connect to 5 GHz first. Later this should be replaced by using station metadata from UBUS to avoid this detection method.
  • Avoided creating an unnecessary socket in local mode to improve security
  • The load threshold was never applied due to a true == false check
  • Syslog is the default logger during startup. During this phase, no logging occurred unless debugging was enabled because syslog must first be loaded by UBUS config interface.

lowjoel and others added 6 commits May 24, 2026 23:43
This shows how the different values must be set relative to each other
so that the configs make sense.

In the long term this should be implemented as checks in the config
loading code and emit warnings, but documenting them is better than
status quo at the moment.

Signed-off-by: Joel Low <joel@joelsplace.sg>
Load threshold is not evaluated correctly as the result of the comparison can
be false only.

- policy: load comparison of actual and target node

Signed-off-by: Nils Hendrik Rottgardt <n.rottgardt@gmail.com>
- main,; usteer config: consolidated initial config values
- main:, usteer config: added roam_scan_snr and signal_diff_threshold default values
- README: added config recommendation

Signed-off-by: Nils Hendrik Rottgardt <n.rottgardt@gmail.com>

default

Signed-off-by: Nils Hendrik Rottgardt <n.rottgardt@gmail.com>
- main: local_mode initalisation to false explicitly
- remote: avoid socket creation in local mode

Signed-off-by: Nils Hendrik Rottgardt <n.rottgardt@gmail.com>
…lebility

- sta: moved aggressivemess update to correct location
- sta: easied entry of mac as it is case insensitve now
- ubus: added aggressivness to connected_clients api

Signed-off-by: Nils Hendrik Rottgardt <n.rottgardt@gmail.com>
Signed-off-by: Nils Hendrik Rottgardt <n.rottgardt@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants