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

Reply via email to