Commit 512b5527 authored by Maarten Lankhorst's avatar Maarten Lankhorst

drm/i915/gen9+: Preserve old allocation from crtc_state.

This is the last bit required for making nonblocking modesets work
correctly. The state in intel_crtc->hw_ddb is updated in the
nonblocking part of a nonblocking commit.

This means that even attempting a commit before a nonblocking modeset
completes will fail, because intel_crtc->hw_ddb still has stale values.
The stale values are 0 if the crtc is being enabled resulting in a
failure during atomic check, but it may also result in double use of
ddb allocations.

Fix this by explicitly copying the ddb allocation from the old state.
This has to be done explicitly, because a modeset that doesn't change
active pipes, or a modeset converted to a fastset will will clear the
current state.
Signed-off-by: default avatarMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1478609742-13603-4-git-send-email-maarten.lankhorst@linux.intel.com
[mlankhorst: Reword commit message.]
Reviewed-by: default avatarMatt Roper <matthew.d.roper@intel.com>
parent e62929b3
...@@ -14337,7 +14337,7 @@ static void skl_update_crtcs(struct drm_atomic_state *state, ...@@ -14337,7 +14337,7 @@ static void skl_update_crtcs(struct drm_atomic_state *state,
* new ddb allocation to take effect. * new ddb allocation to take effect.
*/ */
if (!skl_ddb_entry_equal(&cstate->wm.skl.ddb, if (!skl_ddb_entry_equal(&cstate->wm.skl.ddb,
&intel_crtc->hw_ddb) && &to_intel_crtc_state(old_crtc_state)->wm.skl.ddb) &&
!crtc->state->active_changed && !crtc->state->active_changed &&
intel_state->wm_results.dirty_pipes != updated) intel_state->wm_results.dirty_pipes != updated)
vbl_wait = true; vbl_wait = true;
......
...@@ -3120,7 +3120,11 @@ skl_ddb_get_pipe_allocation_limits(struct drm_device *dev, ...@@ -3120,7 +3120,11 @@ skl_ddb_get_pipe_allocation_limits(struct drm_device *dev,
* we currently hold. * we currently hold.
*/ */
if (!intel_state->active_pipe_changes) { if (!intel_state->active_pipe_changes) {
*alloc = to_intel_crtc(for_crtc)->hw_ddb; /*
* alloc may be cleared by clear_intel_crtc_state,
* copy from old state to be sure
*/
*alloc = to_intel_crtc_state(for_crtc->state)->wm.skl.ddb;
return; return;
} }
......
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