From 96a4faf524fcfbb6c1f2ce91238e81062895ec09 Mon Sep 17 00:00:00 2001 From: Chris Wilson Date: Tue, 28 Apr 2020 12:43:07 +0100 Subject: [PATCH] drm/i915/selftests: Tweak the tolerance for clock ticks to 12.5% Give a small bump for our tolerance on comparing the expected vs measured clock ticks/time from 10% to 12.5% to accommodate a bad result on Sandybridge that was off by 10.3%. Hopefully, that is the worst we will see. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/1802 Signed-off-by: Chris Wilson Reviewed-by: Mika Kuoppala Link: https://patchwork.freedesktop.org/patch/msgid/20200428114307.5153-1-chris@chris-wilson.co.uk --- drivers/gpu/drm/i915/gt/selftest_rps.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/selftest_rps.c b/drivers/gpu/drm/i915/gt/selftest_rps.c index 6ce7003bf92f..21338c1980de 100644 --- a/drivers/gpu/drm/i915/gt/selftest_rps.c +++ b/drivers/gpu/drm/i915/gt/selftest_rps.c @@ -312,15 +312,15 @@ int live_rps_clock_interval(void *arg) engine->name, cycles, time, ktime_to_ns(dt), expected, gt->clock_frequency / 1000); - if (10 * time < 9 * ktime_to_ns(dt) || - 10 * time > 11 * ktime_to_ns(dt)) { + if (10 * time < 8 * ktime_to_ns(dt) || + 8 * time > 10 * ktime_to_ns(dt)) { pr_err("%s: rps clock time does not match walltime!\n", engine->name); err = -EINVAL; } - if (10 * expected < 9 * cycles || - 10 * expected > 11 * cycles) { + if (10 * expected < 8 * cycles || + 8 * expected > 10 * cycles) { pr_err("%s: walltime does not match rps clock ticks!\n", engine->name); err = -EINVAL; -- 2.39.5