[PATCH v3 2/4] scan: Add a configuration option to disable 6GHz collocated scanning

[Ben Greear]

On 4/11/22 6:24 AM, Peer, Ilan wrote:
> Hi,
> 
>> -----Original Message-----
>> From: Ben Greear <greearb at candelatech.com>
>> Sent: Sunday, April 10, 2022 23:26
>> To: Peer, Ilan <ilan.peer at intel.com>; hostap at lists.infradead.org
>> Subject: Re: [PATCH v3 2/4] scan: Add a configuration option to disable 6GHz
>> collocated scanning
>>
>> On 4/10/22 12:51 AM, Ilan Peer wrote:
>>> This configuration option can be used to disable 6GHz collocated scan
>>> logic which would result with passively scanning 6GHz non PSC channels.
>>
>> It would be nice if you could add the example config option to
>> wpa_supplicant.conf with appropriate comments about acceptable values
>> and what it means.
>>
>> Otherwise, a user has to get lucky reading code to even know the option is
>> available.
>>
> 
> Updated the patch set.
>   
>> Also, even if this fixes the ax210 5.18 patch that causes the regression (and I
>> have not yet had time to test this), then you should probably revert that
>> ax210 patch so that it works with older supplicants?  It often takes a long time
>> to get supplicant updated in standard OS distributions.
>>
> 
> I'll consider it. The main problem reverting this logic is that scans might take very
> long time (up to 30 seconds in some cases)  do to passively scanning all the non PSC
> channels.

It takes way longer than that when it doesn't work at all.

Maybe toggle between the two modes if user space does not provide guidance (try fast way first, then slow if
fast didn't find anything), and have the supplicant send down explicit request for each behaviour so that in the
driver you know if supplicant is up to date or not?

Thanks,
Ben

> 
> Regards,
> 
> Ilan.
> 


-- 
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc  http://www.candelatech.com