Add the message from your reply. On ke, 2017-02-22 at 09:41 +0000, Chris Wilson wrote: > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> <SNIP> > @@ -1613,23 +1613,16 @@ i915_gem_sw_finish_ioctl(struct drm_device *dev, void *data, > { > struct drm_i915_gem_sw_finish *args = data; > struct drm_i915_gem_object *obj; > - int err = 0; > > obj = i915_gem_object_lookup(file, args->handle); > if (!obj) > return -ENOENT; > > /* Pinned buffers may be scanout, so flush the cache */ > - if (READ_ONCE(obj->pin_display)) { > - err = i915_mutex_lock_interruptible(dev); > - if (!err) { > - i915_gem_object_flush_cpu_write_domain(obj); > - mutex_unlock(&dev->struct_mutex); > - } > - } > - > + i915_gem_object_flush_to_display(obj); .._flush_if_display() and this is Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx