Bug 104099

Summary:	[CI] igt@pm_rc6_residency@rc6-accuracy - fail - Test assertion failure function residency_accuracy - Failed assertion: ratio > 0.9 && ratio < 1.05 -
Product:	DRI	Reporter:	Marta Löfstedt <marta.lofstedt>
Component:	DRM/Intel	Assignee:	Marta Löfstedt <marta.lofstedt>
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:	KBL	i915 features:	power/Other

Description Marta Löfstedt 2017-12-05 10:26:02 UTC

https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_3456/shard-kbl5/igt@pm_rc6_residency@rc6-accuracy.html
	

(pm_rc6_residency:1949) CRITICAL: Test assertion failure function residency_accuracy, file pm_rc6_residency.c:85:
(pm_rc6_residency:1949) CRITICAL: Failed assertion: ratio > 0.9 && ratio < 1.05
(pm_rc6_residency:1949) CRITICAL: Last errno: 9, Bad file descriptor
(pm_rc6_residency:1949) CRITICAL: Sysfs RC6 residency counter is inaccurate.
Subtest rc6-accuracy failed.

I was pondering re-opening the old bug 102566. However, in that one we never had the "Last errno: 9, Bad file descriptor", so maybe this is something else.

Comment 1 Jani Saarinen 2017-12-05 13:06:57 UTC

Reference: https://patchwork.freedesktop.org/series/34897/

Comment 2 Chris Wilson 2017-12-05 13:09:09 UTC

Let's try

commit 66b399b61b5051ac5a1933da0291dbc547a921ca
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Tue Dec 5 10:37:13 2017 +0000

    igt/pm_rc6_residency: Replace hard-coded sleep before rc6 with a probe
    
    Instead of trying to sleep for 2 evaluations intervals and then assuming
    that rc6 is working, poll the rc6 residency instead.
    
    References: https://bugs.freedesktop.org/show_bug.cgi?id=104099
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
    Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

Comment 3 Marta Löfstedt 2017-12-05 13:15:17 UTC

Shure I will monitor once the fix is integrated

Comment 4 Marta Löfstedt 2017-12-07 09:54:21 UTC

Fix integrated in CI_DRM_3459 test is green I close

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.