On Mon, 2024-08-05 at 21:23 +0200, Felix Fietkau wrote: > This series adds support for restricting vifs to a set of radios. > The allowed radios mask is used to restrict scanning, off-channel activity. > This is also used for per-radio start/stop ops, monitor state and filter > flags. It can also limit per-monitor-vif rx to specific radios. > I see the idea, I guess, but but honestly I don't like the way this is all mixed up between cfg80211 and mac80211. I think more of this should be in cfg80211 (e.g. not sure why the changes to ieee80211_change_iface, nothing there that cfg80211 couldn't do; and generally would prefer to have the patch split), but since it can never be perfect there (internal scans like in mac80211, etc.), I guess it really also should come with a feature flag. That'd also make the series more robust, with enabling the feature only in the last commit. johannes
