diff options
author | Jason Ekstrand <jason@jlekstrand.net> | 2021-04-12 17:15:48 -0500 |
---|---|---|
committer | Jason Ekstrand <jason@jlekstrand.net> | 2021-07-08 12:45:33 -0500 |
commit | 065ffe656870cfee323a892ff95db12c8ce5bef5 (patch) | |
tree | cfda59d67aa49dcedeed4d16668eb3e279eabca1 /tests/i915/gem_ctx_engines.c | |
parent | 8a63746f1218562ceea8f1acd5a02d1dc8fdc9ae (diff) |
tests/i915/gem_ctx_engines: Use better engine iteration
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Acked-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Diffstat (limited to 'tests/i915/gem_ctx_engines.c')
-rw-r--r-- | tests/i915/gem_ctx_engines.c | 32 |
1 files changed, 19 insertions, 13 deletions
diff --git a/tests/i915/gem_ctx_engines.c b/tests/i915/gem_ctx_engines.c index ac03206c..531495c3 100644 --- a/tests/i915/gem_ctx_engines.c +++ b/tests/i915/gem_ctx_engines.c @@ -444,7 +444,7 @@ static void execute_allforone(int i915) i = 0; memset(&engines, 0, sizeof(engines)); - __for_each_physical_engine(i915, e) { + for_each_physical_engine(i915, e) { engine_class(&engines, i) = e->class; engine_instance(&engines, i) = e->instance; i++; @@ -453,7 +453,7 @@ static void execute_allforone(int i915) gem_context_set_param(i915, ¶m); i = 0; - __for_each_physical_engine(i915, e) { + for_each_physical_engine(i915, e) { struct drm_i915_gem_busy busy = {}; igt_spin_t *spin; @@ -490,7 +490,8 @@ static bool has_cs_timestamp(const struct intel_execution_engine2 *e, int gen) return e->class == I915_ENGINE_CLASS_RENDER; } -static void independent(int i915, const struct intel_execution_engine2 *e) +static void independent(int i915, const intel_ctx_t *base_ctx, + const struct intel_execution_engine2 *e) { #define RCS_TIMESTAMP (mmio_base + 0x358) const unsigned int gen = intel_gen(intel_get_drm_devid(i915)); @@ -498,7 +499,6 @@ static void independent(int i915, const struct intel_execution_engine2 *e) const int has_64bit_reloc = gen >= 8; I915_DEFINE_CONTEXT_PARAM_ENGINES(engines, I915_EXEC_RING_MASK + 1); struct drm_i915_gem_context_param param = { - .ctx_id = gem_context_clone_with_engines(i915, 0), .param = I915_CONTEXT_PARAM_ENGINES, .value = to_user_pointer(&engines), .size = sizeof(engines), @@ -518,7 +518,7 @@ static void independent(int i915, const struct intel_execution_engine2 *e) struct drm_i915_gem_execbuffer2 execbuf = { .buffers_ptr = to_user_pointer(&results), .buffer_count = 1, - .rsvd1 = param.ctx_id, + .rsvd1 = base_ctx->id, .flags = e->flags, }; gem_write(i915, results.handle, 0, &bbe, sizeof(bbe)); @@ -531,6 +531,7 @@ static void independent(int i915, const struct intel_execution_engine2 *e) engine_class(&engines, i) = e->class; engine_instance(&engines, i) = e->instance; } + param.ctx_id = gem_context_create(i915); gem_context_set_param(i915, ¶m); gem_set_caching(i915, results.handle, I915_CACHING_CACHED); @@ -590,19 +591,20 @@ static void independent(int i915, const struct intel_execution_engine2 *e) gem_context_destroy(i915, param.ctx_id); } -static void independent_all(int i915) +static void independent_all(int i915, const intel_ctx_t *ctx) { const unsigned int gen = intel_gen(intel_get_drm_devid(i915)); const struct intel_execution_engine2 *e; igt_spin_t *spin = NULL; - __for_each_physical_engine(i915, e) { + for_each_ctx_engine(i915, ctx, e) { if (spin) { spin->execbuf.flags &= ~63; spin->execbuf.flags |= e->flags; gem_execbuf(i915, &spin->execbuf); } else { - spin = igt_spin_new(i915, .engine = e->flags, + spin = igt_spin_new(i915, .ctx = ctx, + .engine = e->flags, .flags = (IGT_SPIN_NO_PREEMPTION | IGT_SPIN_POLL_RUN)); } @@ -610,7 +612,7 @@ static void independent_all(int i915) igt_require(spin); igt_spin_busywait_until_started(spin); - __for_each_physical_engine(i915, e) { + for_each_ctx_engine(i915, ctx, e) { if (!gem_engine_mmio_base(i915, e->name)) continue; @@ -618,7 +620,7 @@ static void independent_all(int i915) continue; igt_fork(child, 1) - independent(i915, e); + independent(i915, ctx, e); } sched_yield(); igt_spin_free(i915, spin); @@ -727,14 +729,18 @@ igt_main execute_allforone(i915); igt_subtest_with_dynamic("independent") { + const intel_ctx_t *ctx; + igt_require(gem_scheduler_enabled(i915)); igt_require(intel_gen(intel_get_drm_devid(i915) >= 6)); - __for_each_physical_engine(i915, e) { + + ctx = intel_ctx_create_all_physical(i915); + for_each_ctx_engine(i915, ctx, e) { igt_dynamic_f("%s", e->name) - independent(i915, e); + independent(i915, ctx, e); } igt_dynamic("all") - independent_all(i915); + independent_all(i915, ctx); } igt_subtest("libapi") |