malhuda/msm-4.14
1
0
Fork 0
mirror of https://github.com/rd-stuffs/msm-4.14.git synced 2025-02-20 11:45:48 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

qcacld-3.0: Fix conditional umac_stop_complete_cb() implementation

Browse Source 
After turning the location services off (to actually turn off or on the
Wi-Fi device), users started reporting a 30 seconds delay when their
PCIe Wi-Fi device is brought back up.

This is due to Qualcomm's improper conditional umac_stop_complete_cb()
implementation that simply neglects `qdf_event_set(stop_evt)` when
QDF_ENABLE_TRACING is not defined.

As our local modification disables QDF_ENABLE_TRACING, this
implementation bug triggers a 30 seconds delayed reset.

All functions/APIs used within umac_stop_complete_cb() are available
without QDF_ENABLE_TRACING defined, so remove the conditional
implementation.

Fixes: bcd3d019d8e1 ("qcacld-3.0: Execute sme_stop and mac_stop in mc thread context")
Change-Id: I6055404c5df4e0232ea344e1c2669871e61cb9a7
Signed-off-by: Juhyung Park <qkrwngud825@gmail.com>
Signed-off-by: Richard Raya <rdxzv.dev@gmail.com>
This commit is contained in:
Juhyung Park 2025-02-01 23:18:39 +09:00 committed by Richard Raya
parent 74413c1378
commit 238de8f978
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
drivers/staging/qcacld-3.0/core/mac/src/sys/common/src/wlan_qct_sys.c
View File

@ -56,7 +56,6 @@ QDF_STATUS sys_build_message_header(SYS_MSG_ID msg_id,
*
* Return: none
*/
#ifdef QDF_ENABLE_TRACING
static void umac_stop_complete_cb(void *user_data)
{
qdf_event_t *stop_evt = (qdf_event_t *) user_data;
@ -64,12 +63,6 @@ static void umac_stop_complete_cb(void *user_data)
QDF_ASSERT(QDF_IS_STATUS_SUCCESS(qdf_status));
}
#else
static void umac_stop_complete_cb(void *user_data)
{
return;
}
#endif
static inline QDF_STATUS umac_stop_flush_cb(struct scheduler_msg *msg)
{