[PATCH i-g-t 62/81] tests/i915/gem_vm_create: Delete destroy racing tests
Jason Ekstrand
jason at jlekstrand.net
Wed Jul 7 21:03:29 UTC 2021
The races these tests attempt to exercise won't exist once we rework the
context creation to make the VM immutable state.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
---
tests/i915/gem_vm_create.c | 104 -------------------------------------
1 file changed, 104 deletions(-)
diff --git a/tests/i915/gem_vm_create.c b/tests/i915/gem_vm_create.c
index 70b43dc6d..4b77920bc 100644
--- a/tests/i915/gem_vm_create.c
+++ b/tests/i915/gem_vm_create.c
@@ -353,104 +353,6 @@ static void isolation(int i915)
gem_vm_destroy(i915, vm[0]);
}
-static void async_destroy(int i915)
-{
- struct drm_i915_gem_context_param arg = {
- .ctx_id = gem_context_create(i915),
- .value = gem_vm_create(i915),
- .param = I915_CONTEXT_PARAM_VM,
- };
- igt_spin_t *spin[2];
- int err;
-
- spin[0] = igt_spin_new(i915,
- .ctx_id = arg.ctx_id,
- .flags = IGT_SPIN_POLL_RUN);
- igt_spin_busywait_until_started(spin[0]);
-
- err = __gem_context_set_param(i915, &arg);
- if (err == -EBUSY) /* update while busy may be verboten, let it ride. */
- err = 0;
- igt_assert_eq(err, 0);
-
- spin[1] = __igt_spin_new(i915, .ctx_id = arg.ctx_id);
-
- igt_spin_end(spin[0]);
- gem_sync(i915, spin[0]->handle);
-
- gem_vm_destroy(i915, arg.value);
- gem_context_destroy(i915, arg.ctx_id);
-
- igt_spin_end(spin[1]);
- gem_sync(i915, spin[1]->handle);
-
- for (int i = 0; i < ARRAY_SIZE(spin); i++)
- igt_spin_free(i915, spin[i]);
-}
-
-static void destroy_race(int i915)
-{
- const int ncpus = sysconf(_SC_NPROCESSORS_ONLN);
- uint32_t *vm;
-
- /* Check we can execute a polling spinner */
- igt_spin_free(i915, igt_spin_new(i915, .flags = IGT_SPIN_POLL_RUN));
-
- vm = mmap(NULL, 4096, PROT_WRITE, MAP_SHARED | MAP_ANON, -1, 0);
- igt_assert(vm != MAP_FAILED);
-
- for (int child = 0; child < ncpus; child++)
- vm[child] = gem_vm_create(i915);
-
- igt_fork(child, ncpus) {
- uint32_t ctx = gem_context_create(i915);
- igt_spin_t *spin;
-
- spin = __igt_spin_new(i915, ctx, .flags = IGT_SPIN_POLL_RUN);
- while (!READ_ONCE(vm[ncpus])) {
- struct drm_i915_gem_context_param arg = {
- .ctx_id = ctx,
- .param = I915_CONTEXT_PARAM_VM,
- .value = READ_ONCE(vm[child]),
- };
- igt_spin_t *nxt;
-
- if (__gem_context_set_param(i915, &arg))
- continue;
-
- nxt = __igt_spin_new(i915, ctx,
- .flags = IGT_SPIN_POLL_RUN);
-
- igt_spin_end(spin);
- gem_sync(i915, spin->handle);
- igt_spin_free(i915, spin);
-
- usleep(1000 + hars_petruska_f54_1_random_unsafe() % 2000);
-
- spin = nxt;
- }
-
- igt_spin_free(i915, spin);
- gem_context_destroy(i915, ctx);
- }
-
- igt_until_timeout(5) {
- for (int child = 0; child < ncpus; child++) {
- gem_vm_destroy(i915, vm[child]);
- vm[child] = gem_vm_create(i915);
- }
- usleep(1000 + hars_petruska_f54_1_random_unsafe() % 2000);
- }
-
- vm[ncpus] = 1;
- igt_waitchildren();
-
- for (int child = 0; child < ncpus; child++)
- gem_vm_destroy(i915, vm[child]);
-
- munmap(vm, 4096);
-}
-
igt_main
{
int i915 = -1;
@@ -480,12 +382,6 @@ igt_main
igt_subtest("create-ext")
create_ext(i915);
-
- igt_subtest("async-destroy")
- async_destroy(i915);
-
- igt_subtest("destroy-race")
- destroy_race(i915);
}
igt_fixture {
--
2.31.1
More information about the Intel-gfx-trybot
mailing list