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

Commit 89029271 authored by Maheshwar Ajja's avatar Maheshwar Ajja
Browse files

msm: vidc: Don't kill the session in response handler thread 



Kill session will call abort to video hardware and waits for
abort response. So calling kill session from within response
handler thread would always results in wait timeout which in
turn causes session error manipulated to sys error. So don't
kill the session but send error client and client will close
the session.

CRs-Fixed: 2071607
Change-Id: Ic7b57724d57eaf84e90512a732fd9cfb05281b1b
Signed-off-by: default avatarMaheshwar Ajja <majja@codeaurora.org>
parent aebe5ecd

drivers/media/platform/msm/vidc/msm_vidc_common.c

+0 −2
Original line number Diff line number Diff line
@@ -5076,7 +5076,6 @@ int msm_vidc_check_session_supported(struct msm_vidc_inst *inst) 
	rc = msm_vidc_load_supported(inst);

	if (rc) {

		change_inst_state(inst, MSM_VIDC_CORE_INVALID);

		msm_comm_kill_session(inst);

		dprintk(VIDC_WARN,

			"%s: Hardware is overloaded\n", __func__);

		return rc;
@@ -5126,7 +5125,6 @@ int msm_vidc_check_session_supported(struct msm_vidc_inst *inst) 
	}

	if (rc) {

		change_inst_state(inst, MSM_VIDC_CORE_INVALID);

		msm_comm_kill_session(inst);

		dprintk(VIDC_ERR,

			"%s: Resolution unsupported\n", __func__);

	}