Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 3c36d4ac authored by Jingxiang Ge's avatar Jingxiang Ge Committed by Madan Koyyalamudi
Browse files

qcacmn: Limit rx diag event count in each work process 

Current in wmi_rx_diag_event_work, rx diag event is processing
in a while loop, if there is continuous diag event comes from
fw, it is possible that it occupy the work queue, and other work
are not able to run.

Break the while loop so give a chance to other work.

Change-Id: I7af6d60aeb8c0524cc51d663658d5b17349daa60
CRs-Fixed: 2948839
parent e46d37a6

wmi/src/wmi_unified.c

+11 −0
Original line number Diff line number Diff line
@@ -101,6 +101,8 @@ typedef PREPACK struct { 
/* Allocation of size 2048 bytes */

#define WMI_WBUFF_POOL_3_SIZE 8



#define RX_DIAG_EVENT_WORK_PROCESS_MAX_COUNT 500



#ifdef WMI_INTERFACE_EVENT_LOGGING

#if (LINUX_VERSION_CODE >= KERNEL_VERSION(4, 3, 0))

/* TODO Cleanup this backported function */
@@ -2833,6 +2835,7 @@ static void wmi_rx_diag_event_work(void *arg) 
	struct wmi_unified *wmi = arg;

	qdf_timer_t wd_timer;

	struct wmi_wq_dbg_info info;

	uint32_t diag_event_process_count = 0;



	if (!wmi) {

		wmi_err("Invalid WMI handle");
@@ -2853,6 +2856,14 @@ static void wmi_rx_diag_event_work(void *arg) 
		info.task = qdf_get_current_task();

		__wmi_control_rx(wmi, buf);

		qdf_timer_stop(&wd_timer);



		if (diag_event_process_count++ >

		    RX_DIAG_EVENT_WORK_PROCESS_MAX_COUNT) {

			qdf_queue_work(0, wmi->wmi_rx_diag_work_queue,

				       &wmi->rx_diag_event_work);

			break;

		}



		qdf_spin_lock_bh(&wmi->diag_eventq_lock);

		buf = qdf_nbuf_queue_remove(&wmi->diag_event_queue);

		qdf_spin_unlock_bh(&wmi->diag_eventq_lock);