Page MenuHomeFreeBSD

net80211: add valid VHT MCS combinations and helper functions
ClosedPublic

Authored by adrian on Jan 23 2025, 2:38 AM.
Referenced Files
F133056184: D48608.id151524.diff
Wed, Oct 22, 12:54 PM
Unknown Object (File)
Mon, Oct 20, 3:32 AM
Unknown Object (File)
Sun, Oct 19, 4:05 AM
Unknown Object (File)
Sat, Oct 18, 4:14 PM
Unknown Object (File)
Fri, Oct 17, 2:41 AM
Unknown Object (File)
Tue, Oct 14, 4:50 AM
Unknown Object (File)
Fri, Oct 10, 5:25 AM
Unknown Object (File)
Thu, Oct 9, 10:16 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 Skipped
Unit
Tests Skipped
Build Status
Buildable 62052
Build 58936: arc lint + arc unit

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!