Re: [PATCH v3 0/3] dbus: expose per-BSS RSN capabilities

Hi Aleks,

On 7/14/26 3:48 PM, Aleksman4o wrote:
> From: Aleks Man <[email protected]>
>
> James, thanks for confirming that extending BasicServiceSet is a
> reasonable approach.
>
> Network.Type intentionally represents WPA2-Personal, WPA3-Personal, and
> transition networks as "psk".  This series exposes the RSN information
> that iwd already retains for each BSS so consumers can distinguish those
> networks without changing the existing Network API.
>
> The optional BasicServiceSet.RSN dictionary uses the same KeyMgmt,
> Pairwise, and Group fields and values as wpa_supplicant's BSS.RSN
> property.  NetworkManager can therefore reuse its existing parser rather
> than add another translation layer.  Older consumers ignore the optional
> property, and consumers can retain their Network.Type fallback for older
> iwd versions.
>
> A corresponding NetworkManager series reads and combines the per-BSS
> capabilities for its synthetic ESS access point.  It will be submitted
> separately after this API series.
>
> Validation:
>
>    - iwd builds successfully with make -j4.
>    - On a physical WPA2/WPA3 transition network, an initial PSK connection
>      was reported as WPA2.  After changing the saved profile to SAE, a
>      disconnect/reconnect through the stock Plasma applet completed without
>      a password prompt and was reported as WPA3-SAE.
>    - Cycling Wi-Fi subsequently auto-connected the SAE profile and kept it
>      visible as connected in the applet.
>    - The upstream CI auto-t runner passed the v2 series.
>
> Changes in v3:
>    - Shorten the auto-t filename so the generated patch metadata complies
>      with GitLint's 80-column limit.  There are no code changes.
>
> Changes in v2:
>    - Resend with git-send-email after the Gmail web client corrupted the
>      inline v1 patches.  There are no code changes.
>
> RFC:
> https://lore.kernel.org/iwd/CAMq8zouhDMRW4Q7qSpMCFftp21wCzuC%[email protected]/
>
> Related NetworkManager issues:
> https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/issues/1489
> https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/issues/1693
>
> Aleks Man (3):
>    network: expose RSN capabilities on BSS
>    doc: document BasicServiceSet RSN property
>    auto-t: test BasicServiceSet RSN property
>
>   .../testBasicServiceSetRSN/bss_rsn_test.py    |  70 +++++++++++
>   autotests/testBasicServiceSetRSN/hw.conf      |   9 ++
>   autotests/testBasicServiceSetRSN/ssidSAE.conf |  11 ++
>   .../ssidTransition.conf                       |  12 ++
>   .../testBasicServiceSetRSN/ssidWPA2.conf      |   8 ++
>   doc/basic-service-set.txt                     |  23 ++++
>   src/network.c                                 | 110 ++++++++++++++++++
>   7 files changed, 243 insertions(+)
>   create mode 100644 autotests/testBasicServiceSetRSN/bss_rsn_test.py
>   create mode 100644 autotests/testBasicServiceSetRSN/hw.conf
>   create mode 100644 autotests/testBasicServiceSetRSN/ssidSAE.conf
>   create mode 100644 autotests/testBasicServiceSetRSN/ssidTransition.conf
>   create mode 100644 autotests/testBasicServiceSetRSN/ssidWPA2.conf
>
Overall this looks good. It would be nice to have some support in iwctl 
to see these fields. Those properties and getters/updaters can be 
implemented here if possible:

https://git.kernel.org/pub/scm/network/wireless/iwd.git/tree/client/bss.c

We will need to wait for Denis to comment/merge though.

Thanks,

James