ath10k: pci: Only dump ATH10K_MEM_REGION_TYPE_IOREG when safe
authorBryan O'Donoghue <bryan.odonoghue@linaro.org>
Thu, 19 Dec 2019 13:15:38 +0000 (13:15 +0000)
committerKalle Valo <kvalo@codeaurora.org>
Sun, 26 Jan 2020 10:21:36 +0000 (12:21 +0200)
commitd239380196c4e27a26fa4bea73d2bf994c14ec2d
tree0272640d441671c1641448a232b5d0d03781983b
parentf89ee9927a49d805ae355c6d88a22e11a7ec083a
ath10k: pci: Only dump ATH10K_MEM_REGION_TYPE_IOREG when safe

ath10k_pci_dump_memory_reg() will try to access memory of type
ATH10K_MEM_REGION_TYPE_IOREG however, if a hardware restart is in progress
this can crash a system.

Individual ioread32() time has been observed to jump from 15-20 ticks to >
80k ticks followed by a secure-watchdog bite and a system reset.

Work around this corner case by only issuing the read transaction when the
driver state is ATH10K_STATE_ON.

Tested-on: QCA9988 PCI 10.4-3.9.0.2-00044

Fixes: 219cc084c6706 ("ath10k: add memory dump support QCA9984")
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ath/ath10k/pci.c