qcacmn: Revert "qcacmn: Not set "force_set" if event completed"

This reverts commit I31f947169153ccbeb8435c539faab1059c055c04 since
it has introduced, use after free issue during dp_rx_thread shutdown.

Change-Id: If8318b28883ae8ddd4fee36013f28c45f30fa426
CRs-Fixed: 2707451
This commit is contained in:
Aditya Kodukula
2020-06-16 11:35:10 -07:00
committad av nshrivas
förälder 0f08390fa4
incheckning aad7669d56
2 ändrade filer med 2 tillägg och 8 borttagningar

Visa fil

@@ -30,13 +30,11 @@
* qdf_event_t - manages events
* @complete: instance to completion
* @cookie: unsigned int
* @done: indicate completion
* @force_set: indicate forceful completion
*/
typedef struct qdf_evt {
struct completion complete;
uint32_t cookie;
bool done;
bool force_set;
} __qdf_event_t;

Visa fil

@@ -102,7 +102,6 @@ QDF_STATUS qdf_event_set(qdf_event_t *event)
return QDF_STATUS_E_INVAL;
complete(&event->complete);
event->done = true;
return QDF_STATUS_SUCCESS;
}
@@ -135,7 +134,6 @@ QDF_STATUS qdf_event_reset(qdf_event_t *event)
return QDF_STATUS_E_INVAL;
/* (re)initialize event */
event->done = false;
event->force_set = false;
INIT_COMPLETION(event->complete);
@@ -254,10 +252,8 @@ void qdf_complete_wait_events(void)
event_node = qdf_container_of(list_node,
struct qdf_evt_node, node);
if (!event_node->pevent->done) {
event_node->pevent->force_set = true;
qdf_event_set(event_node->pevent);
}
event_node->pevent->force_set = true;
qdf_event_set(event_node->pevent);
status = qdf_list_peek_next(&qdf_wait_event_list,
&event_node->node, &list_node);