Page MenuHomeFreeBSD

net80211: add valid VHT MCS combinations and helper functions
ClosedPublic

Authored by adrian on Jan 23 2025, 2:38 AM.
Referenced Files
F163992663: D48608.id149737.diff
Mon, Jul 27, 5:11 PM
Unknown Object (File)
Sun, Jul 26, 8:52 PM
Unknown Object (File)
Sat, Jul 25, 11:44 PM
Unknown Object (File)
Sat, Jul 25, 5:07 PM
Unknown Object (File)
Sat, Jul 25, 9:00 AM
Unknown Object (File)
Fri, Jul 24, 4:07 AM
Unknown Object (File)
Thu, Jul 23, 3:31 AM
Unknown Object (File)
Wed, Jul 22, 2:37 PM

Details

Summary

Not all NSS / MCS / channel bandwidth combinations are valid.

This will be important for rate control and transmit rate
setup (eg static management, unicast, multicast rates.)

This is based on 802.11-2020 Section 21.5 (Parameters for VHT-MCSs.)

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I am fine with this either way. If you don't want to change this is accepted too.

sys/net80211/ieee80211_phy.c
680

Surprised you don't use a default here for future-proof-ness.

sys/net80211/ieee80211_phy.c
680

I want the compiler to complain, it's an enum, right? :)

bz added inline comments.
sys/net80211/ieee80211_phy.c
680

Do you ever expect more supported bw in VHT? Fine by me either way.

This revision is now accepted and ready to land.Feb 16 2025, 10:15 PM

Not for VHT, but who knows what we'll end up doing for EHT/HE, what we want to do for half/quarter rates, etc.

Right now ni_chw is the protocol width in the IE, not specifically whatever the PHY is configured as. All kinds of fun stuff may happen in the future!