From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
Converts all per-engine tests into dynamic subtests and in the process:
* Put back I915_EXEC_BSD legacy coverage.
* Remove one added static engine list usage.
* Compact code by driving two groups of the name/func table.
v2:
* Convert smoketest to proper all engines.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
Cc: Bommu Krishnaiah <krishnaiah.bommu@xxxxxxxxx>
---
tests/i915/gem_ctx_persistence.c | 138 ++++++++++++++-----------------
1 file changed, 63 insertions(+), 75 deletions(-)
diff --git a/tests/i915/gem_ctx_persistence.c b/tests/i915/gem_ctx_persistence.c
index 8b9633b214ff..3b4b55644635 100644
--- a/tests/i915/gem_ctx_persistence.c
+++ b/tests/i915/gem_ctx_persistence.c
@@ -693,6 +693,7 @@ static void __smoker(int i915, unsigned int engine, int expected)
int fd, extra;
fd = gem_reopen_driver(i915);
+ gem_context_copy_engines(i915, 0, fd, 0);
gem_context_set_persistence(fd, 0, expected > 0);
spin = igt_spin_new(fd, .engine = engine, .flags = IGT_SPIN_FENCE_OUT);
@@ -721,7 +722,7 @@ static void __smoker(int i915, unsigned int engine, int expected)
}
spin->handle = 0;
- igt_spin_free(i915, spin);
+ igt_spin_free(fd, spin);
}
static void smoker(int i915, unsigned int engine, unsigned int *ctl)
@@ -734,6 +735,7 @@ static void smoker(int i915, unsigned int engine, unsigned int *ctl)
static void smoketest(int i915)
{
+ const struct intel_execution_engine2 *e;
uint32_t *ctl;
/*
@@ -744,9 +746,9 @@ static void smoketest(int i915)
ctl = mmap(0, 4096, PROT_WRITE, MAP_SHARED | MAP_ANON, -1, 0);
igt_assert(ctl != MAP_FAILED);
- for_each_physical_engine(e, i915) {
+ __for_each_physical_engine(i915, e) {
igt_fork(child, 4)
- smoker(i915, eb_ring(e), ctl);
+ smoker(i915, e->flags, ctl);
}
sleep(20);
@@ -759,7 +761,20 @@ static void smoketest(int i915)
igt_main
{
- const struct intel_execution_engine2 *e;
+ struct {
+ const char *name;
+ void (*func)(int fd, unsigned int engine);
+ } *test, tests[] = {
+ { "persistence", test_persistence },
+ { "cleanup", test_nonpersistent_cleanup },
+ { "queued", test_nonpersistent_queued },
+ { "mixed", test_nonpersistent_mixed },
+ { "mixed-process", test_process_mixed },
+ { "hostile", test_nonpersistent_hostile },
+ { "hostile-preempt", test_nonpersistent_hostile_preempt },
+ { "hang", test_nonpersistent_hang },
+ { NULL, NULL },
+ };
int i915;
igt_fixture {
@@ -772,91 +787,64 @@ igt_main
igt_allow_hang(i915, 0, 0);
}
- igt_subtest("idempotent")
- test_idempotent(i915);
-
- igt_subtest("clone")
- test_clone(i915);
-
- igt_subtest("file")
- test_nonpersistent_file(i915);
-
- igt_subtest("process")
- test_process(i915);
-
- igt_subtest("processes")
- test_processes(i915);
-
- igt_subtest("hostile")
- test_nohangcheck_hostile(i915);
- igt_subtest("hang")
- test_nohangcheck_hang(i915);
-
- __for_each_static_engine(e) {
- igt_subtest_group {
- igt_fixture {
- gem_require_ring(i915, e->flags);
- gem_require_contexts(i915);
- }
+ /* Legacy execbuf engine selection flags. */
+ igt_subtest_group {
+ igt_fixture
+ gem_require_contexts(i915);
- igt_subtest_f("legacy-%s-persistence", e->name)
- test_persistence(i915, e->flags);
+ igt_subtest("idempotent")
+ test_idempotent(i915);
- igt_subtest_f("legacy-%s-cleanup", e->name)
- test_nonpersistent_cleanup(i915, e->flags);
+ igt_subtest("clone")
+ test_clone(i915);
- igt_subtest_f("legacy-%s-queued", e->name)
- test_nonpersistent_queued(i915, e->flags);
+ igt_subtest("file")
+ test_nonpersistent_file(i915);
- igt_subtest_f("legacy-%s-mixed", e->name)
- test_nonpersistent_mixed(i915, e->flags);
+ igt_subtest("process")
+ test_process(i915);
- igt_subtest_f("legacy-%s-mixed-process", e->name)
- test_process_mixed(i915, e->flags);
+ igt_subtest("processes")
+ test_processes(i915);
- igt_subtest_f("legacy-%s-hostile", e->name)
- test_nonpersistent_hostile(i915, e->flags);
+ igt_subtest("hostile")
+ test_nohangcheck_hostile(i915);
+ igt_subtest("hang")
+ test_nohangcheck_hang(i915);
- igt_subtest_f("legacy-%s-hostile-preempt", e->name)
- test_nonpersistent_hostile_preempt(i915,
- e->flags);
+ for (test = tests; test->name; test++) {
+ igt_subtest_with_dynamic_f("legacy-engines-%s",
+ test->name) {
+ for_each_engine(e, i915) {
+ igt_dynamic_f("%s", e->name)
+ test->func(i915, eb_ring(e));
+ }
+ }
}
- }
-
- __for_each_physical_engine(i915, e) {
- igt_subtest_group {
- igt_fixture
- gem_require_contexts(i915);
-
- igt_subtest_f("%s-persistence", e->name)
- test_persistence(i915, e->flags);
-
- igt_subtest_f("%s-cleanup", e->name)
- test_nonpersistent_cleanup(i915, e->flags);
- igt_subtest_f("%s-queued", e->name)
- test_nonpersistent_queued(i915, e->flags);
-
- igt_subtest_f("%s-mixed", e->name)
- test_nonpersistent_mixed(i915, e->flags);
-
- igt_subtest_f("%s-mixed-process", e->name)
- test_process_mixed(i915, e->flags);
+ /* Assert things are under control. */
+ igt_assert(!gem_context_has_engine_map(i915, 0));
+ }
- igt_subtest_f("%s-hostile", e->name)
- test_nonpersistent_hostile(i915, e->flags);
+ /* New way of selecting engines. */
+ igt_subtest_group {
+ const struct intel_execution_engine2 *e;
- igt_subtest_f("%s-hostile-preempt", e->name)
- test_nonpersistent_hostile_preempt(i915,
- e->flags);
+ igt_fixture
+ gem_require_contexts(i915);
- igt_subtest_f("%s-hang", e->name)
- test_nonpersistent_hang(i915, e->flags);
+ for (test = tests; test->name; test++) {
+ igt_subtest_with_dynamic_f("engines-%s", test->name) {
+ __for_each_physical_engine(i915, e) {
+ igt_dynamic_f("%s", e->name)
+ test->func(i915, e->flags);
+ }
+ }
}
- }
- igt_subtest("smoketest")
- smoketest(i915);
+ igt_subtest("smoketest")
+ smoketest(i915);
+ }
igt_fixture {
close(i915);
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
