[PATCH v5 4/4] P2P: Do not use collocated scan logic when scanning 6GHz

[Jouni Malinen]

On Sun, May 08, 2022 at 08:03:54AM +0000, Peer, Ilan wrote:
> > On Sun, Apr 24, 2022 at 12:57:55PM +0300, Ilan Peer wrote:
> > > Passively scan 6GHz channels during P2P find full scan as P2P GOs are
> > > not expected to be published in RNR or other APs on the 2GHz and 5GHz
> > > bands.

> > > diff --git a/wpa_supplicant/p2p_supplicant.c
> > wpas_p2p_scan(void *ctx, enum p2p_scan_type type, int freq,
> > > -	if (!wpa_s->conf->p2p_6ghz_disable)
> > > +	if (!wpa_s->conf->p2p_6ghz_disable) {
> > >  		params->p2p_include_6ghz = include_6ghz;
> > > +
> > > +		/*
> > > +		 * passively scan non PSC channels without relying on
> > collocated
> > > +		 * logic
> > > +		 */
> > > +		params->non_coloc_6ghz = include_6ghz;
> > > +	}
> > 
> > This sounds very incorrect. Why would a P2P device scan for non-PSC
> > channels on the 6 GHz band? The P2P GO would be a 6 GHz-only AP and as
> > such, such set the primary 20 MHz to coincide with a PSC based on IEEE Std
> > 802.11ax-2021 expectations.

> This handles the case that non-PSC channels are included in the scan request. If they are not included
> this setting does not have any effect. Note that currently if the 'p2p_include_6ghz' is set in the scan
> parameters and no frequencies are configured in the scan parameters then all channels would be scanned,
> which would include the non-PSC channels in case the drivers supports them.

Ah.. It would be good to update that comment in wpas_p2p_scan() to make
this part clear.. The way it is currently worded seems to imply that
non-PSC channels are scanned and that is not what I'd like to see.. It
would be more helpful to note that the neighbor report information would
not be available for co-located APs and as such, that mechanism to
select channels cannot be used here.

> As noted previously, we can change this behavior to only include PSC channels if 'p2p_include_6ghz' channels
> Is set. Would you like such a change?

Yes, it sounds better to explicitly limit the P2P 6 GHz cases to scan
only PSC channels.

-- 
Jouni Malinen                                            PGP id EFC895FA