wifi: iwlwifi: mvm: don't request statistics in restart
authorMiri Korenblit <miriam.rachel.korenblit@intel.com>
Mon, 6 May 2024 07:04:18 +0000 (10:04 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 6 May 2024 14:33:26 +0000 (16:33 +0200)
During restart mac80211 notifies the driver about the association,
(if we was associated before the restart) which causes the driver to
request statistics from the FW. This causes to an immediate exit from
EMLSR after the restart is done, when the statistics notif is handled.
(too low TPT). There is no point in requesting statistics wnyway, since
the FW just started and don't have any.

Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Reviewed-by: Johannes Berg <johannes.berg@intel.com>
Link: https://msgid.link/20240506095953.16638dec9f7b.I093514312179bae566ad8d73ffb0355c6eee288a@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/intel/iwlwifi/mvm/utils.c

index 74452b2112b0da37680a5aa6194b459af5dd0259..47283a358ffddc45fd065f412ee94bd163e921a7 100644 (file)
@@ -435,6 +435,13 @@ int iwl_mvm_request_statistics(struct iwl_mvm *mvm, bool clear)
                                           IWL_FW_CMD_VER_UNKNOWN);
        int ret;
 
+       /*
+        * Don't request statistics during restart, they'll not have any useful
+        * information right after restart, nor is clearing needed
+        */
+       if (test_bit(IWL_MVM_STATUS_IN_HW_RESTART, &mvm->status))
+               return 0;
+
        if (cmd_ver != IWL_FW_CMD_VER_UNKNOWN)
                return iwl_mvm_request_system_statistics(mvm, clear, cmd_ver);