Bug 103885

Summary:	[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
Product:	DRI	Reporter:	Marta Löfstedt <marta.lofstedt>
Component:	DRM/Intel	Assignee:	Intel GFX Bugs mailing list <intel-gfx-bugs>
Status:	CLOSED FIXED	QA Contact:	Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity:	normal
Priority:	medium	CC:	intel-gfx-bugs
Version:	DRI git
Hardware:	Other
OS:	All
Whiteboard:	ReadyForDev
i915 platform:	SNB	i915 features:	Perf/OA

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.