From 6102e2eca949b6827def000407224f3c30a37204 Mon Sep 17 00:00:00 2001 From: Chris Wilson Date: Tue, 3 May 2016 21:12:12 +0100 Subject: igt/gem_exec_flush: Give even batch write a unique value One property lost in the expansion for various coherency checks was ensuring that every time we overwrote the batch it had a unique value (to ensure that the GPU was seeing the latest value). Signed-off-by: Chris Wilson --- tests/gem_exec_flush.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/gem_exec_flush.c b/tests/gem_exec_flush.c index fcd62bb3..bcc539f3 100644 --- a/tests/gem_exec_flush.c +++ b/tests/gem_exec_flush.c @@ -351,7 +351,7 @@ static void batch(int fd, unsigned ring, int nchild, int timeout, b[-1] -= 1; *b++ = offset; } - *b++ = i; + *b++ = cycles + i; *b++ = MI_BATCH_BUFFER_END; switch (mode) { @@ -371,7 +371,6 @@ static void batch(int fd, unsigned ring, int nchild, int timeout, break; } gem_execbuf(fd, &execbuf); - cycles++; execbuf.batch_start_offset += 64; reloc.offset += 64; @@ -379,14 +378,15 @@ static void batch(int fd, unsigned ring, int nchild, int timeout, if (!(flags & COHERENT)) { gem_set_domain(fd, obj[0].handle, - I915_GEM_DOMAIN_CPU, - I915_GEM_DOMAIN_CPU); + I915_GEM_DOMAIN_CPU, + I915_GEM_DOMAIN_CPU); } else gem_sync(fd, obj[0].handle); for (i = 0; i < 1024; i++) { - igt_assert_eq(map[i], i); - map[i] = 0xabcdabcd; + igt_assert_eq_u32(map[i], cycles + i); + map[i] = 0xabcdabcd ^ cycles; } + cycles += 1024; if (mode == BATCH_USER) gem_sync(fd, obj[1].handle); -- cgit v1.2.3