diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2022-04-21 08:09:53 +0200 |
---|---|---|
committer | Zbigniew Kempczyński <zbigniew.kempczynski@intel.com> | 2022-04-22 12:41:00 +0200 |
commit | 3c413017acb7c600af19dcb17b66cb3490f2f12e (patch) | |
tree | e3d0695b6573d5827dc8747d735c80ee2660fce9 /tests/i915/gem_exec_schedule.c | |
parent | 8c2ac78f0d80a2af3c7b47dd2df064cdd07f24b6 (diff) |
i915/gem_exec_schedule: Fix context creation for shared-vm
submit_slice() is using a single address space for a couple of contexts
to prevent rebinding while active, and so requires that address space to
be shared during context construction.
v2: Add separate context for background spinner (Zbigniew)
Fixes: a9987a8dcb8f ("tests/i915/gem_exec_schedule: Convert to intel_ctx_t (v3)")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Diffstat (limited to 'tests/i915/gem_exec_schedule.c')
-rw-r--r-- | tests/i915/gem_exec_schedule.c | 23 |
1 files changed, 15 insertions, 8 deletions
diff --git a/tests/i915/gem_exec_schedule.c b/tests/i915/gem_exec_schedule.c index cf7e4d4e..caeac255 100644 --- a/tests/i915/gem_exec_schedule.c +++ b/tests/i915/gem_exec_schedule.c @@ -818,8 +818,8 @@ static void submit_slice(int i915, const intel_ctx_cfg_t *cfg, intel_ctx_cfg_t engine_cfg = { .num_engines = 1, }; - const intel_ctx_t *ctx; - uint64_t ahnd0 = get_reloc_ahnd(i915, 0); + const intel_ctx_t *ctx, *bg_ctx; + uint64_t ahnd, bg_ahnd; /* * When using a submit fence, we do not want to block concurrent work, @@ -828,19 +828,25 @@ static void submit_slice(int i915, const intel_ctx_cfg_t *cfg, igt_require(gem_scheduler_has_timeslicing(i915)); igt_require(intel_gen(intel_get_drm_devid(i915)) >= 8); + igt_require(gem_has_vm(i915)); + + engine_cfg.vm = gem_vm_create(i915); + ahnd = intel_allocator_open_vm(i915, engine_cfg.vm, INTEL_ALLOCATOR_RELOC); + bg_ctx = intel_ctx_create(i915, cfg); + bg_ahnd = get_reloc_ahnd(i915, bg_ctx->id); for_each_ctx_cfg_engine(i915, cfg, cancel) { igt_spin_t *bg, *spin; int timeline = -1; int fence = -1; - uint64_t ahndN; if (!gem_class_can_store_dword(i915, cancel->class)) continue; igt_debug("Testing cancellation from %s\n", e->name); - bg = igt_spin_new(i915, .ahnd = ahnd0, .engine = e->flags); + bg = igt_spin_new(i915, .ahnd = bg_ahnd, .ctx = bg_ctx, + .engine = e->flags); if (flags & LATE_SUBMIT) { timeline = sw_sync_timeline_create(); @@ -850,8 +856,7 @@ static void submit_slice(int i915, const intel_ctx_cfg_t *cfg, engine_cfg.engines[0].engine_class = e->class; engine_cfg.engines[0].engine_instance = e->instance; ctx = intel_ctx_create(i915, &engine_cfg); - ahndN = get_reloc_ahnd(i915, ctx->id); - spin = igt_spin_new(i915, .ahnd = ahndN, .ctx = ctx, + spin = igt_spin_new(i915, .ahnd = ahnd, .ctx = ctx, .fence = fence, .flags = IGT_SPIN_POLL_RUN | @@ -880,10 +885,12 @@ static void submit_slice(int i915, const intel_ctx_cfg_t *cfg, igt_spin_free(i915, bg); intel_ctx_destroy(i915, ctx); - put_ahnd(ahndN); } - put_ahnd(ahnd0); + gem_vm_destroy(i915, engine_cfg.vm); + intel_ctx_destroy(i915, bg_ctx); + put_ahnd(bg_ahnd); + put_ahnd(ahnd); } static uint32_t __batch_create(int i915, uint32_t offset) |