Bug 103885 - [CI] igt@perf_pmu@rc6p - fail - Failed assertion: (double)(idle[i] - prev[i]) <= (1.0 + tolerance) * (double)slept && (double)(idle[i] - prev[i]) >= (1.0 - tolerance) * (double)slept
Summary: [CI] igt@perf_pmu@rc6p - fail - Failed assertion: (double)(idle[i] - prev[i])...
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other All
: medium normal
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: ReadyForDev
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-24 09:21 UTC by Marta Löfstedt
Modified: 2017-11-27 10:12 UTC (History)
1 user (show)

See Also:
i915 platform: SNB
i915 features: Perf/OA


Attachments

Description Marta Löfstedt 2017-11-24 09:21:48 UTC
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_3372/shard-snb6/igt@perf_pmu@rc6p.html

(perf_pmu:7111) CRITICAL: Test assertion failure function test_rc6p, file perf_pmu.c:1058:
(perf_pmu:7111) CRITICAL: Failed assertion: (double)(idle[i] - prev[i]) <= (1.0 + tolerance) * (double)slept && (double)(idle[i] - prev[i]) >= (1.0 - tolerance) * (double)slept
(perf_pmu:7111) CRITICAL: 'idle[i] - prev[i]' != 'slept' (0.000000 not within 5.000000% tolerance of 2000105898.000000)
Subtest rc6p failed.
Comment 1 Marta Löfstedt 2017-11-24 09:27:32 UTC
Note this was previously handled in bug 103857. However, that bug was fixed for a lot of other tests and machines.
Comment 2 Chris Wilson 2017-11-24 21:45:42 UTC
The test is obsolete from

commit 3452fa3095e91acbcb1f6290e0d70fa7d3695a3a
Author: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Date:   Fri Nov 24 17:13:31 2017 +0000

    drm/i915/pmu: Aggregate all RC6 states into one counter
    
    Chris has discovered that RC6, RC6p and RC6pp counters are mutually
    exclusive, and even that on some SNB SKUs you get RC6p increasing, and on
    the others RC6.
    
    Furthermore RC6p and RC6pp were only present starting from GEN6 until,
    GEN7, not including Haswell.
    
    All this combined makes it questionable whether we need to reserve new ABI
    for these counters. One idea was to just combine them all under the RC6
    counter to simplify things for userspace. So that is what this patch does.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.