On Tue, Apr 7, 2015 at 6:28 PM, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
> /* Pinned buffers may be scanout, so flush the cache */
> - if (obj->pin_display)
> + if (obj->pin_display) {
> + ret = i915_mutex_lock_interruptible(dev);
> + if (ret)
> + goto unref;
I think and ACCESS_ONCE here and in the previous one would be good.
Wanted to do that and merge both, but they seem to conflict with the
lack of read-read ...
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
