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

Commit 928edd93 authored by Gaurav Jindal's avatar Gaurav Jindal Committed by Gerrit - the friendly Code Review server
Browse files

msm: camera: jpeg: Add mutex lock to protect jpeg list corruption 



Due to race conditions, situation can arise where the process_irq
and flush for jpeg are being handled in parallel. This will cause
the jpeg list corruption.
This commit protects the code of adding back to free list
in process_irq with the mutex.

CRs-Fixed: 2578247
Change-Id: I28ee48bc0d5cfcf3ae4a936b2eb2976226ad88d5
Signed-off-by: default avatarGaurav Jindal <gjindal@codeaurora.org>
parent 911c7152

drivers/cam_jpeg/jpeg_hw/cam_jpeg_hw_mgr.c

+2 −0
Original line number Diff line number Diff line
@@ -177,7 +177,9 @@ static int cam_jpeg_mgr_process_irq(void *priv, void *data) 
		PTR_TO_U64(p_cfg_req->hw_cfg_args.priv);

	ctx_data->ctxt_event_cb(ctx_data->context_priv, 0, &buf_data);



	mutex_lock(&g_jpeg_hw_mgr.hw_mgr_mutex);

	list_add_tail(&p_cfg_req->list, &hw_mgr->free_req_list);

	mutex_unlock(&g_jpeg_hw_mgr.hw_mgr_mutex);

	return rc;

}