Beware the slithy t'oves.

Forked GTT access on icl is notoriously slow, so rather than spend an
eternity checking the whole object, check for a completion event after
handling the pagefault. It's is the race of the pagefault vs reset that
we care most about, and we expect the bug to result in the pagefault
being blocked indefinitely, so checking afterwards does not reduce
coverage.

Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com>
---
 tests/i915/gem_mmap_gtt.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/i915/gem_mmap_gtt.c b/tests/i915/gem_mmap_gtt.c
index 0428a1372..91da5a37b 100644
--- a/tests/i915/gem_mmap_gtt.c
+++ b/tests/i915/gem_mmap_gtt.c
@@ -602,6 +602,9 @@ test_hang(int fd)
 
                                gtt[0][x] = patterns[next_pattern];
                                gtt[1][x] = patterns[next_pattern];
+
+                               if (READ_ONCE(control->done))
+                                       break;
                        }
 
                        last_pattern = next_pattern;
-- 
2.23.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to