2023-07-17 03:53:25

by Wen Gong

[permalink] [raw]
Subject: [PATCH] wifi: ath12k: add handler for scan event WMI_SCAN_EVENT_DEQUEUED

When wlan interface is up, and 11d scan is sent to the firmware, then
firmware needs to spend couple of seconds to complete the 11d scan. If
normal scan from user space arrives to ath12k at this moment, then the
normal scan request is also sent to the firmware, but the scan started
event will be reported to ath12k until the 11d scan complete. When timed
out for the scan started in ath12k, ath12k stops the normal scan and the
firmware reports WMI_SCAN_EVENT_DEQUEUED to ath12k for the normal scan.
ath12k has no handler for the event and then timed out for the scan
completed in ath12k_scan_stop(), and ath12k prints the following error
message.

[ 1491.604750] ath12k_pci 0000:02:00.0: failed to receive scan abort comple: timed out
[ 1491.604756] ath12k_pci 0000:02:00.0: failed to stop scan: -110
[ 1491.604758] ath12k_pci 0000:02:00.0: failed to start hw scan: -110

Add a handler for WMI_SCAN_EVENT_DEQUEUED and then complete the scan to
get rid of the above error message.

Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4

Signed-off-by: Wen Gong <[email protected]>
---
drivers/net/wireless/ath/ath12k/wmi.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/net/wireless/ath/ath12k/wmi.c b/drivers/net/wireless/ath/ath12k/wmi.c
index 7ae0bb78b2b5..8329c1bcbb36 100644
--- a/drivers/net/wireless/ath/ath12k/wmi.c
+++ b/drivers/net/wireless/ath/ath12k/wmi.c
@@ -5688,6 +5688,8 @@ static void ath12k_scan_event(struct ath12k_base *ab, struct sk_buff *skb)
ath12k_wmi_event_scan_start_failed(ar);
break;
case WMI_SCAN_EVENT_DEQUEUED:
+ __ath12k_mac_scan_finish(ar);
+ break;
case WMI_SCAN_EVENT_PREEMPTED:
case WMI_SCAN_EVENT_RESTARTED:
case WMI_SCAN_EVENT_FOREIGN_CHAN_EXIT:

base-commit: 0a00db612b6df1fad80485e3642529d1f28ea084
--
2.40.1



2023-08-03 09:32:42

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] wifi: ath12k: add handler for scan event WMI_SCAN_EVENT_DEQUEUED

Wen Gong <[email protected]> wrote:

> When wlan interface is up, and 11d scan is sent to the firmware, then
> firmware needs to spend couple of seconds to complete the 11d scan. If
> normal scan from user space arrives to ath12k at this moment, then the
> normal scan request is also sent to the firmware, but the scan started
> event will be reported to ath12k until the 11d scan complete. When timed
> out for the scan started in ath12k, ath12k stops the normal scan and the
> firmware reports WMI_SCAN_EVENT_DEQUEUED to ath12k for the normal scan.
> ath12k has no handler for the event and then timed out for the scan
> completed in ath12k_scan_stop(), and ath12k prints the following error
> message.
>
> [ 1491.604750] ath12k_pci 0000:02:00.0: failed to receive scan abort comple: timed out
> [ 1491.604756] ath12k_pci 0000:02:00.0: failed to stop scan: -110
> [ 1491.604758] ath12k_pci 0000:02:00.0: failed to start hw scan: -110
>
> Add a handler for WMI_SCAN_EVENT_DEQUEUED and then complete the scan to
> get rid of the above error message.
>
> Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4
>
> Signed-off-by: Wen Gong <[email protected]>
> Signed-off-by: Kalle Valo <[email protected]>

Patch applied to ath-next branch of ath.git, thanks.

9632ea57be65 wifi: ath12k: add handler for scan event WMI_SCAN_EVENT_DEQUEUED

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches