From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D8B401E2CE0; Tue, 3 Sep 2024 20:44:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725396276; cv=none; b=MUsfeIuC2bZfeHA1osjUToiXFCklQ8qcEh7oQlooizC/LTNewqIgP6s67Re+vJi8md9zP8DNUc4DrPuC7zOzA/9qlFIJ2HwYkpXQdnukOXnZtYrjXQuDfv44onDKk8wZLnF3BqGRjfJ5i8rkb1fYUQBulQJ3hIbp8cbVYtnjlVA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725396276; c=relaxed/simple; bh=3TkSIuxAAT2F8XrEHSa3J56fxg8zv/ehjULsrLAdF5M=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=P4nM1xqgjlEjRFoyUBePv6hKPKLGxwSl/eJMLxSgvIwS1z6uYMFABRjiC8CksDTCyteArgpTAf4ypY4twwz/3r2YKy7cQEf2IXdHtyvNCVMDipE7fi5ClIozhcWGKpAaF99NRbvtqSxkzmBjPZtJBb6y8AeeTx3+/xWlr8Vf6lk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ej6/pPGr; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ej6/pPGr" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5D436C4CEC5; Tue, 3 Sep 2024 20:44:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1725396276; bh=3TkSIuxAAT2F8XrEHSa3J56fxg8zv/ehjULsrLAdF5M=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ej6/pPGr3cjtmtifjlgl/n/WvCTV96PSr8KyeF5OKwak+N+emDyN44KTOUoj2KZmP pNVV3twEZiia/7uI5YRxdoIdtdrE4ozzSl7Wh+Zpqf1cKLYBuZNyUasNM4IRSOcw9b 1TTdsJEgq2s+l620b+6fisnRVwyrGAQ4dN82siMYhb/V7Fx4FDAchEW3KMbtNKwHB5 D51f0xgqU8S2pvq+SfsRuHJUd9/1BQsvq76eqMEITsKwd25DkmaeXP5CgVXKbO07m3 xKUXCBkszzDjeKaGjxP9IM6VO1GwLS/qU5zwlEXtKnzVsJrhT4Et7C3wiu89x+9nrr VQBb/mVMvcC2g== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Emmanuel Grumbach , Miri Korenblit , Johannes Berg , Sasha Levin , kvalo@kernel.org, gregory.greenman@intel.com, ilan.peer@intel.com, shaul.triebitz@intel.com, benjamin.berg@intel.com, linux-wireless@vger.kernel.org Subject: [PATCH AUTOSEL 6.6 18/20] wifi: iwlwifi: mvm: don't wait for tx queues if firmware is dead Date: Tue, 3 Sep 2024 15:23:50 -0400 Message-ID: <20240903192425.1107562-18-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240903192425.1107562-1-sashal@kernel.org> References: <20240903192425.1107562-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 6.6.48 Content-Transfer-Encoding: 8bit From: Emmanuel Grumbach [ Upstream commit 3a84454f5204718ca5b4ad2c1f0bf2031e2403d1 ] There is a WARNING in iwl_trans_wait_tx_queues_empty() (that was recently converted from just a message), that can be hit if we wait for TX queues to become empty after firmware died. Clearly, we can't expect anything from the firmware after it's declared dead. Don't call iwl_trans_wait_tx_queues_empty() in this case. While it could be a good idea to stop the flow earlier, the flush functions do some maintenance work that is not related to the firmware, so keep that part of the code running even when the firmware is not running. Signed-off-by: Emmanuel Grumbach Signed-off-by: Miri Korenblit Link: https://patch.msgid.link/20240825191257.a7cbd794cee9.I44a739fbd4ffcc46b83844dd1c7b2eb0c7b270f6@changeid [edit commit message] Signed-off-by: Johannes Berg Signed-off-by: Sasha Levin --- drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c index f973efbbc3795..d2daea3b1f38a 100644 --- a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c +++ b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c @@ -5589,6 +5589,10 @@ static void iwl_mvm_flush_no_vif(struct iwl_mvm *mvm, u32 queues, bool drop) int i; if (!iwl_mvm_has_new_tx_api(mvm)) { + /* we can't ask the firmware anything if it is dead */ + if (test_bit(IWL_MVM_STATUS_HW_RESTART_REQUESTED, + &mvm->status)) + return; if (drop) { mutex_lock(&mvm->mutex); iwl_mvm_flush_tx_path(mvm, @@ -5673,8 +5677,11 @@ void iwl_mvm_mac_flush(struct ieee80211_hw *hw, struct ieee80211_vif *vif, /* this can take a while, and we may need/want other operations * to succeed while doing this, so do it without the mutex held + * If the firmware is dead, this can't work... */ - if (!drop && !iwl_mvm_has_new_tx_api(mvm)) + if (!drop && !iwl_mvm_has_new_tx_api(mvm) && + !test_bit(IWL_MVM_STATUS_HW_RESTART_REQUESTED, + &mvm->status)) iwl_trans_wait_tx_queues_empty(mvm->trans, msk); } -- 2.43.0