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

Commit 019688fd authored by Yashwanth's avatar Yashwanth Committed by Gerrit - the friendly Code Review server
Browse files

disp: msm: sde: add secure session flag in sde_crtc state 



This change adds a flag in sde_crtc_state to identify the
ongoing secure session. Following are the changes included
as part of this:

1) Whenever there is a direct transition from secure UI to
secure camera(&viceversa).
2) In video mode, when null commit is not present for
transition between secure camera & secure video.
3) This change also adds required wait while transitioning
from secure video session to secure camera in cmd mode.

Change-Id: If9a70fa04b70b03315efe8db1a283faa6d32055b
Signed-off-by: default avatarYashwanth <yvulapu@codeaurora.org>
parent 213d0449

msm/sde/sde_crtc.c

+92 −14
Original line number Diff line number Diff line
@@ -1703,8 +1703,12 @@ int sde_crtc_state_find_plane_fb_modes(struct drm_crtc_state *state, 


static void _sde_drm_fb_sec_dir_trans(

	struct sde_kms_smmu_state_data *smmu_state, uint32_t secure_level,

	struct sde_mdss_cfg *catalog, bool old_valid_fb, int *ops)

	struct sde_mdss_cfg *catalog, bool old_valid_fb, int *ops,

	struct drm_crtc_state *old_crtc_state)

{

	struct sde_crtc_state *old_cstate = to_sde_crtc_state(old_crtc_state);

	int old_secure_session = old_cstate->secure_session;



	/* secure display usecase */

	if ((smmu_state->state == ATTACHED)

			&& (secure_level == SDE_DRM_SEC_ONLY)) {
@@ -1725,6 +1729,10 @@ static void _sde_drm_fb_sec_dir_trans( 
		smmu_state->secure_level = secure_level;

		smmu_state->transition_type = PRE_COMMIT;

		*ops |= SDE_KMS_OPS_SECURE_STATE_CHANGE;

		if (old_secure_session ==

			SDE_SECURE_VIDEO_SESSION)

			*ops |= (SDE_KMS_OPS_WAIT_FOR_TX_DONE  |

					SDE_KMS_OPS_CLEANUP_PLANE_FB);

	}

}
@@ -1850,7 +1858,7 @@ int sde_crtc_get_secure_transition_ops(struct drm_crtc *crtc, 
	switch (translation_mode) {

	case SDE_DRM_FB_SEC_DIR_TRANS:

		_sde_drm_fb_sec_dir_trans(smmu_state, secure_level,

				catalog, old_valid_fb, &ops);

			catalog, old_valid_fb, &ops, old_crtc_state);

		if (clone_mode && (ops & SDE_KMS_OPS_SECURE_STATE_CHANGE))

			ops |= SDE_KMS_OPS_WAIT_FOR_TX_DONE;

		break;
@@ -4319,6 +4327,55 @@ static int _sde_crtc_check_secure_single_encoder(struct drm_crtc *crtc, 
	return 0;

}



static int _sde_crtc_check_secure_transition(struct drm_crtc *crtc,

	struct drm_crtc_state *state, bool is_video_mode)

{

	struct sde_crtc_state *old_cstate = to_sde_crtc_state(crtc->state);

	struct sde_crtc_state *new_cstate = to_sde_crtc_state(state);

	int old_secure_session = old_cstate->secure_session;

	int new_secure_session = new_cstate->secure_session;

	int ret = 0;



	/*

	 * Direct transition from Secure Camera to Secure UI(&viceversa)

	 * is not allowed

	 */

	if ((old_secure_session == SDE_SECURE_CAMERA_SESSION &&

			new_secure_session == SDE_SECURE_UI_SESSION) ||

		(old_secure_session == SDE_SECURE_UI_SESSION &&

			new_secure_session == SDE_SECURE_CAMERA_SESSION)) {

		SDE_EVT32(DRMID(crtc), old_secure_session,

			new_secure_session, SDE_EVTLOG_ERROR);

		ret = -EINVAL;

	}



	/*

	 * In video mode, null commit is required for transition between

	 * secure video & secure camera

	 */

	if (is_video_mode &&

		((old_secure_session == SDE_SECURE_CAMERA_SESSION &&

			new_secure_session == SDE_SECURE_VIDEO_SESSION) ||

		(old_secure_session == SDE_SECURE_VIDEO_SESSION &&

			new_secure_session == SDE_SECURE_CAMERA_SESSION))) {

		SDE_EVT32(DRMID(crtc), old_secure_session,

			new_secure_session, SDE_EVTLOG_ERROR);

		ret = -EINVAL;

	}



	if (old_secure_session != new_secure_session)

		SDE_EVT32(DRMID(crtc), old_secure_session,

						new_secure_session);



	SDE_DEBUG("old session: %d new session : %d\n",

			old_secure_session, new_secure_session);

	if (ret)

		SDE_ERROR("invalid transition old:%d new:%d\n",

			old_secure_session, new_secure_session);



	return ret;

}



static int _sde_crtc_check_secure_state_smmu_translation(struct drm_crtc *crtc,

	struct drm_crtc_state *state, struct sde_kms *sde_kms, int secure,

	int fb_ns, int fb_sec, int fb_sec_dir)
@@ -4333,19 +4390,8 @@ static int _sde_crtc_check_secure_state_smmu_translation(struct drm_crtc *crtc, 
						MSM_DISPLAY_VIDEO_MODE);

	}



