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 Not Applicable
Unit
Tests Not Applicable

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
3322

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
3322

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.