From: Ben Greear <greearb@candelatech.com>
To: Luca Coelho <luca@coelho.fi>, kvalo@kernel.org
Cc: linux-wireless@vger.kernel.org
Subject: Re: [PATCH 04/12] iwlwifi: mvm: Passively scan non PSC channels only when requested so
Date: Fri, 25 Mar 2022 10:11:31 -0700 [thread overview]
Message-ID: <f18f11f2-b9b9-9d86-340e-d567247ef7bc@candelatech.com> (raw)
In-Reply-To: <iwlwifi.20220204122220.457da4cc95eb.Ic98472bab5f5475f1e102547644caaae89ce4c4a@changeid>
On 2/4/22 2:25 AM, Luca Coelho wrote:
> From: Ilan Peer <ilan.peer@intel.com>
>
> 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@intel.com>
> Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
> ---
> 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@candelatech.com>
Candela Technologies Inc http://www.candelatech.com
next prev parent reply other threads:[~2022-03-25 17:11 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-04 10:24 [PATCH 00/12] iwlwifi: updates intended for v5.18 2022-02-04 Luca Coelho
2022-02-04 10:25 ` [PATCH 01/12] iwlwifi: mvm: Correctly set fragmented EBS Luca Coelho
2022-02-04 10:25 ` [PATCH 02/12] iwlwifi: yoyo: Avoid using dram data if allocation failed Luca Coelho
2022-02-04 10:25 ` [PATCH 03/12] iwlwifi: scan: Modify return value of a function Luca Coelho
2022-02-04 10:25 ` [PATCH 04/12] iwlwifi: mvm: Passively scan non PSC channels only when requested so Luca Coelho
2022-03-25 17:11 ` Ben Greear [this message]
2022-04-04 19:02 ` Peer, Ilan
2022-04-05 19:05 ` Ben Greear
2022-04-06 6:22 ` Peer, Ilan
2022-02-04 10:25 ` [PATCH 05/12] iwlwifi: pcie: adjust to Bz completion descriptor Luca Coelho
2022-02-04 10:25 ` [PATCH 06/12] iwlwifi: pcie: Adapt rx queue write pointer for Bz family Luca Coelho
2022-02-04 10:25 ` [PATCH 07/12] iwlwifi: drv: load tlv debug data earlier Luca Coelho
2022-02-04 10:25 ` [PATCH 08/12] iwlwifi: eeprom: clean up macros Luca Coelho
2022-02-04 10:25 ` [PATCH 09/12] iwlwifi: remove unused macros Luca Coelho
2022-02-04 10:25 ` [PATCH 10/12] iwlwifi: debugfs: remove useless double condition Luca Coelho
2022-02-04 10:25 ` [PATCH 11/12] iwlwifi: mei: use C99 initializer for device IDs Luca Coelho
2022-02-04 10:25 ` [PATCH 12/12] iwlwifi: mvm: Unify the scan iteration functions Luca Coelho
Reply instructions:
You may reply publicly 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=f18f11f2-b9b9-9d86-340e-d567247ef7bc@candelatech.com \
--to=greearb@candelatech.com \
--cc=kvalo@kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=luca@coelho.fi \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.