	/*

	 * Secure display to secure camera needs without direct

	 * transition is currently not allowed

	 */

	if (fb_sec_dir && secure == SDE_DRM_SEC_NON_SEC &&

		smmu_state->state != ATTACHED &&

		smmu_state->secure_level == SDE_DRM_SEC_ONLY) {



		SDE_EVT32(DRMID(crtc), fb_ns, fb_sec_dir,

			smmu_state->state, smmu_state->secure_level,

			secure);

	if (_sde_crtc_check_secure_transition(crtc, state, is_video_mode))

		goto sec_err;

	}



	/*

	 * In video mode check for null commit before transition
@@ -4411,6 +4457,33 @@ static int _sde_crtc_check_secure_conn(struct drm_crtc *crtc, 
	return 0;

}



static int _sde_crtc_populate_secure_session(struct drm_crtc_state *state,

	int secure, int fb_ns, int fb_sec, int fb_sec_dir)

{

	struct sde_crtc_state *cstate = to_sde_crtc_state(state);



	if (secure == SDE_DRM_SEC_ONLY && fb_sec_dir && !fb_sec && !fb_ns)

		cstate->secure_session = SDE_SECURE_UI_SESSION;

	else if (secure == SDE_DRM_SEC_NON_SEC && fb_sec_dir && !fb_sec)

		cstate->secure_session = SDE_SECURE_CAMERA_SESSION;

	else if (secure == SDE_DRM_SEC_NON_SEC && !fb_sec_dir && fb_sec)

		cstate->secure_session = SDE_SECURE_VIDEO_SESSION;

	else if (secure == SDE_DRM_SEC_NON_SEC && !fb_sec_dir &&

			!fb_sec && fb_ns)

		cstate->secure_session = SDE_NON_SECURE_SESSION;

	else if (!fb_sec_dir && !fb_sec && !fb_ns)

		cstate->secure_session = SDE_NULL_SESSION;

	else {

		SDE_ERROR(

			"invalid session sec:%d fb_sec_dir:%d fb_sec:%d fb_ns:%d\n",

				cstate->secure_session, fb_sec_dir,

				fb_sec, fb_ns);

		return -EINVAL;

	}



	return 0;

}



static int _sde_crtc_check_secure_state(struct drm_crtc *crtc,

		struct drm_crtc_state *state, struct plane_state pstates[],

		int cnt)
@@ -4441,6 +4514,11 @@ static int _sde_crtc_check_secure_state(struct drm_crtc *crtc, 
	if (rc)

		return rc;



	rc = _sde_crtc_populate_secure_session(state, secure,

				fb_ns, fb_sec, fb_sec_dir);

	if (rc)

		return rc;



	rc = _sde_crtc_check_secure_blend_config(crtc, state, pstates, cstate,

			sde_kms, cnt, secure, fb_ns, fb_sec, fb_sec_dir);

	if (rc)

msm/sde/sde_crtc.h

+18 −0
Original line number Diff line number Diff line
@@ -35,6 +35,22 @@ 
/* Expand it to 2x for handling atleast 2 connectors safely */

#define SDE_CRTC_FRAME_EVENT_SIZE	(4 * 2)



/**

 * enum sde_session_type: session type

 * @SDE_SECURE_UI_SESSION:     secure UI usecase

 * @SDE_SECURE_CAMERA_SESSION: secure camera usecase

 * @SDE_SECURE_VIDEO_SESSION:  secure video usecase

 * @SDE_NON_SECURE_SESSION:    non secure usecase

 * @SDE_NULL_SESSION:          null commit usecase

 */

enum sde_session_type {

	SDE_SECURE_UI_SESSION,

	SDE_SECURE_CAMERA_SESSION,

	SDE_SECURE_VIDEO_SESSION,

	SDE_NON_SECURE_SESSION,

	SDE_NULL_SESSION,

};



/**

 * enum sde_crtc_client_type: crtc client type

 * @RT_CLIENT:	RealTime client like video/cmd mode display
@@ -380,6 +396,7 @@ struct sde_crtc { 
 * @ds_cfg: Destination scaler config

 * @scl3_lut_cfg: QSEED3 lut config

 * @new_perf: new performance state being requested

 * @secure_session: Indicates the type of secure session

 */

struct sde_crtc_state {

	struct drm_crtc_state base;
@@ -409,6 +426,7 @@ struct sde_crtc_state { 
	struct sde_hw_scaler3_lut_cfg scl3_lut_cfg;



	struct sde_core_perf_params new_perf;

	int secure_session;

};



enum sde_crtc_irq_state {