Page MenuHomeFreeBSD

iwx: don't log VHT rate lookups in iwx_tx_fill_cmd() for data frames
ClosedPublic

Authored by adrian on Sep 28 2025, 3:17 AM.
Referenced Files
Unknown Object (File)
Mon, Apr 13, 3:06 PM
Unknown Object (File)
Sat, Apr 11, 1:18 PM
Unknown Object (File)
Sat, Apr 4, 2:21 PM
Unknown Object (File)
Sat, Apr 4, 12:46 PM
Unknown Object (File)
Sat, Apr 4, 3:58 AM
Unknown Object (File)
Sun, Mar 29, 12:37 PM
Unknown Object (File)
Mar 18 2026, 6:07 AM
Unknown Object (File)
Feb 19 2026, 9:52 AM
Subscribers

Details

Summary

iwx is now logging a lot of stuff to the kernel when associated at VHT
rates. It's my fault; the replacement tx rate APIs print out a warning
when you're handing the legacy rate /HT rate APIs a VHT rate.

However all of the supported chipsets in iwx will be pushing the
rate control entirely into firmware. We don't need to do per-frame
TX rate control like in previous chips.

So for now, just put in an if (rate == VHT) into the path and
a TODO comment. A few lines later the function will just skip the
rate assignment stuff for data frames, thus saving us the logging.
The other paths (control traffic, multicast traffic) uses legacy
rates / HT rates by default and won't trigger a warning log.

Yes, iwx_tx_fill_cmd() really does deserve a rate control logic
rewrite, but I want to fix this particular issue first.

Locally tested:

  • AX210, STA mode, HT and VHT associations

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 67371
Build 64254: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Sep 30 2025, 10:10 AM