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 93797277C9A for ; Mon, 20 Oct 2025 16:17:43 +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=1760977063; cv=none; b=Iiy1af9iv8gGES1uVwJNvrk1gdGs6OCcmdkZYjwjwN29pqhYygmqIqA9RPpuUouwj9LCttzSkF4OLuVUE+/k1xUFP3hHxhvqmmQul5T5RELRgprdu43+qxFTg1v3cb4u7I4G6Gi43jgcuXluXYZuUTIZF65IEkpHzt9KZ8e4YAk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760977063; c=relaxed/simple; bh=qSql13iqTt41yx+7E3fHHKOVuY02uvRxg6yhs8DZG08=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=QVz82Hcud4v+yVvd6I+RbDAm8sEBgG9P6Y87DAhhGbHbdfJqueUi22hV4VAIYFWGfRSRTcDsxHXKZTcGL2jYoUr0NOp/mLw9Y68dztOgpyYyBQ2We67nqMqGT37hG5Pi2n/BGZ4lOO/2auPk8X9b01NJCdNf9nHyxIPd7ATr97g= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=fVFVO5P8; 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="fVFVO5P8" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7F79BC4CEF9; Mon, 20 Oct 2025 16:17:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1760977063; bh=qSql13iqTt41yx+7E3fHHKOVuY02uvRxg6yhs8DZG08=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fVFVO5P8pjKlZ4JzrUPhlOnjbX8QEulnlSk1xmzI+My0BH76TaKW2e8ug8NG7LGHE 8xVIcY1eWCY8LLSWbq+w5jyVLKi4lgNmTsET8Hc1NcX1SAyQVo0BlrZRez/ida3Vpx Vqw0EhaUddJJ7MwKEeAUbR53RZJO0EWYidqhaooPMZ+mQOKmAx9PKaM6Xo8xgg4KPI bbBsvC5P6g0Uj0BTvFlHNEove0IkS/06fo+lvVUtnhJ+Fwp+YYh0mJyk4uXy9O+9jz btBkVaQf0ryS2WtImBZvlhFhRq09RTEda9/lRHDaB8Ty7fu4G4PMyUYVt14bZggWkM jUMKv6Ped0HYw== From: Sasha Levin To: stable@vger.kernel.org Cc: Muhammad Usama Anjum , Baochen Qiang , Jeff Johnson , Sasha Levin Subject: [PATCH 5.15.y] wifi: ath11k: HAL SRNG: don't deinitialize and re-initialize again Date: Mon, 20 Oct 2025 12:17:40 -0400 Message-ID: <20251020161740.1835172-1-sashal@kernel.org> X-Mailer: git-send-email 2.51.0 In-Reply-To: <2025101625-phonebook-salt-0964@gregkh> References: <2025101625-phonebook-salt-0964@gregkh> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Muhammad Usama Anjum [ Upstream commit 32be3ca4cf78b309dfe7ba52fe2d7cc3c23c5634 ] Don't deinitialize and reinitialize the HAL helpers. The dma memory is deallocated and there is high possibility that we'll not be able to get the same memory allocated from dma when there is high memory pressure. Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-03926.13-QCAHSPSWPL_V2_SILICONZ_CE-2.52297.6 Fixes: d5c65159f289 ("ath11k: driver for Qualcomm IEEE 802.11ax devices") Cc: stable@vger.kernel.org Cc: Baochen Qiang Reviewed-by: Baochen Qiang Signed-off-by: Muhammad Usama Anjum Link: https://patch.msgid.link/20250722053121.1145001-1-usama.anjum@collabora.com Signed-off-by: Jeff Johnson Signed-off-by: Sasha Levin --- drivers/net/wireless/ath/ath11k/core.c | 6 +----- drivers/net/wireless/ath/ath11k/hal.c | 16 ++++++++++++++++ drivers/net/wireless/ath/ath11k/hal.h | 1 + 3 files changed, 18 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/ath/ath11k/core.c b/drivers/net/wireless/ath/ath11k/core.c index e86ecdf433de5..01e2528fc9ca5 100644 --- a/drivers/net/wireless/ath/ath11k/core.c +++ b/drivers/net/wireless/ath/ath11k/core.c @@ -942,14 +942,10 @@ static int ath11k_core_reconfigure_on_crash(struct ath11k_base *ab) mutex_unlock(&ab->core_lock); ath11k_dp_free(ab); - ath11k_hal_srng_deinit(ab); + ath11k_hal_srng_clear(ab); ab->free_vdev_map = (1LL << (ab->num_radios * TARGET_NUM_VDEVS)) - 1; - ret = ath11k_hal_srng_init(ab); - if (ret) - return ret; - clear_bit(ATH11K_FLAG_CRASH_FLUSH, &ab->dev_flags); ret = ath11k_core_qmi_firmware_ready(ab); diff --git a/drivers/net/wireless/ath/ath11k/hal.c b/drivers/net/wireless/ath/ath11k/hal.c index b134470ce2264..eb394ba6f500f 100644 --- a/drivers/net/wireless/ath/ath11k/hal.c +++ b/drivers/net/wireless/ath/ath11k/hal.c @@ -1313,6 +1313,22 @@ void ath11k_hal_srng_deinit(struct ath11k_base *ab) } EXPORT_SYMBOL(ath11k_hal_srng_deinit); +void ath11k_hal_srng_clear(struct ath11k_base *ab) +{ + /* No need to memset rdp and wrp memory since each individual + * segment would get cleared in ath11k_hal_srng_src_hw_init() + * and ath11k_hal_srng_dst_hw_init(). + */ + memset(ab->hal.srng_list, 0, + sizeof(ab->hal.srng_list)); + memset(ab->hal.shadow_reg_addr, 0, + sizeof(ab->hal.shadow_reg_addr)); + ab->hal.avail_blk_resource = 0; + ab->hal.current_blk_index = 0; + ab->hal.num_shadow_reg_configured = 0; +} +EXPORT_SYMBOL(ath11k_hal_srng_clear); + void ath11k_hal_dump_srng_stats(struct ath11k_base *ab) { struct hal_srng *srng; diff --git a/drivers/net/wireless/ath/ath11k/hal.h b/drivers/net/wireless/ath/ath11k/hal.h index 7fdcd8bbf7e98..cbbd714a1fbe0 100644 --- a/drivers/net/wireless/ath/ath11k/hal.h +++ b/drivers/net/wireless/ath/ath11k/hal.h @@ -952,6 +952,7 @@ int ath11k_hal_srng_setup(struct ath11k_base *ab, enum hal_ring_type type, struct hal_srng_params *params); int ath11k_hal_srng_init(struct ath11k_base *ath11k); void ath11k_hal_srng_deinit(struct ath11k_base *ath11k); +void ath11k_hal_srng_clear(struct ath11k_base *ab); void ath11k_hal_dump_srng_stats(struct ath11k_base *ab); void ath11k_hal_srng_get_shadow_config(struct ath11k_base *ab, u32 **cfg, u32 *len); -- 2.51.0