diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-03-24 14:01:13 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-03-24 14:03:23 +0000 |
commit | d1dcf40cc6869ac858586c5ad9f09af6617ce2ee (patch) | |
tree | 7bf880fc79dff919df6e6fe594c7a6732fde353d /tests/i915/i915_pm_rc6_residency.c | |
parent | cc5d8b4a1bd398fc68271fd9fefc939181db6a2d (diff) |
i915/i915_pm_rc6_residency: Reduce the energy used while idle to a warn
The energy measurement is not the main focus of the test, although it is
important that we do minimise it. As that is not so critical make the
assertion into a warning, so that we do check the mostly idle rc6/energy
consumption.
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 | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tests/i915/i915_pm_rc6_residency.c b/tests/i915/i915_pm_rc6_residency.c index f507f76b..a5ec4873 100644 --- a/tests/i915/i915_pm_rc6_residency.c +++ b/tests/i915/i915_pm_rc6_residency.c @@ -348,8 +348,9 @@ static void rc6_idle(int i915) slept, ts[1] - ts[0], rc6); if (rapl_read(&rapl, &sample[1])) { double idle = power_J(&rapl, &sample[0], &sample[1]); - igt_info("Total energy used while idle: %.1fmJ\n", idle * 1e3); - igt_assert(idle < 1e-3); + igt_log(IGT_LOG_DOMAIN, + idle > 1e-3 ? IGT_LOG_WARN : IGT_LOG_INFO, + "Total energy used while idle: %.1fmJ\n", idle * 1e3); } assert_within_epsilon(rc6, ts[1] - ts[0], 5); |