On 4/6/22 11:39 PM, Peer, Ilan wrote:
Hi,
-----Original Message-----
From: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>
Sent: Wednesday, April 06, 2022 18:47
To: greearb <greearb@xxxxxxxxxxxxxxx>; Peer, Ilan <ilan.peer@xxxxxxxxx>;
hostap@xxxxxxxxxxxxxxxxxxx
Cc: Mussai, Tova <tova.mussai@xxxxxxxxx>; Otcheretianski, Andrei
<andrei.otcheretianski@xxxxxxxxx>; Stern, Avraham
<avraham.stern@xxxxxxxxx>
Subject: Re: [PATCH v2 1/2] nl80211: set
NL80211_SCAN_FLAG_COLOCATED_6GHZ in scan
On Wed, 2022-04-06 at 08:40 -0700, Ben Greear wrote:
On 4/6/22 8:15 AM, Ilan Peer wrote:
From: Tova Mussai <tova.mussai@xxxxxxxxx>
Set NL80211_SCAN_FLAG_COLOCATED_6GHZ in the scan params to
enable
scanning for co-located AP's found in 2.4/5 GHz bands when not
scanning passively. Do so only when collocated scanning is not
disabled by higher layer logic.
If I read the double-negatives correctly, this patch enables the new
flag by default, and at least in this patch, there is no way to
disable that.
Should there be a user-configurable in wpa_supplicant.conf to allow
enabling/disabling this feature?
I do not see a real reason for this other than for testing purposes. The IEEE802.11 specification direction for discovery on non PSC channels is such that these channels should be scanned only if there is some other indication that there are APs operating on these channels.
I am all about testing purposes!
I think it should be allowed to be disabled in case this feature
causes other regressions, but fine to default it to enabled if you
think that is a good idea.
Thanks,
--Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap
