From: Harry Wentland Date: Wed, 29 Mar 2017 11:25:24 -0400 Subject: drm/amd/display: Add validate_context to atomic_state Git-commit: bf8aef2ec439673cd43ef6d0edace96aa36c5e7c Patch-mainline: v4.15-rc1 References: FATE#326289 FATE#326079 FATE#326049 FATE#322398 FATE#326166 Signed-off-by: Harry Wentland Reviewed-by: Tony Cheng Acked-by: Harry Wentland Signed-off-by: Alex Deucher Acked-by: Petr Tesarik --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c | 18 +--------------- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.h | 2 - 2 files changed, 3 insertions(+), 17 deletions(-) --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c @@ -2981,7 +2981,6 @@ int amdgpu_dm_atomic_check(struct drm_de struct amdgpu_device *adev = dev->dev_private; struct dc *dc = adev->dm.dc; bool need_to_validate = false; - struct validate_context *context; struct drm_connector *connector; struct drm_connector_state *conn_state; /* @@ -3216,10 +3215,9 @@ int amdgpu_dm_atomic_check(struct drm_de } } - context = dc_get_validate_context(dc, dm_state->set, dm_state->set_count); - - if (need_to_validate == false || dm_state->set_count == 0 || context) { + dm_state->context = dc_get_validate_context(dc, dm_state->set, dm_state->set_count); + if (need_to_validate == false || dm_state->set_count == 0 || dm_state->context) { ret = 0; /* * For full updates case when @@ -3235,18 +3233,6 @@ int amdgpu_dm_atomic_check(struct drm_de } - if (context) { - dc_resource_validate_ctx_destruct(context); - dm_free(context); - } - - for (i = 0; i < dm_state->set_count; i++) - for (j = 0; j < dm_state->set[i].surface_count; j++) - dc_surface_release(dm_state->set[i].surfaces[j]); - - for (i = 0; i < new_stream_count; i++) - dc_stream_release(new_streams[i]); - if (ret != 0) { if (ret == -EDEADLK) DRM_DEBUG_KMS("Atomic check stopped due to to deadlock.\n"); --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.h +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.h @@ -55,7 +55,7 @@ struct dm_atomic_state { struct dc_validation_set set[MAX_STREAMS]; int set_count; - + struct validate_context *context; }; #define to_dm_atomic_state(x) container_of(x, struct dm_atomic_state, base)