| Message ID | 1401473122-3451-2-git-send-email-jbarnes@virtuousgeek.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 8c52038..74310b5 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -10545,10 +10545,6 @@ static int intel_set_mode(struct drm_crtc *crtc, ret = __intel_set_mode(crtc, mode, x, y, fb); - intel_sync_crtcs(crtc->dev->dev_private); - if (ret == 0) - intel_modeset_check_state(crtc->dev); - return ret; }
Now that we can queue CRTC enable/disable calls for later, we can allow userspace mode sets to return immediately. This may mean that userspace will draw into a buffer that's not yet displayed (which is fine) or that it may draw into a buffer it thinks is no longer displayed (which could lead to some visual artifacts until the mode set completes, but is otherwise harmless). Page flip and cursor activity will synchronize with any outstanding activity to avoid problems with the display being off for those operations. It should be possible to queue those ops as well though and further de-couple driver updates of the hw state from userspace queueing of commands. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org> --- drivers/gpu/drm/i915/intel_display.c | 4 ---- 1 file changed, 4 deletions(-)