[PATCH 0/4] hostapd: add support for AP+STA on the same radio

[Jouni Malinen]

On Wed, Mar 29, 2023 at 03:44:51PM +0200, Raphaël Mélotte wrote:
> This series contains patches from the OpenWrt project which add
> support for running an AP and an STA on the same radio and still be
> able to switch channels. A test for it has also been added.

There was a comment on the second patch proposing a different approach
for handling this, but that did not seem to go much further than that
at least as far as anything on the hostap mailing list is concerned.

Has there been any additional consideration on whether this coordination
could happen within the driver itself or is this type of changes to
hostapd and wpa_supplicant still needed and in use with OpenWrt?


> Arnout Vandecappelle (Essensium/Mind) (1):
>   tests: AP+STA on same radio
> 
> Felix Fietkau (3):
>   hostapd: ctrl_iface: add UPDATE command
>   Add support for AP+STA on a single radio
>   hostapd_reload_config: allow update of mode and frequency

Why does the UPDATE command (from patch 1/4) reload the hostapd
configuration file? Patch 2/4 seems to use UPDATE, but it does not
change anything in the configuration file. Is the UPDATE command really
needed when there is already SET command that can be used to update
running configuration dynamically?

Why does patch 1/4 modify hostapd_ctrl_iface_stop_ap()? The commit
message does not seem to say anything about that change and I'm not sure
the change is really correct. STOP_AP could be issued separately for
each BSS.

Patch 3/4 would change functionality that has been there for almost ten
years.. This patch series does not seem to indicate what the use case is
for configuration file changes since the only use here does not actually
change the configuration file at all. The channel parameters for an
operating AP may have changed dynamically during operation and it feels
undesired to override those with the values that were specified in the
configuration file in many cases. I can understand that there are cases
where the channel should be switched, but the cleaner way for doing that
would be to use the channel switch command and/or SET to override those
parameters explicitly.

-- 
Jouni Malinen                                            PGP id EFC895FA