Unverified Commit e4a74493 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!15484 drm/msm/mdp5: Return error code in mdp5_mixer_release when deadlock is detected 

Merge Pull Request from: @ci-robot 
 
PR sync from: Liu Shixin <liushixin2@huawei.com>
https://mailweb.openeuler.org/archives/list/kernel@openeuler.org/message/UBGIRC4RYIBTCVQ27JJRGAMMD74JV7DQ/ 
 
https://gitee.com/src-openeuler/kernel/issues/IBP48W 
 
Link:https://gitee.com/openeuler/kernel/pulls/15484

 

Reviewed-by: default avatarYuan Can <yuancan@huawei.com>
Signed-off-by: default avatarYuan Can <yuancan@huawei.com>
parents 9b569f9f f8170d68

drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c

+8 −2
Original line number Diff line number Diff line
@@ -537,9 +537,15 @@ int mdp5_crtc_setup_pipeline(struct drm_crtc *crtc, 
		if (ret)

			return ret;



		mdp5_mixer_release(new_crtc_state->state, old_mixer);

		ret = mdp5_mixer_release(new_crtc_state->state, old_mixer);

		if (ret)

			return ret;



		if (old_r_mixer) {

			mdp5_mixer_release(new_crtc_state->state, old_r_mixer);

			ret = mdp5_mixer_release(new_crtc_state->state, old_r_mixer);

			if (ret)

				return ret;



			if (!need_right_mixer)

				pipeline->r_mixer = NULL;

		}

drivers/gpu/drm/msm/disp/mdp5/mdp5_mixer.c

+11 −4
Original line number Diff line number Diff line
@@ -127,21 +127,28 @@ int mdp5_mixer_assign(struct drm_atomic_state *s, struct drm_crtc *crtc, 
	return 0;

}



void mdp5_mixer_release(struct drm_atomic_state *s, struct mdp5_hw_mixer *mixer)

int mdp5_mixer_release(struct drm_atomic_state *s, struct mdp5_hw_mixer *mixer)

{

	struct mdp5_global_state *global_state = mdp5_get_global_state(s);

	struct mdp5_hw_mixer_state *new_state = &global_state->hwmixer;

	struct mdp5_hw_mixer_state *new_state;



	if (!mixer)

		return;

		return 0;



	if (IS_ERR(global_state))

		return PTR_ERR(global_state);



	new_state = &global_state->hwmixer;



	if (WARN_ON(!new_state->hwmixer_to_crtc[mixer->idx]))

		return;

		return -EINVAL;



	DBG("%s: release from crtc %s", mixer->name,

	    new_state->hwmixer_to_crtc[mixer->idx]->name);



	new_state->hwmixer_to_crtc[mixer->idx] = NULL;



	return 0;

}



void mdp5_mixer_destroy(struct mdp5_hw_mixer *mixer)

drivers/gpu/drm/msm/disp/mdp5/mdp5_mixer.h

+2 −2
Original line number Diff line number Diff line
@@ -41,7 +41,7 @@ void mdp5_mixer_destroy(struct mdp5_hw_mixer *lm); 
int mdp5_mixer_assign(struct drm_atomic_state *s, struct drm_crtc *crtc,

		      uint32_t caps, struct mdp5_hw_mixer **mixer,

		      struct mdp5_hw_mixer **r_mixer);

void mdp5_mixer_release(struct drm_atomic_state *s,

int mdp5_mixer_release(struct drm_atomic_state *s,

		       struct mdp5_hw_mixer *mixer);



#endif /* __MDP5_LM_H__ */