Page MenuHomeFreeBSD
Differential D8364

[net80211] extend/add crypto key flags for more key / payload offload. * Add flags to say "don't mmic/iv insert", etc - ath10k firmware and other boards want it. * Move the demic routine into ieee80211_crypto.c and add support for being told...
ClosedPublic
Actions

Authored by adrian on Oct 28 2016, 2:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 12:10 AM
Unknown Object (File)
Tue, Apr 2, 12:33 AM
Unknown Object (File)
Fri, Mar 29, 9:32 AM
Unknown Object (File)
Mar 15 2024, 4:13 AM
Unknown Object (File)
Dec 27 2023, 5:34 PM
Unknown Object (File)
Dec 27 2023, 5:09 PM
Unknown Object (File)
Dec 20 2023, 4:13 AM
Unknown Object (File)
Dec 14 2023, 4:39 PM
View All 47 Files
Subscribers
avos
imp

Details

Reviewers
avos
Commits
rS308354: [net80211] begin fleshing out new hardware crypto offload features.
Summary

...that the IV/MIC was stripped and so it can't check with the key, etc.

Diff Detail

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

Event Timeline

adrian updated this revision to Diff 21743.Oct 28 2016, 2:08 AM
adrian retitled this revision from to [net80211] extend/add crypto key flags for more key / payload offload. * Add flags to say "don't mmic/iv insert", etc - ath10k firmware and other boards want it. * Move the demic routine into ieee80211_crypto.c and add support for being told....
adrian updated this object.
adrian edited the test plan for this revision. (Show Details)
Herald added a subscriber: imp. · View Herald TranscriptOct 28 2016, 2:08 AM
adrian added a reviewer: avos.Oct 28 2016, 2:08 AM
adrian added a subscriber: avos.
avos added inline comments.Oct 29 2016, 8:32 AM
sys/net80211/ieee80211_crypto.h
81 ↗(On Diff #21743)

just a note - this flag is not set for WPA-NONE / WEP / <other static> keys

93 ↗(On Diff #21743)

I think new flags should be added into IEEE80211_KEY_DEVICE mask - see rS305850

avos added inline comments.Oct 29 2016, 8:34 AM
sys/net80211/ieee80211_crypto.h
93 ↗(On Diff #21743)

IEEE80211_KEY_BITS needs to be updated too

avos added inline comments.Oct 29 2016, 8:40 AM
sys/net80211/ieee80211_crypto.h
81 ↗(On Diff #21743)

looks like there is off-by-one bug it the sent e-mail - I've marked line 81 (KEY_GROUP) in review, not line 82 (KEY_NOREPLAY)

Closed by commit rS308354: [net80211] begin fleshing out new hardware crypto offload features. (authored by adrian). · Explain WhyNov 5 2016, 10:41 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
net80211/
46 lines
55 lines

Diff 22023

View Options

head/sys/net80211/ieee80211_crypto.h

Loading...
View Options

head/sys/net80211/ieee80211_crypto.c

Loading...