diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-11-21 14:00:58 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-11-21 14:01:21 +0000 |
commit | 76b7d1ecf6a3c0a5a538146bb055d0eb5fe232d0 (patch) | |
tree | 4531aa149d5a573a7fc70787d0c6ca1c9882e415 /tests/i915/gem_set_tiling_vs_blt.c | |
parent | 20d3e1fc41859a2486983b41b94d9f9f48b9c203 (diff) |
i915/gem_set_tiling_vs_blt: Undo accidental context insertion
The target platforms do not support contexts, so don't use them (they
also don't support preemption, so this becomes a test demonstrating the
lack of pipelined fencing).
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Zbigniew KempczyĆski <zbigniew.kempczynski@intel.com>
Diffstat (limited to 'tests/i915/gem_set_tiling_vs_blt.c')
-rw-r--r-- | tests/i915/gem_set_tiling_vs_blt.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/tests/i915/gem_set_tiling_vs_blt.c b/tests/i915/gem_set_tiling_vs_blt.c index 446d446f..ff600f94 100644 --- a/tests/i915/gem_set_tiling_vs_blt.c +++ b/tests/i915/gem_set_tiling_vs_blt.c @@ -91,7 +91,7 @@ static void do_test(struct buf_ops *bops, uint32_t tiling, unsigned stride, int i, fd = buf_ops_get_fd(bops); uint32_t *ptr; uint32_t blt_stride, blt_bits; - uint32_t busy_ctx, ring = I915_EXEC_DEFAULT; + uint32_t ring = I915_EXEC_DEFAULT; bool tiling_changed = false; ibb = intel_bb_create_with_relocs(fd, 4096); @@ -99,8 +99,7 @@ static void do_test(struct buf_ops *bops, uint32_t tiling, unsigned stride, igt_info("filling ring\n"); if (HAS_BLT_RING(ibb->devid)) ring = I915_EXEC_BLT; - busy_ctx = gem_context_create(fd); - busy = igt_spin_new(fd, .ctx = busy_ctx, .engine = ring); + busy = igt_spin_new(fd, .engine = ring); igt_info("playing tricks .. "); /* first allocate the target so it gets out of the way of playing funky @@ -188,6 +187,7 @@ static void do_test(struct buf_ops *bops, uint32_t tiling, unsigned stride, intel_bb_emit_reloc_fenced(ibb, test_buf->handle, I915_GEM_DOMAIN_RENDER, 0, 0, 0); intel_bb_flush_blit(ibb); + igt_spin_free(fd, busy); /* Now try to trick the kernel the kernel into changing up the fencing * too early. */ igt_info("checking .. "); @@ -209,8 +209,6 @@ static void do_test(struct buf_ops *bops, uint32_t tiling, unsigned stride, gem_munmap(ptr, TEST_SIZE); igt_assert(tiling_changed); - igt_spin_free(fd, busy); - gem_context_destroy(fd, busy_ctx); intel_buf_destroy(test_buf); intel_buf_destroy(target_buf); |