Linux-Wireless Archive on lore.kernel.org
 help / color / Atom feed
From: Alexander Wetzel <alexander@wetzel-home.de>
To: Johannes Berg <johannes@sipsolutions.net>, luciano.coelho@intel.com
Cc: linux-wireless@vger.kernel.org, linuxwifi@intel.com
Subject: Re: [PATCH] iwlwifi: Extended Key ID support for mvm and dvm
Date: Mon, 19 Aug 2019 21:57:06 +0200
Message-ID: <da471544-3370-8ba1-2265-d02ab09cdcee@wetzel-home.de> (raw)
In-Reply-To: <204c346ab9fc71865e4cb5f5c29ec33ca05050e2.camel@sipsolutions.net>

>> +
>> +	/* The new Tx API does not allow to pass the key or keyid of a MPDU to
>> +	 * the hw, preventing us to control which key(id) to use per MPDU.
>> +	 * Till that's fixed we can't use Extended Key ID for the newer cards.
> 
> Technically we still don't need per MPDU, we just need to switch which
> one to use for TX after installing it for RX already.

The Extended Key ID API we finally merged in mac80211 is not notifying 
the driver when to switch the key over to the other id.
The current API provides the key/keyid per MPDU and let's mac80211 have 
the full control what's the correct key for each frame.

That's especially critical for drivers setting 
IEEE80211_KEY_FLAG_GENERATE_IV and/or supporting A-MPDU's. Allowing the 
driver to override the mac80211 decision is only safe when the 
driver/card generates the PNs itself and also handles the A-MPDU key 
borders correctly.

While less desirable we still could get that working: The mvm driver 
would have to detect the key borders and then tell the firmware to 
switch over to the other key. But we would have to make sure to not 
re-enable A-MPDU aggregation till the card really has switched.

Alexander


  reply index

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-19 18:05 Alexander Wetzel
2019-08-19 18:51 ` Johannes Berg
2019-08-19 19:57   ` Alexander Wetzel [this message]
2019-08-19 20:03     ` Johannes Berg
2019-08-19 20:09       ` Johannes Berg
2019-08-19 20:50         ` Alexander Wetzel
2019-08-19 20:42       ` Alexander Wetzel
2019-09-06 20:41 ` Alexander Wetzel

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=da471544-3370-8ba1-2265-d02ab09cdcee@wetzel-home.de \
    --to=alexander@wetzel-home.de \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linuxwifi@intel.com \
    --cc=luciano.coelho@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-Wireless Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-wireless/0 linux-wireless/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-wireless linux-wireless/ https://lore.kernel.org/linux-wireless \
		linux-wireless@vger.kernel.org linux-wireless@archiver.kernel.org
	public-inbox-index linux-wireless

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-wireless


AGPL code for this site: git clone https://public-inbox.org/ public-inbox