Bug 104099 - [CI] igt@pm_rc6_residency@rc6-accuracy - fail - Test assertion failure function residency_accuracy - Failed assertion: ratio > 0.9 && ratio < 1.05 -
Summary: [CI] igt@pm_rc6_residency@rc6-accuracy - fail - Test assertion failure functi...
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other All
: medium normal
Assignee: Marta Löfstedt
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: ReadyForDev
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-05 10:26 UTC by Marta Löfstedt
Modified: 2017-12-07 09:54 UTC (History)
1 user (show)

See Also:
i915 platform: KBL
i915 features: power/Other


Attachments

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.