Page MenuHomeFreeBSD

net80211: update ieee80211_node_get_txrate_mbit() to support VHT
ClosedPublic

Authored by adrian on Jan 23 2025, 2:44 AM.
Referenced Files
Unknown Object (File)
Fri, Mar 7, 3:32 PM
Unknown Object (File)
Thu, Feb 27, 10:50 PM
Unknown Object (File)
Thu, Feb 27, 4:52 PM
Unknown Object (File)
Tue, Feb 25, 10:14 AM
Unknown Object (File)
Mon, Feb 24, 2:09 AM
Unknown Object (File)
Mon, Feb 17, 11:03 PM
Unknown Object (File)
Fri, Feb 14, 7:15 PM
Unknown Object (File)
Thu, Feb 13, 11:35 AM

Details

Summary

Update ieee80211_node_get_txrate_mbit() to use
ieee80211_phy_vht_get_mcs_kbit() to calculate VHT rates.

For now just assume long-GI; I'd have to go and do some extra
work to figure out which short/long GI flag to use.

Diff Detail

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

Event Timeline

This revision is now accepted and ready to land.Jan 28 2025, 10:00 AM
This revision now requires review to proceed.Jan 30 2025, 5:15 AM
bz requested changes to this revision.Sat, Feb 22, 11:11 AM
bz added a subscriber: bz.

The commit message already talks about the wrong function (kbit ot mbit).

sys/net80211/ieee80211_node.c
3300

Why not just add the checks for 80/160 and pass the bool accordingly rather than leaving a comment no one will find again the next 10 years? We don't have a ni_flag for it but it's in the iv/ni_vhtcap and probably not duplicating the information but having an accessor function would be a lot better and less confusing anyway.

This revision now requires changes to proceed.Sat, Feb 22, 11:11 AM
sys/net80211/ieee80211_node.c
3300

heh. fine. :-)

I'm happy to take on a task to clean this up once the diff stack lands. I don't wanna try and insert some more commits into this already big diff stack.

I'll take a TODO to write an analogue to the HT shortgi methods, and also write a HT one that takes a bandwidth as well to clean that API up.

This revision was not accepted when it landed; it landed in state Needs Revision.Wed, Feb 26, 7:32 PM
This revision was automatically updated to reflect the committed changes.