Skip to content

yangerd: re-read wds-link ports on kernel station events - #1752

Merged
troglobit merged 3 commits into
mainfrom
yangerd-wds-stale-connected
Oct 30, 2026
Merged

troglobit merged 3 commits into
mainfrom
yangerd-wds-stale-connected

Conversation

@mattiaswal

Copy link
Copy Markdown
Contributor

A station hostapd drops for inactivity is announced disconnected a second before it is unbound from its port, so the re-read on the control event still finds it bound and connected stayed true until the next station came or went. The kernel's delete names the access point, as the station is moved back before removal, so take the nl80211 station events as a cue for the ports too.

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

@mattiaswal mattiaswal added the ci:main Build default defconfig, not minimal label Oct 30, 2026
A station hostapd drops for inactivity is announced disconnected a
second before it is unbound from its port, so the re-read on the
control event still finds it bound and connected stayed true until
the next station came or went.  The kernel's delete names the access
point, as the station is moved back before removal, so take the
nl80211 station events as a cue for the ports too.

Signed-off-by: Mattias Walström <lazzer@gmail.com>
The radio check in netdag_must_del() never matched, so the interface
stayed on the old phy and hostapd failed to find a frequency for the
new radio's channel.

Signed-off-by: Mattias Walström <lazzer@gmail.com>
@mattiaswal
mattiaswal force-pushed the yangerd-wds-stale-connected branch from 5e7c735 to a09d721 Compare October 30, 2026 13:15
Only band steering and the mesh backhaul touched 5 GHz, and only on a
single channel.  A --band option and a wifi-band topology placeholder let
the same test map onto a radio for either band.

Signed-off-by: Mattias Walström <lazzer@gmail.com>
@troglobit
troglobit merged commit 3a5ee67 into main Oct 30, 2026
8 of 11 checks passed
@troglobit
troglobit deleted the yangerd-wds-stale-connected branch October 30, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:main Build default defconfig, not minimal

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants