[12/12] iwlwifi: mvm: set station HE capabilities before authentication

Commit Message

Luca Coelho Nov. 29, 2020, 1:14 p.m. UTC

From: Avraham Stern <[e-mail]>

Some 11ax APs send the authentication response frame as a HE frame.
Since the MAC_FILTER_IN_11AX flag is only set on association, the
frame is being filtered out and the association fails.
Fix it by setting the station HE capabilities before authentication.
This will result in setting the MAC_FILTER_IN_11AX flag for HE
stations before authentication and filtering in the authentication
response.

Signed-off-by: Avraham Stern <[e-mail]>
Signed-off-by: Luca Coelho <[e-mail]>
---
 drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Luca Coelho Dec. 3, 2020, 11:09 a.m. UTC | #1

On Sun, 2020-11-29 at 15:14 +0200, Luca Coelho wrote:
> From: Avraham Stern <[e-mail]>
> 
> Some 11ax APs send the authentication response frame as a HE frame.
> Since the MAC_FILTER_IN_11AX flag is only set on association, the
> frame is being filtered out and the association fails.
> Fix it by setting the station HE capabilities before authentication.
> This will result in setting the MAC_FILTER_IN_11AX flag for HE
> stations before authentication and filtering in the authentication
> response.
> 
> Signed-off-by: Avraham Stern <[e-mail]>
> Signed-off-by: Luca Coelho <[e-mail]>
> ---

I'm going to drop this one, since this implementation already went it
in another patch for fixes.  I apologize for the noise.

--
Cheers,
Luca.

Patch

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
index 688c1125e67b..62e884f3e29e 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
@@ -3127,6 +3127,9 @@  static int iwl_mvm_mac_sta_state(struct ieee80211_hw *hw,
 			goto out_unlock;
 		}
 
+		if (vif->type == NL80211_IFTYPE_STATION)
+			vif->bss_conf.he_support = sta->he_cap.has_he;
+
 		if (sta->tdls &&
 		    (vif->p2p ||
 		     iwl_mvm_tdls_sta_count(mvm, NULL) ==