summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2018-03-27 09:33:09 +0100
committerChris Wilson <chris@chris-wilson.co.uk>2018-03-27 21:25:34 +0100
commitd5657417d6e5ac67c5af17688246977de258bfaa (patch)
tree00b1bf04257f687471ff8aa3eb7f50857d3588f7
parent3fa0b027304ec28cd24b314349d3731b55dfcc0a (diff)
igt/perf_pmu: Most-busy requires at least one busy engine
The test is whether with all but one engine busy we record the correct load on each engine. If we only have one engine, this test degenerates into all-idle/all-busy, so we can skip to avoid crashing on the assumption that we have a busy spinner. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
-rw-r--r--tests/perf_pmu.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c
index f27b7ec7..b59af818 100644
--- a/tests/perf_pmu.c
+++ b/tests/perf_pmu.c
@@ -513,6 +513,7 @@ most_busy_check_all(int gem_fd, const struct intel_execution_engine2 *e,
val[i++] = I915_PMU_ENGINE_BUSY(e_->class, e_->instance);
}
igt_assert(i == num_engines);
+ igt_require(spin); /* at least one busy engine */
fd[0] = -1;
for (i = 0; i < num_engines; i++)