Commit b04c21ab authored by Wenjing Liu's avatar Wenjing Liu Committed by Alex Deucher

drm/amd/display: skip forcing odm in minimal transition

[why]
In minial transitions state, ODM combine shouldn't be forced as it will
make transition non seamless. The force ODM debug option is to control
the end result not the intermediate transition. So we can temporarily
disable ODM forcing when committing minimal transition state.

[how]
Backup stream ODM forcing option and clear it in minimal transition
state. Once minimal transition state is released, we will restore the
original debug option back.
Reviewed-by: default avatarSamson Tam <samson.tam@amd.com>
Acked-by: default avatarWayne Lin <wayne.lin@amd.com>
Signed-off-by: default avatarWenjing Liu <wenjing.liu@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 0ba7ad7e
...@@ -4158,55 +4158,88 @@ struct pipe_split_policy_backup { ...@@ -4158,55 +4158,88 @@ struct pipe_split_policy_backup {
bool dynamic_odm_policy; bool dynamic_odm_policy;
bool subvp_policy; bool subvp_policy;
enum pipe_split_policy mpc_policy; enum pipe_split_policy mpc_policy;
char force_odm[MAX_PIPES];
}; };
static void release_minimal_transition_state(struct dc *dc, static void backup_and_set_minimal_pipe_split_policy(struct dc *dc,
struct dc_state *context, struct pipe_split_policy_backup *policy) struct dc_state *context,
struct pipe_split_policy_backup *policy)
{ {
dc_state_release(context); int i;
/* restore previous pipe split and odm policy */
if (!dc->config.is_vmin_only_asic) {
policy->mpc_policy = dc->debug.pipe_split_policy;
dc->debug.pipe_split_policy = MPC_SPLIT_AVOID;
}
policy->dynamic_odm_policy = dc->debug.enable_single_display_2to1_odm_policy;
dc->debug.enable_single_display_2to1_odm_policy = false;
policy->subvp_policy = dc->debug.force_disable_subvp;
dc->debug.force_disable_subvp = true;
for (i = 0; i < context->stream_count; i++) {
policy->force_odm[i] = context->streams[i]->debug.force_odm_combine_segments;
context->streams[i]->debug.force_odm_combine_segments = 0;
}
}
static void restore_minimal_pipe_split_policy(struct dc *dc,
struct dc_state *context,
struct pipe_split_policy_backup *policy)
{
uint8_t i;
if (!dc->config.is_vmin_only_asic) if (!dc->config.is_vmin_only_asic)
dc->debug.pipe_split_policy = policy->mpc_policy; dc->debug.pipe_split_policy = policy->mpc_policy;
dc->debug.enable_single_display_2to1_odm_policy = policy->dynamic_odm_policy; dc->debug.enable_single_display_2to1_odm_policy =
policy->dynamic_odm_policy;
dc->debug.force_disable_subvp = policy->subvp_policy; dc->debug.force_disable_subvp = policy->subvp_policy;
for (i = 0; i < context->stream_count; i++)
context->streams[i]->debug.force_odm_combine_segments = policy->force_odm[i];
}
static void release_minimal_transition_state(struct dc *dc,
struct dc_state *minimal_transition_context,
struct dc_state *base_context,
struct pipe_split_policy_backup *policy)
{
restore_minimal_pipe_split_policy(dc, base_context, policy);
dc_state_release(minimal_transition_context);
/* restore previous pipe split and odm policy */
}
static void force_vsync_flip_in_minimal_transition_context(struct dc_state *context)
{
uint8_t i;
int j;
struct dc_stream_status *stream_status;
for (i = 0; i < context->stream_count; i++) {
stream_status = &context->stream_status[i];
for (j = 0; j < stream_status->plane_count; j++)
stream_status->plane_states[j]->flip_immediate = false;
}
} }
static struct dc_state *create_minimal_transition_state(struct dc *dc, static struct dc_state *create_minimal_transition_state(struct dc *dc,
struct dc_state *base_context, struct pipe_split_policy_backup *policy) struct dc_state *base_context, struct pipe_split_policy_backup *policy)
{ {
struct dc_state *minimal_transition_context = NULL; struct dc_state *minimal_transition_context = NULL;
unsigned int i, j;
minimal_transition_context = dc_state_create_copy(base_context); minimal_transition_context = dc_state_create_copy(base_context);
if (!minimal_transition_context) if (!minimal_transition_context)
return NULL; return NULL;
if (!dc->config.is_vmin_only_asic) { backup_and_set_minimal_pipe_split_policy(dc, base_context, policy);
policy->mpc_policy = dc->debug.pipe_split_policy;
dc->debug.pipe_split_policy = MPC_SPLIT_AVOID;
}
policy->dynamic_odm_policy = dc->debug.enable_single_display_2to1_odm_policy;
dc->debug.enable_single_display_2to1_odm_policy = false;
policy->subvp_policy = dc->debug.force_disable_subvp;
dc->debug.force_disable_subvp = true;
/* commit minimal state */ /* commit minimal state */
if (dc->res_pool->funcs->validate_bandwidth(dc, minimal_transition_context, false)) { if (dc->res_pool->funcs->validate_bandwidth(dc, minimal_transition_context, false)) {
for (i = 0; i < minimal_transition_context->stream_count; i++) { /* prevent underflow and corruption when reconfiguring pipes */
struct dc_stream_status *stream_status = &minimal_transition_context->stream_status[i]; force_vsync_flip_in_minimal_transition_context(minimal_transition_context);
for (j = 0; j < stream_status->plane_count; j++) {
struct dc_plane_state *plane_state = stream_status->plane_states[j];
/* force vsync flip when reconfiguring pipes to prevent underflow
* and corruption
*/
plane_state->flip_immediate = false;
}
}
} else { } else {
/* this should never happen */ /*
release_minimal_transition_state(dc, minimal_transition_context, policy); * This should never happen, minimal transition state should
* always be validated first before adding pipe split features.
*/
release_minimal_transition_state(dc, minimal_transition_context, base_context, policy);
BREAK_TO_DEBUGGER(); BREAK_TO_DEBUGGER();
minimal_transition_context = NULL; minimal_transition_context = NULL;
} }
...@@ -4293,7 +4326,7 @@ static bool commit_minimal_transition_based_on_new_context(struct dc *dc, ...@@ -4293,7 +4326,7 @@ static bool commit_minimal_transition_based_on_new_context(struct dc *dc,
success = true; success = true;
} }
release_minimal_transition_state( release_minimal_transition_state(
dc, intermediate_context, &policy); dc, intermediate_context, new_context, &policy);
} }
return success; return success;
} }
...@@ -4339,7 +4372,7 @@ static bool commit_minimal_transition_based_on_current_context(struct dc *dc, ...@@ -4339,7 +4372,7 @@ static bool commit_minimal_transition_based_on_current_context(struct dc *dc,
success = true; success = true;
} }
release_minimal_transition_state(dc, intermediate_context, release_minimal_transition_state(dc, intermediate_context,
&policy); dc->current_state, &policy);
} }
/* /*
* Restore stream and plane states back to the values associated with * Restore stream and plane states back to the values associated with
...@@ -4466,7 +4499,7 @@ static bool commit_minimal_transition_state(struct dc *dc, ...@@ -4466,7 +4499,7 @@ static bool commit_minimal_transition_state(struct dc *dc,
transition_base_context, &policy); transition_base_context, &policy);
if (transition_context) { if (transition_context) {
ret = dc_commit_state_no_check(dc, transition_context); ret = dc_commit_state_no_check(dc, transition_context);
release_minimal_transition_state(dc, transition_context, &policy); release_minimal_transition_state(dc, transition_context, transition_base_context, &policy);
} }
if (ret != DC_OK) { if (ret != DC_OK) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment