[Beignet] [RFC PATCH 2/3] drm/i915: IOMMU based SVM implementation v16

[Chris Wilson]

On Mon, Jan 09, 2017 at 06:52:53PM +0200, Mika Kuoppala wrote:
> +static int i915_gem_context_enable_svm(struct i915_gem_context *ctx)
> +{
> +	int ret;
> +
> +	if (!HAS_SVM(ctx->i915))
> +		return -ENODEV;

How does legacy execbuf work with an svm context? It will write the
ppgtt, but those are no longer read by the GPU. So it will generate
faults at random addresses. Am I right in thinking we need to EINVAL if
using execbuf + context_is_svm?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre