drivers/net/wireless/ath/ath11k/pci.c | 2 ++ 1 file changed, 2 insertions(+)
Make sure to clear the IRQ affinity hint also on shutdown to avoid
triggering a WARN_ON_ONCE() in __free_irq() when stopping MHI while
using a single MSI vector.
Fixes: e94b07493da3 ("ath11k: Set IRQ affinity to CPU0 in case of one MSI vector")
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
---
drivers/net/wireless/ath/ath11k/pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/net/wireless/ath/ath11k/pci.c b/drivers/net/wireless/ath/ath11k/pci.c
index dedf1b88ddf6..487a303b3077 100644
--- a/drivers/net/wireless/ath/ath11k/pci.c
+++ b/drivers/net/wireless/ath/ath11k/pci.c
@@ -920,7 +920,9 @@ static void ath11k_pci_remove(struct pci_dev *pdev)
static void ath11k_pci_shutdown(struct pci_dev *pdev)
{
struct ath11k_base *ab = pci_get_drvdata(pdev);
+ struct ath11k_pci *ab_pci = ath11k_pci_priv(ab);
+ ath11k_pci_set_irq_affinity_hint(ab_pci, NULL);
ath11k_pci_power_down(ab);
}
--
2.35.1
Johan Hovold <johan+linaro@kernel.org> wrote: > Make sure to clear the IRQ affinity hint also on shutdown to avoid > triggering a WARN_ON_ONCE() in __free_irq() when stopping MHI while > using a single MSI vector. > > Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-03125-QCAHSPSWPL_V1_V2_SILICONZ_LITE-3 > > Fixes: e94b07493da3 ("ath11k: Set IRQ affinity to CPU0 in case of one MSI vector") > Signed-off-by: Johan Hovold <johan+linaro@kernel.org> > Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com> Patch applied to ath-next branch of ath.git, thanks. 3bd0c69653ac ath11k: fix IRQ affinity warning on shutdown -- https://patchwork.kernel.org/project/linux-wireless/patch/20220523143258.24818-1-johan+linaro@kernel.org/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
On Mon, May 23, 2022 at 04:32:58PM +0200, Johan Hovold wrote: > Make sure to clear the IRQ affinity hint also on shutdown to avoid > triggering a WARN_ON_ONCE() in __free_irq() when stopping MHI while > using a single MSI vector. Forgot the tested-on tag, sorry. Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-03125-QCAHSPSWPL_V1_V2_SILICONZ_LITE-3 > Fixes: e94b07493da3 ("ath11k: Set IRQ affinity to CPU0 in case of one MSI vector") > Signed-off-by: Johan Hovold <johan+linaro@kernel.org> Let me know if I should resend. Johan
Johan Hovold <johan@kernel.org> writes: > On Mon, May 23, 2022 at 04:32:58PM +0200, Johan Hovold wrote: >> Make sure to clear the IRQ affinity hint also on shutdown to avoid >> triggering a WARN_ON_ONCE() in __free_irq() when stopping MHI while >> using a single MSI vector. > > Forgot the tested-on tag, sorry. > > Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-03125-QCAHSPSWPL_V1_V2_SILICONZ_LITE-3 Thanks, added in the pending branch. >> Fixes: e94b07493da3 ("ath11k: Set IRQ affinity to CPU0 in case of one MSI vector") >> Signed-off-by: Johan Hovold <johan+linaro@kernel.org> > > Let me know if I should resend. No need, I can easily edit commit logs. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
© 2016 - 2024 Red Hat, Inc.