A-Team_Digital_Solutions/android_kernel_motorola_sm6375
Watch
2
0
Fork
You've already forked android_kernel_motorola_sm6375
 0
mirror of https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git synced 2026-10-09 21:59:12 -04:00
Code Issues Projects Releases Packages Wiki Activity

qcacld-3.0: Tx dropped for osif_vdev_sync_op_start failed

Browse source 
osif_psoc_sync_trans_start_wait is wrongly used in
Idceff767d62c1e81046a696cb58f70a897cbd074, which will block any
vdev operation which results in tethering failure and Tx packets
also won't be transmitted whenever ipa event is processed as
we have vdev check in hard transmit too.
Fix:
Don't call osif_psoc_sync_trans_start_wait when handle IPA
event, just check recoverying state.

Change-Id: Ibb5d94e0e22413fdd9dad294fc8da58eba8a700b
CRs-Fixed: 2680423
This commit is contained in:
Jianmin Zhu 2020-05-07 16:03:15 +08:00 • committed by nshrivas
commit 25800f4de6
1 changed files with 3 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

22
components/ipa/core/src/wlan_ipa_core.c
View file

@ -26,8 +26,7 @@
#include "sir_api.h"
#include "host_diag_core_event.h"
#include "wlan_objmgr_vdev_obj.h"
#include "osif_psoc_sync.h"
#include <wlan_cfg80211.h>
#include "qdf_platform.h"
static struct wlan_ipa_priv *gp_ipa;
static void wlan_ipa_set_pending_tx_timer(struct wlan_ipa_priv *ipa_ctx);
@ -3429,27 +3428,12 @@ static void __wlan_ipa_uc_fw_op_event_handler(void *data)
*/
static void wlan_ipa_uc_fw_op_event_handler(void *data)
{
int ret;
struct osif_psoc_sync *psoc_sync;
struct uc_op_work_struct *uc_op_work =
(struct uc_op_work_struct *)data;
if (!uc_op_work || !uc_op_work->osdev) {
ipa_err("Invalid op work");
return;
}
/* This event handler needs to wait for psoc transition */
ret = osif_psoc_sync_trans_start_wait(uc_op_work->osdev->dev,
&psoc_sync);
if (ret) {
ipa_err("op start ret: %d", ret);
if (qdf_is_recovering()) {
ipa_err("in recovering");
return;
}
__wlan_ipa_uc_fw_op_event_handler(data);
osif_psoc_sync_trans_stop(psoc_sync);
}
/**