On 9/25/23 18:40, Johannes Berg wrote:
On Mon, 2023-09-25 at 17:28 +0530, Aditya Kumar Singh wrote:
Currently, during channel switch, deflink (or link_id 0) is always
considered. However, with Multi-Link Operation (MLO), there is a
need to handle link specific data structures based on the actual
operational link_id during channel switch operation.
Hence, add support for the same. Non-MLO based operations will use
link_id as 0 or deflink member as applicable.
Much of this commit log really applies to mac80211 - perhaps make a
simple cfg80211 patch first only?
Okay sure, will do. Thanks for the suggestion.
+ link_data = sdata_dereference(sdata->link[link_id], sdata);
We should probably use wiphy_dereference() now?
Hmm yeah, missed this. Sure will use this instead.
Perhaps we should just get rid of sdata_dereference() entirely, after
all, it's the same now, just the arguments are switched for no good
reason.
Yup agreed. Are you already aware of any WIP in this regards? If not, I
can take care for the CSA part at least in same series (in a different
patch obviously :) )
- conf = rcu_dereference_protected(sdata->vif.bss_conf.chanctx_conf,
+ conf = rcu_dereference_protected(link_conf->chanctx_conf,
lockdep_is_held(&local->hw.wiphy->mtx));
Hm, I guess that was an artifact of the (mechanical) conversions - but
really that should probably be wiphy_dereference() as well now?
Yeah. Let me update accordingly and validate. Thanks for your inputs.
