summaryrefslogtreecommitdiff
path: root/tests/i915/gem_set_tiling_vs_blt.c
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2020-11-21 14:00:58 +0000
committerChris Wilson <chris@chris-wilson.co.uk>2020-11-21 14:01:21 +0000
commit76b7d1ecf6a3c0a5a538146bb055d0eb5fe232d0 (patch)
tree4531aa149d5a573a7fc70787d0c6ca1c9882e415 /tests/i915/gem_set_tiling_vs_blt.c
parent20d3e1fc41859a2486983b41b94d9f9f48b9c203 (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.c8
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);