Bladeren bron

qcacmn: Add debug logs in cp_stats event path

In the issue reported, the clear stats request from the
user space is timing out because event id from firmware is not received.
From the logs it is not clear whether host driver received it or dropped
it in the error path.

Add debug logs, to make sure cp_stats event id is received in the host
driver and also in error cases.

Change-Id: I634714281db4a82826791725ec38fd589334cde8
CRs-Fixed: 2926978
Srinivas Girigowda 4 jaren geleden
bovenliggende
commit
216b52af2f
3 gewijzigde bestanden met toevoegingen van 8 en 2 verwijderingen
  1. 2 0
      target_if/cp_stats/src/target_if_cp_stats.c
  2. 3 1
      wmi/src/wmi_unified_cp_stats_tlv.c
  3. 3 1
      wmi/src/wmi_unified_tlv.c

+ 2 - 0
target_if/cp_stats/src/target_if_cp_stats.c

@@ -168,6 +168,8 @@ int target_if_infra_cp_stats_event_handler(ol_scn_t scn, uint8_t *data,
 	struct wmi_unified *wmi_handle;
 	struct wmi_unified *wmi_handle;
 	struct wlan_lmac_if_cp_stats_rx_ops *rx_ops;
 	struct wlan_lmac_if_cp_stats_rx_ops *rx_ops;
 
 
+	cp_stats_debug("Enter");
+
 	if (!scn || !data) {
 	if (!scn || !data) {
 		cp_stats_err("scn: 0x%pK, data: 0x%pK", scn, data);
 		cp_stats_err("scn: 0x%pK, data: 0x%pK", scn, data);
 		return -EINVAL;
 		return -EINVAL;

+ 3 - 1
wmi/src/wmi_unified_cp_stats_tlv.c

@@ -135,8 +135,10 @@ QDF_STATUS wmi_stats_handler(void *buff, int32_t len,
 	uint8_t *tag_start_ptr;
 	uint8_t *tag_start_ptr;
 
 
 	param_buf = (WMI_CTRL_PATH_STATS_EVENTID_param_tlvs *)buff;
 	param_buf = (WMI_CTRL_PATH_STATS_EVENTID_param_tlvs *)buff;
-	if (!param_buf)
+	if (!param_buf) {
+		wmi_err_rl("param_buf is NULL");
 		return QDF_STATUS_E_FAILURE;
 		return QDF_STATUS_E_FAILURE;
+	}
 	ev = (wmi_ctrl_path_stats_event_fixed_param *)param_buf->fixed_param;
 	ev = (wmi_ctrl_path_stats_event_fixed_param *)param_buf->fixed_param;
 
 
 	curr_tlv_tag = WMITLV_GET_TLVTAG(ev->tlv_header);
 	curr_tlv_tag = WMITLV_GET_TLVTAG(ev->tlv_header);

+ 3 - 1
wmi/src/wmi_unified_tlv.c

@@ -4938,8 +4938,10 @@ extract_cp_stats_more_pending_tlv(wmi_unified_t wmi, void *evt_buf,
 	wmi_ctrl_path_stats_event_fixed_param *ev;
 	wmi_ctrl_path_stats_event_fixed_param *ev;
 
 
 	param_buf = (WMI_CTRL_PATH_STATS_EVENTID_param_tlvs *)evt_buf;
 	param_buf = (WMI_CTRL_PATH_STATS_EVENTID_param_tlvs *)evt_buf;
-	if (!param_buf)
+	if (!param_buf) {
+		wmi_err_rl("param_buf is NULL");
 		return QDF_STATUS_E_FAILURE;
 		return QDF_STATUS_E_FAILURE;
+	}
 	ev = (wmi_ctrl_path_stats_event_fixed_param *)param_buf->fixed_param;
 	ev = (wmi_ctrl_path_stats_event_fixed_param *)param_buf->fixed_param;
 
 
 	*more_flag = ev->more;
 	*more_flag = ev->more;