Bug 88097 - [BDW Bisected]igt/pm_rpm/system-suspend fails at the 2nd cycle when execlists enabled
Summary: [BDW Bisected]igt/pm_rpm/system-suspend fails at the 2nd cycle when execlists...
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: high normal
Assignee: Thomas Daniel
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-06 09:03 UTC by lu hua
Modified: 2017-10-06 14:32 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (124.83 KB, text/plain)
2015-01-06 09:03 UTC, lu hua
no flags Details

Description lu hua 2015-01-06 09:03:00 UTC
Created attachment 111834 [details]
dmesg

==System Environment==
--------------------------
Regression: Yes

Non-working platforms:  BDW

==kernel==
--------------------------
drm-intel-next-queued/27401d126b5b1c8dd4df98bbb60b09ff2b3d5e60

==Bug detailed description==
-----------------------------
It fails at the 2nd cycle on BDW with -nightly and -queued kernel(We disabled this case in regular testing due to system lost IP after suspend,pls refer to the blacklist).
Bisect shows: 27401d126b5b1c8dd4df98bbb60b09ff2b3d5e60 is the first bad commit
commit 27401d126b5b1c8dd4df98bbb60b09ff2b3d5e60
Author:     Thomas Daniel <thomas.daniel@intel.com>
AuthorDate: Thu Dec 11 12:48:35 2014 +0000
Commit:     Daniel Vetter <daniel.vetter@ffwll.ch>
CommitDate: Mon Dec 15 11:25:28 2014 +0100

    drm/i915/bdw: Enable execlists by default where supported

    Execlist support in the i915 driver is now considered good enough for the
    feature to be enabled by default on Gen8 and later and routinely tested.
    Adjusted i915 parameters structure initialization to reflect this and updated
    the comment in intel_sanitize_enable_execlists().

    There's still work to do before we can let the wider massive onto it,
    but there's still time left before the 3.20 cutoff.

    v2: Update the MODULE_PARM_DESC too.

    Issue: VIZ-2020
    Signed-off-by: Thomas Daniel <thomas.daniel@intel.com>
    [danvet: Add note that there's still some work left to do.]
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

output:
rtcwake: wakeup from "mem" using /dev/rtc0 at Mon Jan  1 23:46:19 2001
IGT-Version: 1.9-gcb512b6 (x86_64) (Linux: 3.18.0-rc7_kcloud_27401d_20141219+ x86_64)
Runtime PM support: 1
PC8 residency support: 1
Test assertion failure function system_suspend_subtest, file pm_rpm.c:1398:
Failed assertion: wait_for_suspended()
Last errno: 2, No such file or directory
Subtest system-suspend: FAIL (13.438s)

==Reproduce steps==
---------------------------- 
1.
Comment 1 Daniel Vetter 2015-01-08 07:57:50 UTC
commit 9ed67ab76f24ec1bf9b0414e3c8ac39c843f4d55
Author: Thomas Daniel <thomas.daniel@intel.com>
Date:   Wed Jan 7 16:09:30 2015 +0000

    drm/i915: Reset CSB read pointer when enabling contexts

... should fix this.
Comment 2 Daniel Vetter 2015-01-08 08:00:19 UTC
Strike that, please instead test this patch

http://lists.freedesktop.org/archives/intel-gfx/2015-January/058125.html
Comment 3 fangxun 2015-01-09 05:07:34 UTC
It passes with the patch.

output:
Runtime PM support: 1
PC8 residency support: 1
rtcwake: assuming RTC uses UTC ...
rtcwake: wakeup from "mem" using /dev/rtc0 at Sat Jan 17 18:33:02 2015
Subtest system-suspend: SUCCESS (3.635s)
Comment 4 Chris Harris 2015-01-15 09:50:11 UTC
The patch is now merged in nightly - http://cgit.freedesktop.org/drm-intel/commit/?id=c0a03a2e4c4e954d9acffd3ce3521e64654c9dc8

Can this defect now be closed?
Comment 5 fangxun 2015-01-16 05:02:38 UTC
It passes on latest nightly branch kernel.
Comment 6 Elizabeth 2017-10-06 14:32:34 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.