On Mon, 2023-04-17 at 22:13 +0800, Wen Gong wrote: > On 4/11/2023 3:38 PM, Johannes Berg wrote: > > On Tue, 2023-04-04 at 11:28 +0800, Wen Gong wrote: > > > May I also add a field such as "u16 active_links_count" in struct > > > wiphy_iftype_ext_capab, > > > and add logic in function ieee80211_set_vif_links_bitmaps() for station > > > like this ?: > > > if (active_links_count && hweight16(links) <= active_links_count) > > > then sdata->vif.active_links = links; > > > > > Also here, not sure it makes sense in cfg80211 level? > > > > Though I'm not sure what the idea here is at all - you can refuse to > > link switch etc, what would you use this for? > If I use ieee80211_set_active_links(), > then I need add BSS_CHANGED_ASSOC and key for 2nd link in lower-driver. > > I would like to active all links while assoc, > then BSS_CHANGED_ASSOC and key will auto set for the 2nd link to > lower-driver from mac80211. I'm not convinced that makes sense. You're going to have to be able to deal with changing links after association _anyway_, unless you plan on breaking the entire connection once any of the links is getting out of range or something? So anyway you're going to have to be able to this for new links anyway? I mean doing key management when link switching, and "association" (in quotes, because as a term doesn't even make sense since this state is on the MLD level, not the link level)... So not sure I get it? johannes
