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

[Peer, Ilan]

Hi,

> -----Original Message-----
> From: Jouni Malinen <j at w1.fi>
> Sent: Monday, April 18, 2022 18:22
> To: Peer, Ilan <ilan.peer at intel.com>
> Cc: hostap at lists.infradead.org
> Subject: Re: [PATCH v4 4/4] P2P: Do not use collocated scan logic when
> scanning 6GHz
> 
> On Mon, Apr 11, 2022 at 05:22:01PM +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.
> 
> Is this really suggesting all 6 GHz channels to be scanned passively?
> Wouldn't that take very long time?
> 

Yes. It would indeed take long time. I believe that regardless of this patch set, this
is what is happening today.

> > diff --git a/wpa_supplicant/p2p_supplicant.c
> > b/wpa_supplicant/p2p_supplicant.c @@ -444,8 +444,16 @@ static int
> 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;
> > +	}
> 
> Why would anyone run a P2P group on a non-PSC channel in the 6 GHz band?
> 

It is not disallowed (the entire P2P operation on 6GHz is not fully defined yet).
Would you prefer a patch that instead limits the scanned channels to only include
PSC channels?

Regards,

Ilan.