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 _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap
