| Message ID | 20200214194016.4054376-1-chris@chris-wilson.co.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [i-g-t,1/4] i915/gem_ctx_engines: Exercise 0 engines[] | expand |
On 14/02/20 11:40, Chris Wilson wrote: > Setup a context with no engines, and make sure we reject all execution > attempts. > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Antonio Argenziano <antonio.argenziano@intel.com> > --- > tests/i915/gem_ctx_engines.c | 45 ++++++++++++++++++++++++++++++++++++ > 1 file changed, 45 insertions(+) > > diff --git a/tests/i915/gem_ctx_engines.c b/tests/i915/gem_ctx_engines.c > index cb82f08ef..063140e0f 100644 > --- a/tests/i915/gem_ctx_engines.c > +++ b/tests/i915/gem_ctx_engines.c > @@ -242,6 +242,48 @@ static void idempotent(int i915) > gem_context_destroy(i915, p.ctx_id); > } > > +static uint32_t batch_create(int i915) > +{ > + const uint32_t bbe = MI_BATCH_BUFFER_END; > + uint32_t handle = gem_create(i915, 4096); > + > + gem_write(i915, handle, 0, &bbe, sizeof(bbe)); > + return handle; > +} > + > +static void none(int i915) > +{ > + struct i915_context_param_engines engines = {}; > + struct drm_i915_gem_context_param p = { > + .ctx_id = gem_context_create(i915), > + .param = I915_CONTEXT_PARAM_ENGINES, > + .value = to_user_pointer(&engines), > + .size = sizeof(engines), > + }; > + > + gem_context_set_param(i915, &p); > + > + { > + struct drm_i915_gem_exec_object2 obj = { > + .handle = batch_create(i915), > + }; > + struct drm_i915_gem_execbuffer2 execbuf = { > + .buffers_ptr = to_user_pointer(&obj), > + .buffer_count = 1, > + .rsvd1 = p.ctx_id, > + }; > + > + for (execbuf.flags = 0; > + execbuf.flags <= I915_EXEC_RING_MASK; > + execbuf.flags++) > + igt_assert_eq(__gem_execbuf(i915, &execbuf), -EINVAL); > + > + gem_close(i915, obj.handle); > + } > + > + gem_context_destroy(i915, p.ctx_id); > +} > + > static void execute_one(int i915) > { > I915_DEFINE_CONTEXT_PARAM_ENGINES(engines , I915_EXEC_RING_MASK + 1); > @@ -527,6 +569,9 @@ igt_main > igt_subtest("idempotent") > idempotent(i915); > > + igt_subtest("none") > + none(i915); > + > igt_subtest("execute-one") > execute_one(i915); > >
diff --git a/tests/i915/gem_ctx_engines.c b/tests/i915/gem_ctx_engines.c index cb82f08ef..063140e0f 100644 --- a/tests/i915/gem_ctx_engines.c +++ b/tests/i915/gem_ctx_engines.c @@ -242,6 +242,48 @@ static void idempotent(int i915) gem_context_destroy(i915, p.ctx_id); } +static uint32_t batch_create(int i915) +{ + const uint32_t bbe = MI_BATCH_BUFFER_END; + uint32_t handle = gem_create(i915, 4096); + + gem_write(i915, handle, 0, &bbe, sizeof(bbe)); + return handle; +} + +static void none(int i915) +{ + struct i915_context_param_engines engines = {}; + struct drm_i915_gem_context_param p = { + .ctx_id = gem_context_create(i915), + .param = I915_CONTEXT_PARAM_ENGINES, + .value = to_user_pointer(&engines), + .size = sizeof(engines), + }; + + gem_context_set_param(i915, &p); + + { + struct drm_i915_gem_exec_object2 obj = { + .handle = batch_create(i915), + }; + struct drm_i915_gem_execbuffer2 execbuf = { + .buffers_ptr = to_user_pointer(&obj), + .buffer_count = 1, + .rsvd1 = p.ctx_id, + }; + + for (execbuf.flags = 0; + execbuf.flags <= I915_EXEC_RING_MASK; + execbuf.flags++) + igt_assert_eq(__gem_execbuf(i915, &execbuf), -EINVAL); + + gem_close(i915, obj.handle); + } + + gem_context_destroy(i915, p.ctx_id); +} + static void execute_one(int i915) { I915_DEFINE_CONTEXT_PARAM_ENGINES(engines , I915_EXEC_RING_MASK + 1); @@ -527,6 +569,9 @@ igt_main igt_subtest("idempotent") idempotent(i915); + igt_subtest("none") + none(i915); + igt_subtest("execute-one") execute_one(i915);
Setup a context with no engines, and make sure we reject all execution attempts. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> --- tests/i915/gem_ctx_engines.c | 45 ++++++++++++++++++++++++++++++++++++ 1 file changed, 45 insertions(+)