Re: [PATCH 14/15] drm/i915: Track frontbuffer invalidation/flushing

Chris Wilson
2014-06-17T06:52:42Z

On Mon, Jun 16, 2014 at 07:51:34PM +0200, Daniel Vetter wrote:
> +static void intel_mark_fb_busy(struct drm_device *dev,
> + unsigned frontbuffer_bits,
> + struct intel_engine_cs *ring)
> {
> - struct drm_device *dev = obj->base.dev;
> enum pipe pipe;
>
> - intel_edp_psr_exit(dev);
> -
> if (!i915.powersave)
> return;

I think this wants to be moved from here down to
intel_increase_pllclock. It's a little dangerous here as we use it
inconsistently, and we certainly want to keep tracking fb bits for other
reasons.
-Chris

--
Chris Wilson, Intel Open Source Technology Centre