-----Original Message-----
From: Ben Greear <greearb@xxxxxxxxxxxxxxx>
Sent: Friday, March 25, 2022 20:12
To: Luca Coelho <luca@xxxxxxxxx>; kvalo@xxxxxxxxxx
Cc: linux-wireless@xxxxxxxxxxxxxxx
Subject: Re: [PATCH 04/12] iwlwifi: mvm: Passively scan non PSC channels
only when requested so
On 2/4/22 2:25 AM, Luca Coelho wrote:
From: Ilan Peer <ilan.peer@xxxxxxxxx>
Non PSC channels should generally be scanned based on information
about collocated APs obtained during scan on legacy bands, and
otherwise should not be scanned unless specifically requested so (as
there are relatively many non PSC channels, scanning them passively is
time consuming and interferes with regular data traffic).
Thus, modify the scan logic to avoid passively scanning PSC channels
if there is no information about collocated APs and the scan is not a
passive scan.
Hello,
This breaks association against a Cisco test AP on frequency 5995.
Here are logs of the previous commit working (scan takes longer, but SSID is
found), and at the bottom, scan with this commit, which fails to detect the
SSID.
# ethtool -i sta0000
driver: iwlwifi
version: 5.17.0+
firmware-version: 68.01d30b0c.0 ty-a0-gf-a0-68.uc
expansion-rom-version:
bus-info: 0000:05:00.0
supports-statistics: yes
supports-test: no
supports-eeprom-access: no
supports-register-dump: no
supports-priv-flags: no
It breaks on version 71 firmware too.
2022-03-25 09:56:35.464 1.1: sta0000 (phy #0): scan started
2022-03-25 09:56:42.095 1.1: sta0000 (phy #0): scan finished: 2412 2417 2422
2427 2432 2437 2442 2447 2452 2457 2462 2467 2472 5180 5200 5220 5240 5260
5280
5300 5320 5500 5520 5540 5560 5580 5600 5620 5640 5660 5680 5700 5720 5745
5765 5785 5805 5825 5955 5975 5995 6015 6035 6055 6075 6095 6115 6135 6155
6175 6195
6215 6235 6255 6275 6295 6315 6335 6355 6375 6395 6415 6435 6455 6475 6495
6515 6535 6555 6575 6595 6615 6635 6655 6675 6695 6715 6735 6755 6775 6795
6815 6835
6855 6875 6895 6915 6935 6955 6975 6995 7015 7035 7055 7075 7095 7115, ""
2022-03-25 09:56:42.101 1.1: IFNAME=sta0000 <3>SME: Trying to
authenticate with 68:7d:b4:60:04:b8 (SSID=
2022-03-25 09:56:42.122 1.1: sta0000: new station 68:7d:b4:60:04:b8
2022-03-25 09:56:42.252 1.1: sta0000 (phy #0): auth 68:7d:b4:60:04:b8 ->
a4:6b:b6:5a:b1:da status: 126: <unknown>
2022-03-25 09:56:42.254 1.1: IFNAME=sta0000 <3>SME: Trying to
authenticate with 68:7d:b4:60:04:b8 (SSID=
2022-03-25 09:56:42.310 1.1: sta0000 (phy #0): auth 68:7d:b4:60:04:b8 ->
a4:6b:b6:5a:b1:da status: 0: Successful
2022-03-25 09:56:42.310 1.1: IFNAME=sta0000 <3>PMKSA-CACHE-ADDED
68:7d:b4:60:04:b8 0
2022-03-25 09:56:42.311 1.1: IFNAME=sta0000 <3>Trying to associate with
68:7d:b4:60:04:b8 (SSID=
2022-03-25 09:56:42.312 1.1: IFNAME=sta0000 <3>EAPOL-RX
68:7d:b4:60:04:b8 121
2022-03-25 09:56:42.313 1.1: sta0000 (phy #0): assoc 68:7d:b4:60:04:b8 ->
a4:6b:b6:5a:b1:da status: 0: Successful
2022-03-25 09:56:42.313 1.1: IFNAME=sta0000 <3>Associated with
68:7d:b4:60:04:b8
2022-03-25 09:56:42.314 1.1: IFNAME=sta0000 <3>EAPOL-RX
68:7d:b4:60:04:b8 121
2022-03-25 09:56:42.321 1.1: IFNAME=sta0000 <3>CTRL-EVENT-SUBNET-
STATUS-UPDATE status=0
2022-03-25 09:56:42.322 1.1: sta0000 (phy #0): ctrl. port TX status (cookie 1):
acked
2022-03-25 09:56:42.323 1.1: IFNAME=sta0000 <3>EAPOL-RX
68:7d:b4:60:04:b8 195
2022-03-25 09:56:42.328 1.1: sta0000 (phy #0): ctrl. port TX status (cookie 2):
acked
2022-03-25 09:56:42.364 1.1: IFNAME=sta0000 <3>WPA: Key negotiation
completed with 68:7d:b4:60:04:b8 [PTK=CCMP GTK=CCMP]
2022-03-25 09:56:42.403 1.1: IFNAME=sta0000 <3>CTRL-EVENT-CONNECTED
- Connection to 68:7d:b4:60:04:b8 completed [id=0 id_str=]
2022-03-25 09:56:42.403 1.1: IFNAME=sta0000 <3>WPA: Key negotiation
completed with 68:7d:b4:60:04:b8 [PTK=CCMP GTK=CCMP]
2022-03-25 10:05:52.416 1.1: sta0000 (phy #7): scan started
2022-03-25 10:05:56.215 1.1: sta0000 (phy #7): scan finished: 2412 2417 2422
2427 2432 2437 2442 2447 2452 2457 2462 2467 2472 5180 5200 5220 5240 5260
5280
5300 5320 5500 5520 5540 5560 5580 5600 5620 5640 5660 5680 5700 5720 5745
5765 5785 5805 5825, ""
2022-03-25 10:05:56.215 1.1: phy #7: regulatory domain change (phy): set to
US by a driver request on phy7
2022-03-25 10:05:56.216 1.1: IFNAME=sta0000 <3>CTRL-EVENT-NETWORK-
NOT-FOUND
2022-03-25 10:05:56.216 1.1: IFNAME=sta0000 <3>CTRL-EVENT-REGDOM-
CHANGE init=DRIVER type=COUNTRY alpha2=US
2022-03-25 10:06:01.217 1.1: IFNAME=sta0000 <3>CTRL-EVENT-SCAN-
STARTED
2022-03-25 10:06:01.217 1.1: sta0000 (phy #7): scan started
2022-03-25 10:06:02.739 1.1: sta0000 (phy #7): scan finished: 2412 2417 2422
2427 2432 2437 2442 2447 2452 2457 2462 2467 2472 5180 5200 5220 5240 5260
5280
5300 5320 5500 5520 5540 5560 5580 5600 5620 5640 5660 5680 5700 5720 5745
5765 5785 5805 5825 5955 5975 5995 6015 6035 6055 6075 6095 6115 6135 6155
6175 6195
6215 6235 6255 6275 6295 6315 6335 6355 6375 6395 6415 6435 6455 6475 6495
6515 6535 6555 6575 6595 6615 6635 6655 6675 6695 6715 6735 6755 6775 6795
6815 6835
6855 6875 6895 6915 6935 6955 6975 6995 7015 7035 7055 7075 7095 7115, ""
[ SSID is not found, sta never associates]
Thanks,
Ben
Signed-off-by: Ilan Peer <ilan.peer@xxxxxxxxx>
Signed-off-by: Luca Coelho <luciano.coelho@xxxxxxxxx>
---
drivers/net/wireless/intel/iwlwifi/mvm/scan.c | 42 ++++++++++++++-----
1 file changed, 32 insertions(+), 10 deletions(-)
diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/scan.c
b/drivers/net/wireless/intel/iwlwifi/mvm/scan.c
index 8c7cb491330d..901df916baa4 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/scan.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/scan.c
@@ -1728,27 +1728,37 @@ iwl_mvm_umac_scan_fill_6g_chan_list(struct
iwl_mvm *mvm,
}
/* TODO: this function can be merged with
iwl_mvm_scan_umac_fill_ch_p_v6 */ -static void
-iwl_mvm_umac_scan_cfg_channels_v6_6g(struct iwl_mvm_scan_params
*params,
+static u32
+iwl_mvm_umac_scan_cfg_channels_v6_6g(struct iwl_mvm *mvm,
+ struct iwl_mvm_scan_params *params,
u32 n_channels,
struct iwl_scan_probe_params_v4 *pp,
struct iwl_scan_channel_params_v6 *cp,
enum nl80211_iftype vif_type)
{
- struct iwl_scan_channel_cfg_umac *channel_cfg = cp-
channel_config;
int i;
struct cfg80211_scan_6ghz_params *scan_6ghz_params =
params->scan_6ghz_params;
+ u32 ch_cnt;
- for (i = 0; i < params->n_channels; i++) {
+ for (i = 0, ch_cnt = 0; i < params->n_channels; i++) {
struct iwl_scan_channel_cfg_umac *cfg =
- &cp->channel_config[i];
+ &cp->channel_config[ch_cnt];
u32 s_ssid_bitmap = 0, bssid_bitmap = 0, flags = 0;
u8 j, k, s_max = 0, b_max = 0, n_used_bssid_entries;
bool force_passive, found = false, allow_passive = true,
unsolicited_probe_on_chan = false, psc_no_listen = false;
+ /*
+ * Avoid performing passive scan on non PSC channels unless
the
+ * scan is specifically a passive scan, i.e., no SSIDs
+ * configured in the scan command.
+ */
+ if (!cfg80211_channel_is_psc(params->channels[i]) &&
+ !params->n_6ghz_params && params->n_ssids)
+ continue;
+
cfg->v1.channel_num = params->channels[i]->hw_value;
cfg->v2.band = 2;
cfg->v2.iter_count = 1;
@@ -1868,8 +1878,16 @@
iwl_mvm_umac_scan_cfg_channels_v6_6g(struct iwl_mvm_scan_params
*params,
else
flags |= bssid_bitmap | (s_ssid_bitmap << 16);
- channel_cfg[i].flags |= cpu_to_le32(flags);
+ cfg->flags |= cpu_to_le32(flags);
+ ch_cnt++;
}
+
+ if (params->n_channels > ch_cnt)
+ IWL_DEBUG_SCAN(mvm,
+ "6GHz: reducing number channels: (%u->%u)\n",
+ params->n_channels, ch_cnt);
+
+ return ch_cnt;
}
static u8 iwl_mvm_scan_umac_chan_flags_v2(struct iwl_mvm *mvm,
@@
-2415,10 +2433,14 @@ static int
iwl_mvm_scan_umac_v14_and_above(struct
iwl_mvm *mvm,
iwl_mvm_umac_scan_fill_6g_chan_list(mvm, params, pb);
- iwl_mvm_umac_scan_cfg_channels_v6_6g(params,
- params->n_channels,
- pb, cp, vif->type);
- cp->count = params->n_channels;
+ cp->count = iwl_mvm_umac_scan_cfg_channels_v6_6g(mvm,
params,
+ params-
n_channels,
+ pb, cp, vif->type);
+ if (!cp->count) {
+ mvm->scan_uid_status[uid] = 0;
+ return -EINVAL;
+ }
+
if (!params->n_ssids ||
(params->n_ssids == 1 && !params->ssids[0].ssid_len))
cp->flags |=
IWL_SCAN_CHANNEL_FLAG_6G_PSC_NO_FILTER;
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
