Re: [PATCH v2 1/8] doc: Document station Affinities property

[Denis Kenzior <denkenz@gmail.com>]

Hi James,

On 8/19/24 12:51 PM, James Prestwood wrote:
> This documents new DBus property that expose a bit more control to
> how IWD roams.
> 
> Setting the affinity on the connected BSS effectively "locks" IWD to
> that BSS (except at critical RSSI levels, explained below). This can
> be useful for clients that have access to more information about the
> environment than IWD. For example, if a client is stationary there
> is likely no point in trying to roam until it has moved elsewhere.
> 
> A new main.conf option would also be added:
> 
> [General].CriticalRoamThreshold
> 
> This would be the new roam threshold set if the currently connected
> BSS is in the Affinities list. If the RSSI continues to drop below
> this level IWD will still attempt to roam.
> ---
>   doc/station-api.txt | 17 +++++++++++++++++
>   src/iwd.config.rst  | 16 ++++++++++++++++
>   2 files changed, 33 insertions(+)

Couple of questions:

> +			Setting the affinity will lower the roaming threshold,
> +			effectively locking IWD to the current BSS unless the
> +			RSSI drops below the critical threshold set by
> +			[General].CriticalRoamThreshold{_5GHz} at which point

Should '{_5GHz}' be without the '_'?

<snip>

> +
> +   * - CriticalRoamThreshold5G

What about 6G?

> +     - Value: rssi dBm value, from -100 to 1, default: **-82**
> +
> +       This has the same effect as ``CriticalRoamThreshold``, but for the 5GHz
> +       band.
> +
>      * - RoamRetryInterval
>        - Value: unsigned int value in seconds (default: **60**)
>   

Regards,
-Denis