On Fri, 12 Aug 2022 11:53:45 +0200 Janusz Krzysztofik <janusz.krzyszto...@linux.intel.com> wrote:
> When i = 0, condition out[i] == i will be true even if a batch supposed > to store i in out[i] fails. Don't use i = 0 in batches that store values > in out[i]. > > v2: still check for out[0] == 0 (Chris) > > Signed-off-by: Janusz Krzysztofik <janusz.krzyszto...@linux.intel.com> > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: Mauro Carvalho Chehab <mauro.che...@linux.intel.com> LGTM. Reviewed-by: Mauro Carvalho Chehab <mche...@kernel.org> > --- > tests/i915/gem_exec_fence.c | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/tests/i915/gem_exec_fence.c b/tests/i915/gem_exec_fence.c > index 048870053a..78d83460f7 100644 > --- a/tests/i915/gem_exec_fence.c > +++ b/tests/i915/gem_exec_fence.c > @@ -334,6 +334,8 @@ static void test_fence_await(int fd, const intel_ctx_t > *ctx, > if (!gem_class_can_store_dword(fd, e2->class)) > continue; > > + i++; > + > if (flags & NONBLOCK) { > igt_store_word(fd, ahnd, ctx, e2, spin->out_fence, > scratch, scratch_offset, i, i); > @@ -345,8 +347,6 @@ static void test_fence_await(int fd, const intel_ctx_t > *ctx, > put_ahnd(ahnd); > } > } > - > - i++; > } > > igt_spin_busywait_until_started(spin); > @@ -356,7 +356,7 @@ static void test_fence_await(int fd, const intel_ctx_t > *ctx, > if ((flags & HANG) == 0) { > /* Check for invalidly completing the task early */ > igt_assert(fence_busy(spin->out_fence)); > - for (int n = 0; n < i; n++) > + for (int n = 0; n <= i; n++) > igt_assert_eq_u32(out[n], 0); > > igt_spin_end(spin); > @@ -366,8 +366,11 @@ static void test_fence_await(int fd, const intel_ctx_t > *ctx, > > gem_set_domain(fd, scratch, I915_GEM_DOMAIN_GTT, 0); > igt_assert(!fence_busy(spin->out_fence)); > - while ((flags & HANG) == 0 && i--) > - igt_assert_eq_u32(out[i], i); > + if ((flags & HANG) == 0) { > + do > + igt_assert_eq_u32(out[i], i); > + while (i--); > + } > munmap(out, 4096); > > igt_spin_free(fd, spin);