diff options
| author | Caleb Connolly <[email protected]> | 2025-03-18 20:50:27 +0000 |
|---|---|---|
| committer | Jeff Johnson <[email protected]> | 2025-06-07 14:36:01 +0000 |
| commit | 1650d32b92b01db03a1a95d69ee74fcbc34d4b00 (patch) | |
| tree | cd585327af20ff37b5d77473687a44c014c76271 | |
| parent | Merge tag 'net-next-6.16' of git://git.kernel.org/pub/scm/linux/kernel/git/ne... (diff) | |
| download | kernel-1650d32b92b01db03a1a95d69ee74fcbc34d4b00.tar.gz kernel-1650d32b92b01db03a1a95d69ee74fcbc34d4b00.zip | |
ath10k: snoc: fix unbalanced IRQ enable in crash recovery
In ath10k_snoc_hif_stop() we skip disabling the IRQs in the crash
recovery flow, but we still unconditionally call enable again in
ath10k_snoc_hif_start().
We can't check the ATH10K_FLAG_CRASH_FLUSH bit since it is cleared
before hif_start() is called, so instead check the
ATH10K_SNOC_FLAG_RECOVERY flag and skip enabling the IRQs during crash
recovery.
This fixes unbalanced IRQ enable splats that happen after recovering from
a crash.
Fixes: 0e622f67e041 ("ath10k: add support for WCN3990 firmware crash recovery")
Signed-off-by: Caleb Connolly <[email protected]>
Tested-by: Loic Poulain <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jeff Johnson <[email protected]>
| -rw-r--r-- | drivers/net/wireless/ath/ath10k/snoc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath10k/snoc.c b/drivers/net/wireless/ath/ath10k/snoc.c index 866bad2db334..65673b1aba55 100644 --- a/drivers/net/wireless/ath/ath10k/snoc.c +++ b/drivers/net/wireless/ath/ath10k/snoc.c @@ -937,7 +937,9 @@ static int ath10k_snoc_hif_start(struct ath10k *ar) dev_set_threaded(ar->napi_dev, true); ath10k_core_napi_enable(ar); - ath10k_snoc_irq_enable(ar); + /* IRQs are left enabled when we restart due to a firmware crash */ + if (!test_bit(ATH10K_SNOC_FLAG_RECOVERY, &ar_snoc->flags)) + ath10k_snoc_irq_enable(ar); ath10k_snoc_rx_post(ar); clear_bit(ATH10K_SNOC_FLAG_RECOVERY, &ar_snoc->flags); |