Explorar el Código

qcacmn: Fix memory allocation latency in beacon process

Currently if the number of APs in the STA environment
are many, then the STA will receive many beacons, whose
beacon process path can take long time, in the kernel
work queue, hence the other processes have to wait
for them to complete, and may get timeout, if the
the time to process the beaocns is larger than their
process timeout.

Fix is to :-
1. Add rate limit to failure conditions of memory
not allocated
2. Make memory allocation in path of beacon process
atomic.

Change-Id: I0b15fd4924bb7d696a33adeb1875f1f9c277db18
CRs-Fixed: 2360624
gaurank kathpalia hace 6 años
padre
commit
0f9f019505

+ 3 - 2
umac/cmn_services/mgmt_txrx/dispatcher/src/wlan_mgmt_txrx_tgt_api.c

@@ -809,9 +809,10 @@ static QDF_STATUS wlan_mgmt_txrx_rx_handler_list_copy(
 	struct mgmt_rx_handler *rx_handler_node;
 
 	while (rx_handler) {
-		rx_handler_node = qdf_mem_malloc(sizeof(*rx_handler_node));
+		rx_handler_node =
+				qdf_mem_malloc_atomic(sizeof(*rx_handler_node));
 		if (!rx_handler_node) {
-			mgmt_txrx_err("Couldn't allocate memory for rx handler node");
+			mgmt_txrx_err_rl("Couldn't allocate memory for rx handler node");
 			return QDF_STATUS_E_NOMEM;
 		}
 

+ 3 - 0
umac/p2p/core/src/wlan_p2p_main.h

@@ -44,6 +44,9 @@
 	QDF_TRACE_WARN(QDF_MODULE_ID_P2P, params)
 #define p2p_err(params ...) \
 	QDF_TRACE_ERROR(QDF_MODULE_ID_P2P, params)
+#define p2p_debug_rl(params...) \
+	QDF_TRACE_DEBUG_RL(QDF_MODULE_ID_P2P, params)
+
 #define p2p_alert(params ...) \
 	QDF_TRACE_FATAL(QDF_MODULE_ID_P2P, params)
 

+ 4 - 4
umac/p2p/dispatcher/src/wlan_p2p_tgt_api.c

@@ -331,17 +331,17 @@ QDF_STATUS tgt_p2p_mgmt_frame_rx_cb(struct wlan_objmgr_psoc *psoc,
 		vdev_id = wlan_vdev_get_id(vdev);
 	}
 
-	rx_mgmt_event = qdf_mem_malloc(sizeof(*rx_mgmt_event));
+	rx_mgmt_event = qdf_mem_malloc_atomic(sizeof(*rx_mgmt_event));
 	if (!rx_mgmt_event) {
-		p2p_err("Failed to allocate rx mgmt event");
+		p2p_debug_rl("Failed to allocate rx mgmt event");
 		qdf_nbuf_free(buf);
 		return QDF_STATUS_E_NOMEM;
 	}
 
-	rx_mgmt = qdf_mem_malloc(sizeof(*rx_mgmt) +
+	rx_mgmt = qdf_mem_malloc_atomic(sizeof(*rx_mgmt) +
 			mgmt_rx_params->buf_len);
 	if (!rx_mgmt) {
-		p2p_err("Failed to allocate rx mgmt frame");
+		p2p_debug_rl("Failed to allocate rx mgmt frame");
 		qdf_nbuf_free(buf);
 		return QDF_STATUS_E_NOMEM;
 	}

+ 2 - 2
umac/scan/dispatcher/src/wlan_scan_tgt_api.c

@@ -272,14 +272,14 @@ QDF_STATUS tgt_scan_bcn_probe_rx_callback(struct wlan_objmgr_psoc *psoc,
 	bcn = qdf_mem_malloc_atomic(sizeof(*bcn));
 
 	if (!bcn) {
-		scm_err("Failed to allocate memory for bcn");
+		scm_debug_rl("Failed to allocate memory for bcn");
 		status = QDF_STATUS_E_NOMEM;
 		goto free;
 	}
 	bcn->rx_data =
 		qdf_mem_malloc_atomic(sizeof(*rx_param));
 	if (!bcn->rx_data) {
-		scm_err("Failed to allocate memory for rx_data");
+		scm_debug_rl("Failed to allocate memory for rx_data");
 		status = QDF_STATUS_E_NOMEM;
 		goto free;
 	}

+ 3 - 0
umac/tdls/core/src/wlan_tdls_main.h

@@ -59,6 +59,9 @@
 
 #define tdls_debug(params...) \
 	QDF_TRACE_DEBUG(QDF_MODULE_ID_TDLS, params)
+#define tdls_debug_rl(params...) \
+	QDF_TRACE_DEBUG_RL(QDF_MODULE_ID_TDLS, params)
+
 #define tdls_notice(params...) \
 	QDF_TRACE_INFO(QDF_MODULE_ID_TDLS, params)
 #define tdls_warn(params...) \

+ 4 - 4
umac/tdls/dispatcher/src/wlan_tdls_tgt_api.c

@@ -289,16 +289,16 @@ QDF_STATUS tgt_tdls_mgmt_frame_process_rx_cb(
 		vdev_id = wlan_vdev_get_id(vdev);
 	}
 
-	rx_mgmt_event = qdf_mem_malloc(sizeof(*rx_mgmt_event));
+	rx_mgmt_event = qdf_mem_malloc_atomic(sizeof(*rx_mgmt_event));
 	if (!rx_mgmt_event) {
-		tdls_err("Failed to allocate rx mgmt event");
+		tdls_debug_rl("Failed to allocate rx mgmt event");
 		return QDF_STATUS_E_NOMEM;
 	}
 
-	rx_mgmt = qdf_mem_malloc(sizeof(*rx_mgmt) +
+	rx_mgmt = qdf_mem_malloc_atomic(sizeof(*rx_mgmt) +
 			mgmt_rx_params->buf_len);
 	if (!rx_mgmt) {
-		tdls_err("Failed to allocate rx mgmt frame");
+		tdls_debug_rl("Failed to allocate rx mgmt frame");
 		qdf_mem_free(rx_mgmt_event);
 		return QDF_STATUS_E_NOMEM;
 	}