On Wed, Mar 02, 2016 at 03:00:14PM +0100, Tomeu Vizoso wrote:
> @@ -1006,8 +1019,9 @@ static cairo_surface_t *get_cairo_surface(int fd, 
> struct igt_fb *fb)
>                       create_cairo_surface__gtt(fd, fb);
>       }
>  
> -     gem_set_domain(fd, fb->gem_handle,
> -                    I915_GEM_DOMAIN_CPU, I915_GEM_DOMAIN_CPU);
> +     if (!fb->is_dumb)
> +             gem_set_domain(fd, fb->gem_handle, I915_GEM_DOMAIN_CPU,
> +                            I915_GEM_DOMAIN_CPU);

At the risk of opening a can-of-worms, what is the synchronisation
protocol for dumb buffers? Even CPU access to a dumb needs set-domain on
Intel.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to