[Intel-gfx] [PATCH] drm/i915: optimize vblank waits in set_base_atomic

Chris Wilson chris at chris-wilson.co.uk
Mon Jun 3 09:49:13 CEST 2013


On Sun, Jun 02, 2013 at 05:23:01PM +0200, Daniel Vetter wrote:
> We only need to do them if the pipe is actually running and if the
> framebuffers have changed. Removes two "wait for vblank timed out"
> messages when doing a suspend/resume cycle on my i855gm.
> 
> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>

Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>

Comments inline.
> ---
>  drivers/gpu/drm/i915/intel_display.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index 0b27c57..0669729 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -2203,7 +2203,8 @@ intel_pipe_set_base(struct drm_crtc *crtc, int x, int y,
>  	crtc->y = y;
>  
>  	if (old_fb) {
> -		intel_wait_for_vblank(dev, intel_crtc->pipe);
> +		if (to_intel_crtc(crtc)->active && old_fb != fb)
                if (intel_crtc->active && old_fb != fb)

> +			intel_wait_for_vblank(dev, intel_crtc->pipe);
>  		intel_unpin_fb_obj(to_intel_framebuffer(old_fb)->obj);
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list