The test already tried to use a working mapping by first trying legacy
WC, then GTT.  Use gem_mmap__device_coherent() helper instead of
approaching its implementation locally.

Signed-off-by: Janusz Krzysztofik <[email protected]>
---
 tests/i915/gem_exec_gttfill.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/tests/i915/gem_exec_gttfill.c b/tests/i915/gem_exec_gttfill.c
index f810dafd1..27277df48 100644
--- a/tests/i915/gem_exec_gttfill.c
+++ b/tests/i915/gem_exec_gttfill.c
@@ -155,15 +155,10 @@ static void fillgtt(int fd, unsigned ring, int timeout)
        igt_assert(batches);
        for (unsigned i = 0; i < count; i++) {
                batches[i].handle = gem_create(fd, BATCH_SIZE);
-               batches[i].ptr =
-                       __gem_mmap__wc(fd, batches[i].handle,
-                                      0, BATCH_SIZE, PROT_WRITE);
-               if (!batches[i].ptr) {
-                       batches[i].ptr =
-                               __gem_mmap__gtt(fd, batches[i].handle,
-                                               BATCH_SIZE, PROT_WRITE);
-               }
-               igt_require(batches[i].ptr);
+               batches[i].ptr = gem_mmap__device_coherent(fd,
+                                                          batches[i].handle, 0,
+                                                          BATCH_SIZE,
+                                                          PROT_WRITE);
        }
 
        /* Flush all memory before we start the timer */
-- 
2.21.1

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to