[Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

[Chris Wilson]

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