HomeFreeBSD

net80211: improve logging about state transitions lost

Description

net80211: improve logging about state transitions lost

It is possible that we call ieee80211_new_state_locked() again before
a previous task finished to completion (not run yet or unlocked in
between) since 5efea30f039c4 (and follow-up).
In either case we would overwrite the new state and argument in the vap.

While most drivers somehow deal with that (or not), LinuxKPI 802.11 compat
code has KASSERTs to keep net80211, LinuxKPI and driver/firmware state in
sync and they may trigger due to a missing transition or more likely a
changed ni/lsta.

Enhance the wlandebug +state logging for these cases so they
are easier to debug.

While here remove the unconditional logging to the message buffer;
it has been here for a good decade but not helped to actually identify
and sort the problem.

Sponsored by: The FreeBSD Foundation
MFC after: 3 days
Reviewed by: cc
Differential Revision: https://reviews.freebsd.org/D42560

Details

Provenance
bzAuthored on Nov 12 2023, 11:51 PM
Reviewer
cc
Differential Revision
D42560: net80211: improve logging about state transitions lost
Parents
rG45438f9c8b5b: find: Add a warning about -delete.
Branches
Unknown
Tags
Unknown