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

[James Prestwood <prestwoj@gmail.com>]

Hi Denis,

On 8/23/24 9:03 AM, Denis Kenzior wrote:
> 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 '_'?

Yep, that was a typo

>
> <snip>
>
>> +
>> +   * - CriticalRoamThreshold5G
>
> What about 6G?
We could add one for 6G as well, I was just maintaining what we had 
today with respect to the existing thresholds. I can add 6GHz thresholds 
across the board in a later patch set.
>
>
>> +     - 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