Commit f1b1dd51 authored by Johannes Berg's avatar Johannes Berg
Browse files

wifi: iwlwifi: mvm: cycle FW link on chanctx removal 



When the vif is in MLD mode, we'll get a vif links change from
non-zero to zero on disassociation, which removes all links in
the firmware and adds the 'deflink' the driver/mac80211 has.
This causes the firmware to clear some internal state.

However, in non-MLD mode, this doesn't happen, and causes some
state to be left around in firmware, which can particularly
cause trouble with the ref-BSSID in multi-BSSID, leading to an
assert later if immediately making a new multi-BSSID connection
with a different ref-BSSID.

Fix this by removing/re-adding the link in the non-MLD case
when the channel is removed from the vif. This way, all of the
state will get cleared out, even if we need the deflink, which
is more for software architecture purposes than otherwise.

Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Signed-off-by: default avatarGregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20231022173519.90c82837ba4d.I341fa30c480f7673b14b48a0e29a2241472c2e13@changeid


Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 0b67ab5d

drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c

+7 −0
Original line number Diff line number Diff line
@@ -462,10 +462,17 @@ static void iwl_mvm_mld_unassign_vif_chanctx(struct ieee80211_hw *hw, 
					     struct ieee80211_bss_conf *link_conf,

					     struct ieee80211_chanctx_conf *ctx)

{

	struct iwl_mvm_vif *mvmvif = iwl_mvm_vif_from_mac80211(vif);

	struct iwl_mvm *mvm = IWL_MAC80211_GET_MVM(hw);



	mutex_lock(&mvm->mutex);

	__iwl_mvm_mld_unassign_vif_chanctx(mvm, vif, link_conf, ctx, false);

	/* in the non-MLD case, remove/re-add the link to clean up FW state */

	if (!ieee80211_vif_is_mld(vif) && !mvmvif->ap_sta &&

	    !WARN_ON_ONCE(vif->cfg.assoc)) {

		iwl_mvm_remove_link(mvm, vif, link_conf);

		iwl_mvm_add_link(mvm, vif, link_conf);

	}

	mutex_unlock(&mvm->mutex);

}