diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-04-28 12:43:07 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-04-28 14:25:21 +0100 |
| commit | 96a4faf524fcfbb6c1f2ce91238e81062895ec09 (patch) | |
| tree | b7beaac2048422c13da2095928fc9f3cff8c86cd | |
| parent | d631461d5cc3d6dd3b22bdfb810f668bc1935179 (diff) | |
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 <chris@chris-wilson.co.uk>
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200428114307.5153-1-chris@chris-wilson.co.uk
| -rw-r--r-- | drivers/gpu/drm/i915/gt/selftest_rps.c | 8 |
1 files 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; |