diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-03-24 21:49:38 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-03-24 21:52:20 +0000 |
commit | 190245120758e754813d76b2c6c613413a0dba29 (patch) | |
tree | 2b10269b69bec2e4f411f04729071eab1823a973 /tests/i915/i915_pm_rc6_residency.c | |
parent | d663bfe84ed9ad122ccf87cc05d153ee28554c01 (diff) |
i915/i915_pm_rc6_residency: Suppress idle leakage warning on older arches
Sandybridge's rc6p while only consuming half the power of rc6, still
consumes a non-zero power. It appears Ivybridge was the first platform
to achieve negligible power draw (at least reported by the GPU) while in
rc6p (and later platforms in rc6). So exclude Sandybridge from
generating a warning as we have no hope of fixing.
Closes: https://gitlab.freedesktop.org/drm/intel/issues/1518
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Diffstat (limited to 'tests/i915/i915_pm_rc6_residency.c')
-rw-r--r-- | tests/i915/i915_pm_rc6_residency.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/i915/i915_pm_rc6_residency.c b/tests/i915/i915_pm_rc6_residency.c index 0e3bb98b..5cbaaf1c 100644 --- a/tests/i915/i915_pm_rc6_residency.c +++ b/tests/i915/i915_pm_rc6_residency.c @@ -329,6 +329,7 @@ static void rc6_idle(int i915) { const int64_t duration_ns = SLEEP_DURATION * (int64_t)NSEC_PER_SEC; const int tolerance = 20; /* Some RC6 is better than none! */ + const int gen = intel_gen(intel_get_drm_devid(i915)); struct { const char *name; unsigned int flags; @@ -360,7 +361,7 @@ static void rc6_idle(int i915) if (rapl_read(&rapl, &sample[1])) { double idle = power_J(&rapl, &sample[0], &sample[1]); igt_log(IGT_LOG_DOMAIN, - idle > 1e-3 ? IGT_LOG_WARN : IGT_LOG_INFO, + idle > 1e-3 && gen > 6 ? IGT_LOG_WARN : IGT_LOG_INFO, "Total energy used while idle: %.1fmJ\n", idle * 1e3); } assert_within_epsilon(rc6, ts[1] - ts[0], 5); |