Thread (18 messages) 18 messages, 3 authors, 2018-08-29

Re: [PATCH v6 1/3] nl80211: Add ATOMIC_KEY_REPLACE API

From: Johannes Berg <johannes@sipsolutions.net>
Date: 2018-08-28 12:37:17

On Sat, 2018-08-18 at 22:53 +0200, Alexander Wetzel wrote:
quoted
This looks good to me from a userspace perspective.  I will try to
implement support for this in iwd soon to give you a prototype to play
with.
Sounds promising, thank you!

I'm still unsure if we really need the API changes to fix that issue:
"Tagging" the new requirements to current set_key calls would also work.
With the downside that there would be no way to detect "broken"
drivers... replace_key is basically only there to differentiate between
audited/fixed drivers and those not.

But since my current impression is, that ptk rekeys are mostly broken
independent of mac80211 or even linux a driver flag signaling support
for it sounds like a good idea regardless how we want to fix the issue
in mac80211. Just wondering if we should name it differently for that
and I'm considering renaming it to NL80211_EXT_FEATURE_CAN_REKEY_PTK0 in
the next patch.
And then keep set_key() for both, rather than adding replace_key()?
Seems reasonable to me, I guess.
As for mac80211 driver status:
The only known "really broken" driver at the moment is ath9k. With
iwlwifi, - and less thorough tested - ath10k to be ok from a driver
point of view. (ath9k needs just a driver flush as minimal fix.)
iwlwifi is also broken for CCMP-256/GCMP keys, so the situation is
slightly more complex.

johannes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help