drm/i915/selftests: Flush the active barriers before asserting
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 28 Jul 2020 15:33:25 +0000 (16:33 +0100)
committerJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Mon, 7 Sep 2020 10:14:15 +0000 (13:14 +0300)
Before we peek at the barrier status for an assert, first serialise with
its callbacks so that we see a stable value.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Thomas Hellström <thomas.hellstrom@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200728153325.28351-1-chris@chris-wilson.co.uk
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
drivers/gpu/drm/i915/gt/selftest_context.c

index 52af1cee9a94eb4845766538d2598537d9dd7120..1f4020e906a8aecf64503f97da76176cc8a6be07 100644 (file)
@@ -68,6 +68,8 @@ static int context_sync(struct intel_context *ce)
        } while (!err);
        mutex_unlock(&tl->mutex);
 
+       /* Wait for all barriers to complete (remote CPU) before we check */
+       i915_active_unlock_wait(&ce->active);
        return err;
 }