Bug 84886 - [BDW/BSW Bisected]igt/pm_rc6_residency/residency-accuracy fails
Summary: [BDW/BSW Bisected]igt/pm_rc6_residency/residency-accuracy fails
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: Other All
: high normal
Assignee: Rodrigo Vivi
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-11 00:52 UTC by Guo Jinxian
Modified: 2017-10-06 14:35 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Guo Jinxian 2014-10-11 00:52:38 UTC
==System Environment==
--------------------------
Regression: Yes. 

Non-working platforms: BDW

==kernel==
--------------------------
origin/drm-intel-nightly: ea4bec8e96ea8b33b49a7892c1c7f20041a56da6(fails)
    drm-intel-nightly: 2014y-10m-09d-07h-58m-45s UTC integration manifest


==Bug detailed description==
igt/pm_rc6_residency/residency-accuracy fails

Output:
IGT-Version: 1.8-g7f82289 (x86_64) (Linux: 3.17.0_prts_ff1ea7_20141009_debug+ x86_64)
Test assertion failure function readit, file pm_rc6_residency.c:48:
Failed assertion: file
Last errno: 2, No such file or directory
Subtest residency-accuracy: FAIL


==Reproduce steps==
---------------------------- 
1. ./pm_rc6_residency --run-subtest residency-accuracy

==Bisect results from PRTS==
----------------------------
Bisect shows: bb530a44ceaec228bdf7329e303875d2cc82c115 is the first bad commit

commit bb530a44ceaec228bdf7329e303875d2cc82c115
Author:     Rodrigo Vivi <rodrigo.vivi@intel.com>
AuthorDate: Tue Oct 7 07:06:50 2014 -0700
Commit:     Daniel Vetter <daniel.vetter@ffwll.ch>
CommitDate: Wed Oct 8 11:22:39 2014 +0200

    drm/i915: Do not export RC6p and RC6pp if they don't exist
    
    Avoid to expose RC6 and RC6pp to the platforms that doesn't support it.
    So powertop can be changed to show RC6p and RC6pp only on the platforms
    they are available.
    
    v2: Simplify by merging RC6p and RC6pp groups and respect the spec that
    mentions deep and deepest RC6 on SNB and IVB although they keep disabled
    by default.
    
    v3: Remove unecessary space.
    
    v4: RC6p and RC6pp is only for SNB and IVB; unify debug msg and use
        has_rc6p() on sanitize options instead of is gen 6 and ivb.
    
    v5: yet another fix on has_rc6p macro. final is_gen6 or is_ivb! To make sure
        we are excluding hsw and baytrail.
    
    References: https://bugs.freedesktop.org/show_bug.cgi?id=84524
    Cc: Josh Triplett <josh.triplett@intel.com>
    Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
    Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
    Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
    Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Comment 1 Rodrigo Vivi 2014-10-15 00:24:49 UTC
Patch merged on intel-gpu-tools mater.
Comment 2 Guo Jinxian 2014-10-16 07:13:23 UTC
The failure still able to reproduce on latest -igt(a0b5c6d2ee784a8ea49534050087b3c630a41bc4), I will retest it later.


root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./pm_rc6_residency --run-subtest residency-accuracyIGT-Version: 1.8-ga0b5c6d (x86_64) (Linux: 3.17.0_drm-intel-nightly_2ea23c_20141016+ x86_64)
Test assertion failure function readit, file pm_rc6_residency.c:48:
Failed assertion: file
Last errno: 2, No such file or directory
Subtest residency-accuracy: FAIL
Comment 3 Paulo Zanoni 2014-10-16 12:50:57 UTC
(In reply to Guo Jinxian from comment #2)
> The failure still able to reproduce on latest
> -igt(a0b5c6d2ee784a8ea49534050087b3c630a41bc4), I will retest it later.
> 
> 
> root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests#
> ./pm_rc6_residency --run-subtest residency-accuracyIGT-Version: 1.8-ga0b5c6d
> (x86_64) (Linux: 3.17.0_drm-intel-nightly_2ea23c_20141016+ x86_64)
> Test assertion failure function readit, file pm_rc6_residency.c:48:
> Failed assertion: file
> Last errno: 2, No such file or directory
> Subtest residency-accuracy: FAIL

You need a newer IGT. Also, the subtest names changed, so you need to run jus t"./pm_rc6_residency".
Comment 4 Guo Jinxian 2014-10-17 04:43:44 UTC
(In reply to Paulo Zanoni from comment #3)
> (In reply to Guo Jinxian from comment #2)
> > The failure still able to reproduce on latest
> > -igt(a0b5c6d2ee784a8ea49534050087b3c630a41bc4), I will retest it later.
> > 
> > 
> > root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests#
> > ./pm_rc6_residency --run-subtest residency-accuracyIGT-Version: 1.8-ga0b5c6d
> > (x86_64) (Linux: 3.17.0_drm-intel-nightly_2ea23c_20141016+ x86_64)
> > Test assertion failure function readit, file pm_rc6_residency.c:48:
> > Failed assertion: file
> > Last errno: 2, No such file or directory
> > Subtest residency-accuracy: FAIL
> 
> You need a newer IGT. Also, the subtest names changed, so you need to run
> jus t"./pm_rc6_residency".

on latest igt(303fe742fcdd7da6f03f948a4e41b529ee39613a), the test results are expected as below. Verified.


root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./pm_rc6_residency
IGT-Version: 1.8-g303fe74 (x86_64) (Linux: 3.17.0_drm-intel-nightly_1361e3_20141017+ x86_64)
The residency counter: 0.990333
This machine entry rc6 state.
Subtest rc6-accuracy: SUCCESS (0.000s)
This platform doesn't support RC6p
Subtest rc6p-accuracy: SKIP (0.000s)
This platform doesn't support RC6pp
Subtest rc6pp-accuracy: SKIP (0.000s)
Comment 5 Elizabeth 2017-10-06 14:35:02 UTC
Closing old verified.


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.