| Message ID | 20231011072950.4097927-1-make_ruc2021@163.com (mailing list archive) |
|---|---|
| State | Rejected |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | wifi: ath12k: call ath12k_mac_fils_discovery() without condition | expand |
Ma Ke <make_ruc2021@163.com> writes: > Mac80211 does not set flags BSS_CHANGED_FILS_DISCOVERY and > BSS_CHANGED_UNSOL_BCAST_PROBE_RESP if there are no updates to > FILS discovery and unsolicited broadcast probe response transmission > configurations respectively. This results in the transmissions getting > stopped during BSS change operations which do not include these > attributes. Remove the checks for the flags and always send the existing > configuration to firmware. > > Signed-off-by: Ma Ke <make_ruc2021@163.com> This looks almost identical to Aloka's patch: wifi: ath12k: call ath12k_mac_fils_discovery() without condition https://git.kernel.org/netdev/net-next/c/43a10990404f What's happening here?
diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c index 88346e66bb75..94967f810093 100644 --- a/drivers/net/wireless/ath/ath12k/mac.c +++ b/drivers/net/wireless/ath/ath12k/mac.c @@ -2761,9 +2761,7 @@ static void ath12k_mac_op_bss_info_changed(struct ieee80211_hw *hw, } } - if (changed & BSS_CHANGED_FILS_DISCOVERY || - changed & BSS_CHANGED_UNSOL_BCAST_PROBE_RESP) - ath12k_mac_fils_discovery(arvif, info); + ath12k_mac_fils_discovery(arvif, info); if (changed & BSS_CHANGED_EHT_PUNCTURING) arvif->punct_bitmap = info->eht_puncturing;
Mac80211 does not set flags BSS_CHANGED_FILS_DISCOVERY and BSS_CHANGED_UNSOL_BCAST_PROBE_RESP if there are no updates to FILS discovery and unsolicited broadcast probe response transmission configurations respectively. This results in the transmissions getting stopped during BSS change operations which do not include these attributes. Remove the checks for the flags and always send the existing configuration to firmware. Signed-off-by: Ma Ke <make_ruc2021@163.com> --- drivers/net/wireless/ath/ath12k/mac.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)