Commit 6582f4f6 authored by Chris Wilson's avatar Chris Wilson

drm/i915/selftests: Drain the freedlists between exec passes

During the context execution tests, we issue a lot of work and discard a
lot of objects without releasing the lock and allowing the background
reaper to free those objects. Insert a small break between each pass to
flush the worker.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarMatthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190704165317.21060-1-chris@chris-wilson.co.uk
parent e7539b79
...@@ -562,6 +562,8 @@ static int igt_ctx_exec(void *arg) ...@@ -562,6 +562,8 @@ static int igt_ctx_exec(void *arg)
mock_file_free(i915, file); mock_file_free(i915, file);
if (err) if (err)
return err; return err;
i915_gem_drain_freed_objects(i915);
} }
return 0; return 0;
...@@ -672,6 +674,10 @@ static int igt_shared_ctx_exec(void *arg) ...@@ -672,6 +674,10 @@ static int igt_shared_ctx_exec(void *arg)
dw += rem; dw += rem;
} }
mutex_unlock(&i915->drm.struct_mutex);
i915_gem_drain_freed_objects(i915);
mutex_lock(&i915->drm.struct_mutex);
} }
out_test: out_test:
if (igt_live_test_end(&t)) if (igt_live_test_end(&t))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment