Hi Daniel, we've merged the kernel change for this but not the test. I'm
assuming we still want the test case.
Brad
On Thu, Mar 27, 2014 at 11:44:45AM -0700, Volkin, Bradley D wrote:
> From: Brad Volkin <bradley.d.volkin@xxxxxxxxx>
>
> Signed-off-by: Brad Volkin <bradley.d.volkin@xxxxxxxxx>
> ---
> tests/gem_exec_parse.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 48 insertions(+)
>
> diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c
> index 34d097d..853eb57 100644
> --- a/tests/gem_exec_parse.c
> +++ b/tests/gem_exec_parse.c
> @@ -204,6 +204,8 @@ int fd;
> #define PIPE_CONTROL_QW_WRITE (1<<14)
> #define PIPE_CONTROL_LRI_POST_OP (1<<23)
>
> +#define OACONTROL 0x2360
> +
> igt_main
> {
> igt_fixture {
> @@ -337,6 +339,52 @@ igt_main
> 0));
> }
>
> + igt_subtest("oacontrol-tracking") {
> + uint32_t lri_ok[] = {
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x0,
> + MI_BATCH_BUFFER_END,
> + 0
> + };
> + uint32_t lri_bad[] = {
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_BATCH_BUFFER_END,
> + };
> + uint32_t lri_extra_bad[] = {
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x0,
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_BATCH_BUFFER_END,
> + };
> + igt_assert(
> + exec_batch(fd, handle,
> + lri_ok, sizeof(lri_ok),
> + I915_EXEC_RENDER,
> + 0));
> + igt_assert(
> + exec_batch(fd, handle,
> + lri_bad, sizeof(lri_bad),
> + I915_EXEC_RENDER,
> + -EINVAL));
> + igt_assert(
> + exec_batch(fd, handle,
> + lri_extra_bad, sizeof(lri_extra_bad),
> + I915_EXEC_RENDER,
> + -EINVAL));
> + }
> +
> igt_fixture {
> gem_close(fd, handle);
>
> --
> 1.8.3.2
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
