[PATCH] nl80211: set NL80211_SCAN_FLAG_COLOCATED_6GHZ in scan

[Jouni Malinen]

On Tue, Nov 30, 2021 at 05:39:43PM +0200, Andrei Otcheretianski wrote:
> 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.

> diff --git a/src/drivers/driver_nl80211_scan.c b/src/drivers/driver_nl80211_scan.c
> @@ -300,6 +300,9 @@ nl80211_scan_common(struct i802_bss *bss, u8 cmd,
> +	/* scan for 6ghz colocated APs that reported by other APs */
> +	scan_flags |= NL80211_SCAN_FLAG_COLOCATED_6GHZ;
> +
>  	if (scan_flags &&
>  	    nla_put_u32(msg, NL80211_ATTR_SCAN_FLAGS, scan_flags))

Is it really correct to do this unconditionally? Wouldn't this result in
undesired extra latency in scanning when the 6 GHz band is not used
(e.g., for most P2P use cases) or when there is an explicit
configuration (e.g., SETBAND) to disable use of the 6 GHz band?
 
-- 
Jouni Malinen                                            PGP id EFC895FA