This way we'll catch bugs in our code where we depend upon pipe config state not (yet) read out much quicker, through a simple dpms on/off cycle.
This will blow up all over the place for now, hence just a quick idea to toss out there. Signed-off-by: Daniel Vetter <[email protected]> --- drivers/gpu/drm/i915/intel_display.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 7ec8b488bb1d..342915f149b8 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -9287,6 +9287,9 @@ check_crtc_state(struct drm_device *dev) intel_dump_pipe_config(crtc, &crtc->config, "[sw state]"); } + + /* Make sure our code only depends upon stuff we read back. */ + memcpy(&pipe_config, &crtc->config, sizeof(pipe_config)); } } -- 1.8.4.3 _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
