Page MenuHomeFreeBSD
Differential D26087

net80211: VHT correct NSS Set loop boundary
ClosedPublic
Actions

Authored by bz on Aug 17 2020, 4:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 5, 7:31 AM
Unknown Object (File)
Fri, Nov 28, 1:17 PM
Unknown Object (File)
Sat, Nov 22, 7:40 PM
Unknown Object (File)
Thu, Nov 20, 5:27 PM
Unknown Object (File)
Thu, Nov 20, 5:23 PM
Unknown Object (File)
Thu, Nov 20, 5:23 PM
Unknown Object (File)
Thu, Nov 20, 5:16 PM
Unknown Object (File)
Sun, Nov 16, 11:10 PM
View All Files
Subscribers
gnn
imp
melifaro

Details

Reviewers
adrian
Group Reviewers
wireless
Commits
rS364326: net80211: VHT correct NSS Set loop boundary
Summary

For the <VHT-MCS, NSS> tuple, NSS is 1..8 (or in our loop case 0..7
but not 0..6). Correct the boundry to check for < 8 and not < 7.

MFC After: 2 weeks
Reviewed by:
Sponsored by: Rubicon Communications, LLC (d/b/a "Netgate")

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33030
Build 30416: arc lint + arc unit

Event Timeline

bz created this revision.Aug 17 2020, 4:59 PM
Herald added a subscriber: melifaro. · View Herald TranscriptAug 17 2020, 4:59 PM
bz requested review of this revision.Aug 17 2020, 4:59 PM
Harbormaster completed remote builds in B33030: Diff 75896.Aug 17 2020, 4:59 PM
bz edited the summary of this revision. (Show Details)Aug 17 2020, 5:12 PM
adrian accepted this revision.Aug 17 2020, 5:45 PM

Oh yeah, good catch! We should also replace this with a define so we know how many max mcs streams is supported.

This revision is now accepted and ready to land.Aug 17 2020, 5:45 PM
Closed by commit rS364326: net80211: VHT correct NSS Set loop boundary (authored by bz). · Explain WhyAug 17 2020, 8:16 PM
This revision was automatically updated to reflect the committed changes.
bz added a commit: rS364326: net80211: VHT correct NSS Set loop boundary.
Herald added a subscriber: imp. · View Herald TranscriptAug 17 2020, 8:16 PM

Revision Contents
Changeset List

PathSize
sys/
net80211/
2 lines

Diff 75896

View Options

sys/net80211/ieee80211_vht.c

Loading...