diff mbox series

cfg80211: Add feature flag to indicate RSNXE IE support in FT

Message ID 1596511970-8415-1-git-send-email-vjakkam@codeaurora.org (mailing list archive)
State Changes Requested
Delegated to: Johannes Berg
Headers show
Series cfg80211: Add feature flag to indicate RSNXE IE support in FT | expand

Commit Message

Veerendranath Jakkam Aug. 4, 2020, 3:32 a.m. UTC
Add a new feature flag that drivers in which FT handshake is offloaded,
to advertise support for interoperability with legacy clients which
doesn't support using RSNXE IE for MIC calculation in FT handshake. The
userspace can choose to enable/disable FT protocols that require RSNXE IE
to be used based on this flag advertised by drivers.

Signed-off-by: Veerendranath Jakkam <vjakkam@codeaurora.org>

Comments

Johannes Berg Aug. 27, 2020, 9:33 a.m. UTC | #1
On Tue, 2020-08-04 at 09:02 +0530, Veerendranath Jakkam wrote:
> Add a new feature flag that drivers in which FT handshake is offloaded,

"that drivers (in which ...)" what? Can set? Are expected to use?

> to advertise support for interoperability with legacy clients which
> doesn't

don't? or what does "doesn't" refer to?

>  support using RSNXE IE for MIC calculation in FT handshake. The
> userspace can choose to enable/disable FT protocols that require RSNXE IE
> to be used based on this flag advertised by drivers.

I can't say I really understand this.

> Signed-off-by: Veerendranath Jakkam <vjakkam@codeaurora.org>
> 
> diff --git a/include/uapi/linux/nl80211.h b/include/uapi/linux/nl80211.h
> index 631f3a99..83e8079 100644
> --- a/include/uapi/linux/nl80211.h
> +++ b/include/uapi/linux/nl80211.h
> @@ -5820,6 +5820,9 @@ enum nl80211_feature_flags {
>   * @NL80211_EXT_FEATURE_4WAY_HANDSHAKE_AP_PSK: Device wants to do 4-way
>   *	handshake with PSK in AP mode (PSK is passed as part of the start AP
>   *	command).
> + * @NL80211_EXT_FEATURE_FT_RSNXE: Driver supports RSNXE element in FT handshake
> + *	and interoperable with legacy clients that doesn't support using RSNXE
> + *	IE for MIC computation in FT handshake.

Nor this. Please rewrite the commit message and comment to be more
explanatory (and grammatically correct).

johannes
diff mbox series

Patch

diff --git a/include/uapi/linux/nl80211.h b/include/uapi/linux/nl80211.h
index 631f3a99..83e8079 100644
--- a/include/uapi/linux/nl80211.h
+++ b/include/uapi/linux/nl80211.h
@@ -5820,6 +5820,9 @@  enum nl80211_feature_flags {
  * @NL80211_EXT_FEATURE_4WAY_HANDSHAKE_AP_PSK: Device wants to do 4-way
  *	handshake with PSK in AP mode (PSK is passed as part of the start AP
  *	command).
+ * @NL80211_EXT_FEATURE_FT_RSNXE: Driver supports RSNXE element in FT handshake
+ *	and interoperable with legacy clients that doesn't support using RSNXE
+ *	IE for MIC computation in FT handshake.
  *
  * @NUM_NL80211_EXT_FEATURES: number of extended features.
  * @MAX_NL80211_EXT_FEATURES: highest extended feature index.
@@ -5878,6 +5881,7 @@  enum nl80211_ext_feature_index {
 	NL80211_EXT_FEATURE_CONTROL_PORT_OVER_NL80211_TX_STATUS,
 	NL80211_EXT_FEATURE_OPERATING_CHANNEL_VALIDATION,
 	NL80211_EXT_FEATURE_4WAY_HANDSHAKE_AP_PSK,
+	NL80211_EXT_FEATURE_FT_RSNXE,
 
 	/* add new features before the definition below */
 	NUM_NL80211_EXT_FEATURES,