Thread (4 messages) 4 messages, 3 authors, 2021-07-12

Re: [igt-dev] [PATCH i-g-t] tests/i915/gem_ctx_param: Test that you can only set the VM once

From: Daniel Vetter <hidden>
Date: 2021-07-12 15:28:40

On Sat, Jul 10, 2021 at 11:19 PM Jason Ekstrand [off-list ref] wrote:
We're about to add this restriction to i915 so we should test for it
explicitly.

Signed-off-by: Jason Ekstrand <redacted>
Reviewed-by: Daniel Vetter <redacted>
quoted hunk ↗ jump to hunk
---
 tests/i915/gem_ctx_param.c | 9 +++++++++
 1 file changed, 9 insertions(+)
diff --git a/tests/i915/gem_ctx_param.c b/tests/i915/gem_ctx_param.c
index c795f1b45..3b79a3e5e 100644
--- a/tests/i915/gem_ctx_param.c
+++ b/tests/i915/gem_ctx_param.c
@@ -188,6 +188,15 @@ static void test_vm(int i915)
        gem_vm_destroy(i915, arg.value);
        igt_assert_eq(err, -EINVAL);

+       /* Test that we can't set the VM twice */
+       arg.ctx_id = gem_context_create(i915);
+       arg.value = gem_vm_create(i915);
+       gem_context_set_param(i915, &arg);
+       err = __gem_context_set_param(i915, &arg);
+       gem_vm_destroy(i915, arg.value);
+       gem_context_destroy(i915, arg.ctx_id);
+       igt_assert_eq(err, -EINVAL);
+
        /* Test that we can't set the VM after we've done an execbuf */
        arg.ctx_id = gem_context_create(i915);
        spin = igt_spin_new(i915, .ctx_id = arg.ctx_id);
--
2.31.1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev


-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help