From: Wen Gong <[email protected]>
v3: remove time_left set but not used in
"ath11k: add synchronization operation between reconfigure of mac80211 and ath11k_base"
v2: s/initilized/initialized in commit log of patch
"ath11k: add synchronization operation between reconfigure of mac80211 and ath11k_base"
Add support for device recovery.
Wen Gong (3):
ath11k: add ath11k_qmi_free_resource() for recovery
ath11k: add support for device recovery for QCA6390
ath11k: add synchronization operation between reconfigure of mac80211
and ath11k_base
drivers/net/wireless/ath/ath11k/core.c | 119 +++++++++++++++++++++++--
drivers/net/wireless/ath/ath11k/core.h | 18 ++++
drivers/net/wireless/ath/ath11k/mac.c | 40 +++++++++
drivers/net/wireless/ath/ath11k/mhi.c | 33 +++++++
drivers/net/wireless/ath/ath11k/pci.c | 3 +
drivers/net/wireless/ath/ath11k/qmi.c | 5 ++
drivers/net/wireless/ath/ath11k/qmi.h | 1 +
7 files changed, 211 insertions(+), 8 deletions(-)
--
2.31.1
Currently ath11k has device recovery logic, it is introduced by this
patch "ath11k: Add support for subsystem recovery" which is upstream
by https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=ath11k-bringup&id=3a7b4838b6f6f234239f263ef3dc02e612a083ad.
The patch is for AHB devices such as IPQ8074, it has remote proc module
which is used to download the firmware and boots the processor which
firmware is running on. If firmware crashed, remote proc module will
detect it and download and boot firmware again. Below command will
trigger a firmware crash, and then user can test feature of device
recovery.
Test command:
echo assert > /sys/kernel/debug/ath11k/qca6390\ hw2.0/simulate_fw_crash
Unfortunately, QCA6390 is PCIe bus, it does not have the remote proc
module, it use mhi module to communicate between firmware and ath11k.
So ath11k does not support device recovery for QCA6390 currently.
This patch is to add the extra logic which is different for QCA6390.
When firmware crashed, MHI_CB_EE_RDDM event will be indicate by
firmware and then ath11k_mhi_op_status_cb which is the callback of
mhi_controller will receive the MHI_CB_EE_RDDM event, then ath11k
will start to do recovery process, ath11k_core_reset() calls
ath11k_hif_power_down()/ath11k_hif_power_up(), then the mhi/ath11k
will start to download and boot firmware. There are some logic to
avoid deadloop recovery and two simultaneous recovery operations.
And because it has muti-radios for the soc, so it add some logic
in ath11k_mac_op_reconfig_complete() to make sure all radios has
reconfig complete and then complete the device recovery.
Also it add workqueue_aux, because ab->workqueue is used when receive
ATH11K_QMI_EVENT_FW_READY in recovery process(queue_work(ab->workqueue,
&ab->restart_work)), and ath11k_core_reset will wait for max
ATH11K_RESET_TIMEOUT_HZ for the previous restart_work finished, if
ath11k_core_reset also queued in ab->workqueue, then it will delay
restart_work of previous recovery and lead previous recovery fail.
ath11k recovery success for QCA6390 after apply this patch.
Tested-on: QCA6390 hw2.0 PCI WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1
Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-01720.1-QCAHSPSWPL_V1_V2_SILICONZ_LITE-1
Signed-off-by: Wen Gong <[email protected]>
---
drivers/net/wireless/ath/ath11k/core.c | 67 ++++++++++++++++++++++++++
drivers/net/wireless/ath/ath11k/core.h | 13 +++++
drivers/net/wireless/ath/ath11k/mac.c | 18 +++++++
drivers/net/wireless/ath/ath11k/mhi.c | 33 +++++++++++++
drivers/net/wireless/ath/ath11k/pci.c | 3 ++
5 files changed, 134 insertions(+)
diff --git a/drivers/net/wireless/ath/ath11k/core.c b/drivers/net/wireless/ath/ath11k/core.c
index 969bf1a590d9..be788ec08200 100644
--- a/drivers/net/wireless/ath/ath11k/core.c
+++ b/drivers/net/wireless/ath/ath11k/core.c
@@ -1043,6 +1043,65 @@ static void ath11k_core_restart(struct work_struct *work)
complete(&ab->driver_recovery);
}
+static void ath11k_core_reset(struct work_struct *work)
+{
+ struct ath11k_base *ab = container_of(work, struct ath11k_base, reset_work);
+ int reset_count, fail_cont_count;
+ long time_left;
+
+ if (!(test_bit(ATH11K_FLAG_REGISTERED, &ab->dev_flags))) {
+ ath11k_warn(ab, "ignore reset dev flags 0x%lx\n", ab->dev_flags);
+ return;
+ }
+
+ /* Sometimes the recovery will fail and then the next all recovery fail,
+ * this is to avoid infinite recovery since it can not recovery success.
+ */
+ fail_cont_count = atomic_read(&ab->fail_cont_count);
+
+ if (fail_cont_count >= ATH11K_RESET_MAX_FAIL_COUNT_FINAL)
+ return;
+
+ if (fail_cont_count >= ATH11K_RESET_MAX_FAIL_COUNT_FIRST &&
+ time_before(jiffies, ab->reset_fail_timeout))
+ return;
+
+ reset_count = atomic_inc_return(&ab->reset_count);
+
+ if (reset_count > 1) {
+ /* Sometimes it happened another reset worker before the previous one
+ * completed, then the second reset worker will destroy the previous one,
+ * thus below is to avoid that.
+ */
+ ath11k_warn(ab, "already reseting count %d\n", reset_count);
+
+ reinit_completion(&ab->reset_complete);
+ time_left = wait_for_completion_timeout(&ab->reset_complete,
+ ATH11K_RESET_TIMEOUT_HZ);
+
+ if (time_left) {
+ ath11k_dbg(ab, ATH11K_DBG_BOOT, "to skip reset\n");
+ atomic_dec(&ab->reset_count);
+ return;
+ }
+
+ ab->reset_fail_timeout = jiffies + ATH11K_RESET_FAIL_TIMEOUT_HZ;
+ /* Record the continuous recovery fail count when recovery failed*/
+ fail_cont_count = atomic_inc_return(&ab->fail_cont_count);
+ }
+
+ ath11k_dbg(ab, ATH11K_DBG_BOOT, "reset starting\n");
+
+ ab->is_reset = true;
+ atomic_set(&ab->recovery_count, 0);
+
+ ath11k_hif_power_down(ab);
+ ath11k_qmi_free_resource(ab);
+ ath11k_hif_power_up(ab);
+
+ ath11k_dbg(ab, ATH11K_DBG_BOOT, "reset started\n");
+}
+
static int ath11k_init_hw_params(struct ath11k_base *ab)
{
const struct ath11k_hw_params *hw_params = NULL;
@@ -1132,14 +1191,20 @@ struct ath11k_base *ath11k_core_alloc(struct device *dev, size_t priv_size,
if (!ab->workqueue)
goto err_sc_free;
+ ab->workqueue_aux = create_singlethread_workqueue("ath11k_aux_wq");
+ if (!ab->workqueue_aux)
+ goto err_free_wq;
+
mutex_init(&ab->core_lock);
spin_lock_init(&ab->base_lock);
+ init_completion(&ab->reset_complete);
INIT_LIST_HEAD(&ab->peers);
init_waitqueue_head(&ab->peer_mapping_wq);
init_waitqueue_head(&ab->wmi_ab.tx_credits_wq);
init_waitqueue_head(&ab->qmi.cold_boot_waitq);
INIT_WORK(&ab->restart_work, ath11k_core_restart);
+ INIT_WORK(&ab->reset_work, ath11k_core_reset);
timer_setup(&ab->rx_replenish_retry, ath11k_ce_rx_replenish_retry, 0);
init_completion(&ab->htc_suspend);
init_completion(&ab->wow.wakeup_completed);
@@ -1150,6 +1215,8 @@ struct ath11k_base *ath11k_core_alloc(struct device *dev, size_t priv_size,
return ab;
+err_free_wq:
+ destroy_workqueue(ab->workqueue);
err_sc_free:
kfree(ab);
return NULL;
diff --git a/drivers/net/wireless/ath/ath11k/core.h b/drivers/net/wireless/ath/ath11k/core.h
index 018fb2385f2a..9a9f8f24d407 100644
--- a/drivers/net/wireless/ath/ath11k/core.h
+++ b/drivers/net/wireless/ath/ath11k/core.h
@@ -39,6 +39,10 @@
extern unsigned int ath11k_frame_mode;
#define ATH11K_MON_TIMER_INTERVAL 10
+#define ATH11K_RESET_TIMEOUT_HZ (20 * HZ)
+#define ATH11K_RESET_MAX_FAIL_COUNT_FIRST 3
+#define ATH11K_RESET_MAX_FAIL_COUNT_FINAL 5
+#define ATH11K_RESET_FAIL_TIMEOUT_HZ (20 * HZ)
enum ath11k_supported_bw {
ATH11K_BW_20 = 0,
@@ -734,6 +738,15 @@ struct ath11k_base {
struct completion driver_recovery;
struct workqueue_struct *workqueue;
struct work_struct restart_work;
+ struct workqueue_struct *workqueue_aux;
+ struct work_struct reset_work;
+ atomic_t reset_count;
+ atomic_t recovery_count;
+ bool is_reset;
+ struct completion reset_complete;
+ /* continuous recovery fail count */
+ atomic_t fail_cont_count;
+ unsigned long reset_fail_timeout;
struct {
/* protected by data_lock */
u32 fw_crash_counter;
diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
index eb52332dbe3f..b0a2f257f328 100644
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -6032,6 +6032,8 @@ ath11k_mac_op_reconfig_complete(struct ieee80211_hw *hw,
enum ieee80211_reconfig_type reconfig_type)
{
struct ath11k *ar = hw->priv;
+ struct ath11k_base *ab = ar->ab;
+ int recovery_count;
if (reconfig_type != IEEE80211_RECONFIG_TYPE_RESTART)
return;
@@ -6043,6 +6045,22 @@ ath11k_mac_op_reconfig_complete(struct ieee80211_hw *hw,
ar->pdev->pdev_id);
ar->state = ATH11K_STATE_ON;
ieee80211_wake_queues(ar->hw);
+
+ if (ab->is_reset) {
+ recovery_count = atomic_inc_return(&ab->recovery_count);
+ ath11k_dbg(ab, ATH11K_DBG_BOOT,
+ "recovery count %d\n", recovery_count);
+ /* When there are multiple radios in an SOC,
+ * the recovery has to be done for each radio
+ */
+ if (recovery_count == ab->num_radios) {
+ atomic_dec(&ab->reset_count);
+ complete(&ab->reset_complete);
+ ab->is_reset = false;
+ atomic_set(&ab->fail_cont_count, 0);
+ ath11k_dbg(ab, ATH11K_DBG_BOOT, "reset success\n");
+ }
+ }
}
mutex_unlock(&ar->conf_mutex);
diff --git a/drivers/net/wireless/ath/ath11k/mhi.c b/drivers/net/wireless/ath/ath11k/mhi.c
index 75cc2d80fde8..aea21ea2ca47 100644
--- a/drivers/net/wireless/ath/ath11k/mhi.c
+++ b/drivers/net/wireless/ath/ath11k/mhi.c
@@ -281,15 +281,48 @@ static void ath11k_mhi_op_runtime_put(struct mhi_controller *mhi_cntrl)
{
}
+static char *ath11k_mhi_op_callback_to_str(enum mhi_callback reason)
+{
+ switch (reason) {
+ case MHI_CB_IDLE:
+ return "MHI_CB_IDLE";
+ case MHI_CB_PENDING_DATA:
+ return "MHI_CB_PENDING_DATA";
+ case MHI_CB_LPM_ENTER:
+ return "MHI_CB_LPM_ENTER";
+ case MHI_CB_LPM_EXIT:
+ return "MHI_CB_LPM_EXIT";
+ case MHI_CB_EE_RDDM:
+ return "MHI_CB_EE_RDDM";
+ case MHI_CB_EE_MISSION_MODE:
+ return "MHI_CB_EE_MISSION_MODE";
+ case MHI_CB_SYS_ERROR:
+ return "MHI_CB_SYS_ERROR";
+ case MHI_CB_FATAL_ERROR:
+ return "MHI_CB_FATAL_ERROR";
+ case MHI_CB_BW_REQ:
+ return "MHI_CB_BW_REQ";
+ default:
+ return "UNKNOWN";
+ }
+};
+
static void ath11k_mhi_op_status_cb(struct mhi_controller *mhi_cntrl,
enum mhi_callback cb)
{
struct ath11k_base *ab = dev_get_drvdata(mhi_cntrl->cntrl_dev);
+ ath11k_dbg(ab, ATH11K_DBG_BOOT, "mhi notify status reason %s\n",
+ ath11k_mhi_op_callback_to_str(cb));
+
switch (cb) {
case MHI_CB_SYS_ERROR:
ath11k_warn(ab, "firmware crashed: MHI_CB_SYS_ERROR\n");
break;
+ case MHI_CB_EE_RDDM:
+ if (!(test_bit(ATH11K_FLAG_UNREGISTERING, &ab->dev_flags)))
+ queue_work(ab->workqueue_aux, &ab->reset_work);
+ break;
default:
break;
}
diff --git a/drivers/net/wireless/ath/ath11k/pci.c b/drivers/net/wireless/ath/ath11k/pci.c
index 646ad79f309c..1f8e3837cdfb 100644
--- a/drivers/net/wireless/ath/ath11k/pci.c
+++ b/drivers/net/wireless/ath/ath11k/pci.c
@@ -1346,6 +1346,7 @@ static void ath11k_pci_remove(struct pci_dev *pdev)
set_bit(ATH11K_FLAG_UNREGISTERING, &ab->dev_flags);
+ cancel_work_sync(&ab->reset_work);
ath11k_core_deinit(ab);
qmi_fail:
@@ -1357,6 +1358,8 @@ static void ath11k_pci_remove(struct pci_dev *pdev)
ath11k_hal_srng_deinit(ab);
ath11k_ce_free_pipes(ab);
+
+ destroy_workqueue(ab->workqueue_aux);
ath11k_core_free(ab);
}
--
2.31.1
Wen Gong <[email protected]> writes:
> From: Wen Gong <[email protected]>
>
> v3: remove time_left set but not used in
> "ath11k: add synchronization operation between reconfigure of mac80211 and ath11k_base"
>
> v2: s/initilized/initialized in commit log of patch
> "ath11k: add synchronization operation between reconfigure of mac80211 and ath11k_base"
>
> Add support for device recovery.
>
> Wen Gong (3):
> ath11k: add ath11k_qmi_free_resource() for recovery
> ath11k: add support for device recovery for QCA6390
> ath11k: add synchronization operation between reconfigure of mac80211
> and ath11k_base
I did some quick testing (commit 4716d5bb1e16 in pending branch) and it
does not seem to work for me:
oot@nuc1:~# echo assert > /sys/kernel/debug/ath11k/qca6390\ hw2.0/simulate_fw_crash
root@nuc1:~# iw wlan0 scan
command failed: Connection timed out (-110)
root@nuc1:~#
Kernel logs have:
[ 118.903092] ath11k_pci 0000:06:00.0: BAR 0: assigned [mem 0xdb000000-0xdbffffff 64bit]
[ 118.925164] ath11k_pci 0000:06:00.0: MSI vectors: 32
[ 118.925285] ath11k_pci 0000:06:00.0: qca6390 hw2.0
[ 119.093214] mhi mhi0: Requested to power ON
[ 119.095430] mhi mhi0: Power on setup success
[ 119.423735] mhi mhi0: Wait for device to enter SBL or Mission mode
[ 119.543030] ath11k_pci 0000:06:00.0: chip_id 0x0 chip_family 0xb board_id 0xff soc_id 0xffffffff
[ 119.543197] ath11k_pci 0000:06:00.0: fw_version 0x101c06cc fw_build_timestamp 2020-06-24 19:50 fw_build_id
[ 230.101077] ip (2367) used greatest stack depth: 24384 bytes left
[ 283.663895] ath11k_pci 0000:06:00.0: simulating firmware assert crash
[ 283.738422] ath11k_pci 0000:06:00.0: firmware crashed: MHI_CB_SYS_ERROR
[ 295.236831] ath11k_pci 0000:06:00.0: wmi command 12290 timeout
[ 295.237937] ath11k_pci 0000:06:00.0: failed to send WMI_STOP_SCAN_CMDID
[ 295.238467] ath11k_pci 0000:06:00.0: failed to stop wmi scan: -11
[ 295.240049] ath11k_pci 0000:06:00.0: failed to stop scan: -11
[ 295.240558] ath11k_pci 0000:06:00.0: failed to start hw scan: -110
Also there should be a clear ath11k_warn() message when firmware
recovery was successful, I could not find one from patches.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
On 2021-11-15 20:53, Kalle Valo wrote:
> Wen Gong <[email protected]> writes:
>
>> From: Wen Gong <[email protected]>
>>
>> v3: remove time_left set but not used in
>> "ath11k: add synchronization operation between reconfigure of
>> mac80211 and ath11k_base"
>>
>> v2: s/initilized/initialized in commit log of patch
>> "ath11k: add synchronization operation between reconfigure of
>> mac80211 and ath11k_base"
>>
>> Add support for device recovery.
>>
>> Wen Gong (3):
>> ath11k: add ath11k_qmi_free_resource() for recovery
>> ath11k: add support for device recovery for QCA6390
>> ath11k: add synchronization operation between reconfigure of
>> mac80211
>> and ath11k_base
>
> I did some quick testing (commit 4716d5bb1e16 in pending branch) and it
> does not seem to work for me:
>
> oot@nuc1:~# echo assert > /sys/kernel/debug/ath11k/qca6390\
> hw2.0/simulate_fw_crash
> root@nuc1:~# iw wlan0 scan
> command failed: Connection timed out (-110)
> root@nuc1:~#
>
> Kernel logs have:
>
> [ 118.903092] ath11k_pci 0000:06:00.0: BAR 0: assigned [mem
> 0xdb000000-0xdbffffff 64bit]
> [ 118.925164] ath11k_pci 0000:06:00.0: MSI vectors: 32
> [ 118.925285] ath11k_pci 0000:06:00.0: qca6390 hw2.0
> [ 119.093214] mhi mhi0: Requested to power ON
> [ 119.095430] mhi mhi0: Power on setup success
> [ 119.423735] mhi mhi0: Wait for device to enter SBL or Mission mode
> [ 119.543030] ath11k_pci 0000:06:00.0: chip_id 0x0 chip_family 0xb
> board_id 0xff soc_id 0xffffffff
> [ 119.543197] ath11k_pci 0000:06:00.0: fw_version 0x101c06cc
> fw_build_timestamp 2020-06-24 19:50 fw_build_id
> [ 230.101077] ip (2367) used greatest stack depth: 24384 bytes left
> [ 283.663895] ath11k_pci 0000:06:00.0: simulating firmware assert
> crash
> [ 283.738422] ath11k_pci 0000:06:00.0: firmware crashed:
> MHI_CB_SYS_ERROR
> [ 295.236831] ath11k_pci 0000:06:00.0: wmi command 12290 timeout
> [ 295.237937] ath11k_pci 0000:06:00.0: failed to send
> WMI_STOP_SCAN_CMDID
> [ 295.238467] ath11k_pci 0000:06:00.0: failed to stop wmi scan: -11
> [ 295.240049] ath11k_pci 0000:06:00.0: failed to stop scan: -11
> [ 295.240558] ath11k_pci 0000:06:00.0: failed to start hw scan: -110
>
I have sent v4, it is not to fix your fail:)
It added patch "ath11k: fix invalid m3 buffer address".
https://patchwork.kernel.org/project/linux-wireless/cover/[email protected]/
> Also there should be a clear ath11k_warn() message when firmware
> recovery was successful, I could not find one from patches.
Yes, if recovery succes, it will have "success" log.
Wen Gong <[email protected]> writes:
> On 2021-11-15 20:53, Kalle Valo wrote:
>> Wen Gong <[email protected]> writes:
>>
>>> From: Wen Gong <[email protected]>
>>>
>>> v3: remove time_left set but not used in
>>> "ath11k: add synchronization operation between reconfigure of
>>> mac80211 and ath11k_base"
>>>
>>> v2: s/initilized/initialized in commit log of patch
>>> "ath11k: add synchronization operation between reconfigure of
>>> mac80211 and ath11k_base"
>>>
>>> Add support for device recovery.
>>>
>>> Wen Gong (3):
>>> ath11k: add ath11k_qmi_free_resource() for recovery
>>> ath11k: add support for device recovery for QCA6390
>>> ath11k: add synchronization operation between reconfigure of
>>> mac80211
>>> and ath11k_base
>>
>> I did some quick testing (commit 4716d5bb1e16 in pending branch) and it
>> does not seem to work for me:
>>
>> oot@nuc1:~# echo assert > /sys/kernel/debug/ath11k/qca6390\
>> hw2.0/simulate_fw_crash
>> root@nuc1:~# iw wlan0 scan
>> command failed: Connection timed out (-110)
>> root@nuc1:~#
>>
>> Kernel logs have:
>>
>> [ 118.903092] ath11k_pci 0000:06:00.0: BAR 0: assigned [mem
>> 0xdb000000-0xdbffffff 64bit]
>> [ 118.925164] ath11k_pci 0000:06:00.0: MSI vectors: 32
>> [ 118.925285] ath11k_pci 0000:06:00.0: qca6390 hw2.0
>> [ 119.093214] mhi mhi0: Requested to power ON
>> [ 119.095430] mhi mhi0: Power on setup success
>> [ 119.423735] mhi mhi0: Wait for device to enter SBL or Mission mode
>> [ 119.543030] ath11k_pci 0000:06:00.0: chip_id 0x0 chip_family 0xb
>> board_id 0xff soc_id 0xffffffff
>> [ 119.543197] ath11k_pci 0000:06:00.0: fw_version 0x101c06cc
>> fw_build_timestamp 2020-06-24 19:50 fw_build_id
>> [ 230.101077] ip (2367) used greatest stack depth: 24384 bytes left
>> [ 283.663895] ath11k_pci 0000:06:00.0: simulating firmware assert
>> crash
>> [ 283.738422] ath11k_pci 0000:06:00.0: firmware crashed:
>> MHI_CB_SYS_ERROR
>> [ 295.236831] ath11k_pci 0000:06:00.0: wmi command 12290 timeout
>> [ 295.237937] ath11k_pci 0000:06:00.0: failed to send
>> WMI_STOP_SCAN_CMDID
>> [ 295.238467] ath11k_pci 0000:06:00.0: failed to stop wmi scan: -11
>> [ 295.240049] ath11k_pci 0000:06:00.0: failed to stop scan: -11
>> [ 295.240558] ath11k_pci 0000:06:00.0: failed to start hw scan: -110
>>
>
> I have sent v4, it is not to fix your fail:)
> It added patch "ath11k: fix invalid m3 buffer address".
> https://patchwork.kernel.org/project/linux-wireless/cover/[email protected]/
>
>
>> Also there should be a clear ath11k_warn() message when firmware
>> recovery was successful, I could not find one from patches.
>
> Yes, if recovery succes, it will have "success" log.
Where are the printouts? I cannot find any ath11k_warn() messages in v4.
Basically I would want to see something like this when the firmware
crashes:
firmware crashed, restarting it
And once the firmware is running again:
firmware restarted succesfully
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
On 2021-11-17 16:08, Kalle Valo wrote:
> Wen Gong <[email protected]> writes:
>
>> On 2021-11-15 20:53, Kalle Valo wrote:
>>> Wen Gong <[email protected]> writes:
>>>
>>>> From: Wen Gong <[email protected]>
>>>>
...
>>>
>>
>> I have sent v4, it is not to fix your fail:)
>> It added patch "ath11k: fix invalid m3 buffer address".
>> https://patchwork.kernel.org/project/linux-wireless/cover/[email protected]/
>>
>>
>>> Also there should be a clear ath11k_warn() message when firmware
>>> recovery was successful, I could not find one from patches.
>>
>> Yes, if recovery succes, it will have "success" log.
>
> Where are the printouts? I cannot find any ath11k_warn() messages in
> v4.
> Basically I would want to see something like this when the firmware
> crashes:
It is not print in this patch.
It is here:
ath11k_mac_op_reconfig_complete(struct ieee80211_hw *hw,
enum ieee80211_reconfig_type reconfig_type)
if (ar->state == ATH11K_STATE_RESTARTED) {
ath11k_warn(ar->ab, "pdev %d successfully recovered\n",
ar->pdev->pdev_id);
>
> firmware crashed, restarting it
>
> And once the firmware is running again:
>
> firmware restarted succesfully
"firmware crashed" log is here:
static void ath11k_mhi_op_status_cb(struct mhi_controller *mhi_cntrl,
enum mhi_callback cb)
{
struct ath11k_base *ab = dev_get_drvdata(mhi_cntrl->cntrl_dev);
struct ath11k_pci *ar_pci = ath11k_pci_priv(ab);
switch (cb) {
case MHI_CB_SYS_ERROR:
ath11k_warn(ab, "firmware crashed: MHI_CB_SYS_ERROR\n");