Bug 74956 - [HSW/BYT/BDW]igt/pm_rps/reset fails
Summary: [HSW/BYT/BDW]igt/pm_rps/reset fails
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: medium normal
Assignee: Jesse Barnes
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-14 03:02 UTC by lu hua
Modified: 2017-10-06 14:40 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (108.05 KB, text/plain)
2014-02-14 03:02 UTC, lu hua
no flags Details
dmesg on 4c0e55 (113.46 KB, text/plain)
2014-02-17 06:09 UTC, lu hua
no flags Details
dmesg (115.14 KB, text/plain)
2014-03-24 03:32 UTC, lu hua
no flags Details

Description lu hua 2014-02-14 03:02:26 UTC
Created attachment 94038 [details]
dmesg

System Environment:
--------------------------
Platform: Haswell
Kernel: (drm-intel-nightly)2f589112609b0a964b3d78c99c0f3a83ac16add6

Bug detailed description:
-----------------------------
New case pm_rps/reset fails on haswell,baytrail and broad well with -queued, -fixes and -nightly kernel. 

output:
IGT-Version: 1.5-g9597836 (x86_64) (Linux: 3.13.0_drm-intel-nightly_64fcdb_20140214+ x86_64)
Test assertion failure function matchit, file pm_rps.c:126:
Last errno: 2, No such file or directory
Failed assertion: freqs1[CUR] == freqs2[CUR]

Reproduce steps:
----------------------------
1. ./pm_rps --run-subtest reset
Comment 1 Daniel Vetter 2014-02-14 10:35:17 UTC
This is only fixed in dinq with

commit dd0a1aa19bd3d7203e58157b84cea78bbac605ac
Author: Jeff McGee <jeff.mcgee@intel.com>
Date:   Tue Feb 4 11:32:31 2014 -0600

    drm/i915: Restore rps/rc6 on reset

The impact of the bug is small, so no need to port to -fixes.
Comment 2 lu hua 2014-02-17 06:08:40 UTC
Test on -queued branch commit 4c0e552882114d1edb588242d45035246ab078a0.
It passes on the 1st and fails on the 2nd run.

output:
IGT-Version: 1.5-g9597836 (x86_64) (Linux: 3.13.0_drm-intel-next-queued_4c0e55_20140217+ x86_64)
Subtest reset: SUCCESS
[root@x-hsw27 tests]# ./pm_rps --run-subtest reset
IGT-Version: 1.5-g9597836 (x86_64) (Linux: 3.13.0_drm-intel-next-queued_4c0e55_20140217+ x86_64)
Test assertion failure function matchit, file pm_rps.c:126:
Last errno: 2, No such file or directory
Failed assertion: freqs1[CUR] == freqs2[CUR]
Subtest reset: FAIL
Comment 3 lu hua 2014-02-17 06:09:13 UTC
Created attachment 94189 [details]
dmesg on 4c0e55
Comment 4 Daniel Vetter 2014-03-03 17:25:47 UTC
The issue that the ERROR was printed at test enumeration time (e.g. when running piglit-print-commands.sh) is fixed in

commit 36106004e0e590557041e491fe690f01c89bc115
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Sun Jan 19 14:48:07 2014 +0100

    tests: Move pm_rps to the right Makefile target
Comment 5 lu hua 2014-03-24 03:32:01 UTC
It still fails on latest igt and -nightly kernel.
output:
IGT-Version: 1.6-g7a81094 (x86_64) (Linux: 3.14.0-rc7_drm-intel-nightly_ef4aa4_20140323+ x86_64)
Test assertion failure function matchit, file pm_rps.c:128:
Last errno: 2, No such file or directory
Failed assertion: freqs1[CUR] == freqs2[CUR]
error: 1250 == 200
Subtest reset: FAIL
Comment 6 lu hua 2014-03-24 03:32:20 UTC
Created attachment 96266 [details]
dmesg
Comment 7 Daniel Vetter 2014-03-26 20:54:13 UTC
Ok, a game of whack-a-mole. rps is just screwed up. The below patch makes things rather reliable on my ivb:

commit 669a2fca6c5ccd256d2d767107963840f7de438e
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Wed Mar 26 21:30:17 2014 +0100

    tests/pm_rps: quiescent harder
Comment 8 lu hua 2014-03-27 05:36:04 UTC
Verified.Fixed.
Comment 9 Elizabeth 2017-10-06 14:40: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.