Browse Source

qcacld-3.0: Avoid excessive kmsg log in roam stats event handler

Sometimes firmware sends WMI_ROAM_STATS_EVENTID repeatedly with
wrong TLV. This causes event extraction failure at the event
handler and continuous error logs are written to kmsg.

Rate limit debug level prints in wma_roam_stats_event_handler()

Change-Id: I03c1a50198d26096333d9b42190b34204453155a
CRs-Fixed: 2829293
Jyoti Kumari 4 years ago
parent
commit
47316d9679
1 changed files with 14 additions and 14 deletions
  1. 14 14
      core/wma/src/wma_scan_roam.c

+ 14 - 14
core/wma/src/wma_scan_roam.c

@@ -2225,8 +2225,8 @@ int wma_roam_stats_event_handler(WMA_HANDLE handle, uint8_t *event,
 					wma->wmi_handle, event,
 					&roam_info->data_11kv, 0, 0);
 		if (QDF_IS_STATUS_ERROR(status)) {
-			wma_debug("Roam 11kv stats extract failed vdev %d",
-				  vdev_id);
+			wma_debug_rl("Roam 11kv stats extract failed vdev %d",
+				     vdev_id);
 			qdf_mem_free(roam_info);
 			goto err;
 		}
@@ -2251,8 +2251,8 @@ int wma_roam_stats_event_handler(WMA_HANDLE handle, uint8_t *event,
 						wma->wmi_handle, event,
 						&roam_info->trigger, i);
 		if (QDF_IS_STATUS_ERROR(status)) {
-			wma_debug("Extract roam trigger stats failed vdev %d at %d iteration",
-				  vdev_id, i);
+			wma_debug_rl("Extract roam trigger stats failed vdev %d at %d iteration",
+				     vdev_id, i);
 			qdf_mem_free(roam_info);
 			return -EINVAL;
 		}
@@ -2263,8 +2263,8 @@ int wma_roam_stats_event_handler(WMA_HANDLE handle, uint8_t *event,
 							&roam_info->scan, i,
 							num_chan, num_ap);
 		if (QDF_IS_STATUS_ERROR(status)) {
-			wma_debug("Roam scan stats extract failed vdev %d at %d iteration",
-				  vdev_id, i);
+			wma_debug_rl("Roam scan stats extract failed vdev %d at %d iteration",
+				     vdev_id, i);
 			qdf_mem_free(roam_info);
 			return -EINVAL;
 		}
@@ -2276,8 +2276,8 @@ int wma_roam_stats_event_handler(WMA_HANDLE handle, uint8_t *event,
 							wma->wmi_handle, event,
 							&roam_info->result, i);
 		if (QDF_IS_STATUS_ERROR(status)) {
-			wma_debug("Roam result stats extract failed vdev %d at %d iteration",
-				  vdev_id, i);
+			wma_debug_rl("Roam result stats extract failed vdev %d at %d iteration",
+				     vdev_id, i);
 			qdf_mem_free(roam_info);
 			return -EINVAL;
 		}
@@ -2288,8 +2288,8 @@ int wma_roam_stats_event_handler(WMA_HANDLE handle, uint8_t *event,
 							   &roam_info->btm_rsp,
 							   i);
 		if (QDF_IS_STATUS_ERROR(status)) {
-			wma_debug("Roam btm rsp stats extract fail vdev %d at %d iteration",
-				  vdev_id, i);
+			wma_debug_rl("Roam btm rsp stats extract fail vdev %d at %d iteration",
+				     vdev_id, i);
 			qdf_mem_free(roam_info);
 			return -EINVAL;
 		}
@@ -2299,8 +2299,8 @@ int wma_roam_stats_event_handler(WMA_HANDLE handle, uint8_t *event,
 				wma->wmi_handle, event,
 				&roam_info->roam_init_info, i);
 		if (QDF_IS_STATUS_ERROR(status)) {
-			wma_debug("Initial roam stats extract fail vdev %d at %d iteration",
-				  vdev_id, i);
+			wma_debug_rl("Initial roam stats extract fail vdev %d at %d iteration",
+				     vdev_id, i);
 			qdf_mem_free(roam_info);
 			return -EINVAL;
 		}
@@ -2310,8 +2310,8 @@ int wma_roam_stats_event_handler(WMA_HANDLE handle, uint8_t *event,
 				wma->wmi_handle, event,
 				&roam_info->data_11kv, i, num_rpt);
 		if (QDF_IS_STATUS_ERROR(status)) {
-			wma_debug("Roam 11kv stats extract failed vdev %d at %d iteration",
-				  vdev_id, i);
+			wma_debug_rl("Roam 11kv stats extract failed vdev %d at %d iteration",
+				     vdev_id, i);
 			qdf_mem_free(roam_info);
 			return -EINVAL;
 		}