[PATCH v6 03/11] drm/i915: return EACCES for check_cmd() failures

[Chris Wilson]

On Thu, Oct 20, 2016 at 10:19:02PM +0100, Robert Bragg wrote:
> @@ -1333,6 +1333,9 @@ int i915_cmd_parser_get_version(struct drm_i915_private *dev_priv)
>  	 * 5. GPGPU dispatch compute indirect registers.
>  	 * 6. TIMESTAMP register and Haswell CS GPR registers
>  	 * 7. Allow MI_LOAD_REGISTER_REG between whitelisted registers.
> +	 * 8. Don't report cmd_check() failures as EINVAL errors to userspace;
> +	 *    rely on the HW to NOOP disallowed commands as it would without
> +	 *    the parser enabled.

I added a test case to exercise this and at least for OACONTROL we are
fine. That test can supercede the negative BAT in gem_exec_parse.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre