Page MenuHomeFreeBSD

net80211 / ifconfig: cleanup the use of IEEE80211_FVHT_USEVHT*
ClosedPublic

Authored by bz on Aug 7 2020, 5:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 8:32 PM
Unknown Object (File)
Fri, Mar 22, 9:26 PM
Unknown Object (File)
Feb 2 2024, 7:54 AM
Unknown Object (File)
Jan 4 2024, 8:24 PM
Unknown Object (File)
Jan 4 2024, 8:24 PM
Unknown Object (File)
Jan 4 2024, 8:24 PM
Unknown Object (File)
Jan 4 2024, 8:12 PM
Unknown Object (File)
Jan 3 2024, 8:27 AM

Details

Summary

Rather then using magic numbers duplicate IEEE80211_FVHT_VHT* in
ifconfig (cleanup of these and other flags used and not exposed by
net80211 should happen later) and use those.
In the kernel this simplifies one ioctl path (the other one currently
relies on individual bit flags being passed in).
We also re-order the 80P80 and 160 flag for 160 to come before 80+80
and more clearly leave the flags as TODO in one of the 160/80+80 cases.

Sponsored by: Rubicon Communications, LLC (d/b/a "Netgate")

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bz requested review of this revision.Aug 7 2020, 5:43 PM

Anyone? I am planning to commit this next week.

I'm happy with this change although I'd remove the code in #if 0 instead of leaving it there.

sys/net80211/ieee80211_vht.c
164 ↗(On Diff #75584)

Any reason to not just remove this code instead of the ifdef 0 ?

adrian added a subscriber: adrian.
adrian added inline comments.
sys/net80211/ieee80211_vht.c
164 ↗(On Diff #75584)

when i did the bring up i left a bunch of these TODO FIXMEs around where I found things that needed to be taught about VHT flags.

I think it's a good idea to finish this off and implement the vhtcaps check to make sure 40, 80, 80+80 and 160 are used. It shouldn't be /that/ hard.

This revision is now accepted and ready to land.Aug 16 2020, 1:16 AM