Bug 81200

Summary: [BYT Bisected]igt/pm_rpm/debugfs-forcewake-user causes system hang
Product: DRI Reporter: Guo Jinxian <jinxianx.guo>
Component: DRM/IntelAssignee: Daniel Vetter <daniel>
Status: CLOSED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: critical    
Priority: highest CC: intel-gfx-bugs, ming.yao
Version: unspecifiedKeywords: bisected
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
dmesg none

Description Guo Jinxian 2014-07-11 03:09:06 UTC
==System Environment==
--------------------------
Regression: Yes. 
Good commit on -next-queued: 57127efaf1adc5e0bb2d3fa5a01bd97f71af7836

Non-working platforms: BYT

==kernel==
--------------------------
origin/drm-intel-nightly: 16025dad8e9964a5810385f755d43f1c48d6fdcc(fails)
    drm-intel-nightly: 2014y-07m-09d-14h-17m-14s integration manifest
origin/drm-intel-next-queued: d101c8fe9bda6578ae72d6021415cfaad2b422f0(fails)
    drm/i915: remove i915_rstdby_delays debugfs entry
origin/drm-intel-fixes: 01527b3127997ef6370d5ad4fa25d96847fbf12a(works)
    drm/i915/vlv: T12 eDP panel timing enforcement during reboot

==Bug detailed description==
igt/pm_rpm/debugfs-forcewake-user causes system hang. 

We can not get dmesg because system hang.


Output:
root@x-byt06:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./pm_rpm --run-subtest debugfs-forcewake-user
IGT-Version: 1.7-g6b986a2 (x86_64) (Linux: 3.16.0-rc4_drm-intel-nightly_140ef1_20140711+ x86_64)
Runtime PM support: 1
PC8 residency support: 0



==Reproduce steps==
---------------------------- 
1. ./pm_rpm --run-subtest debugfs-forcewake-user
Comment 1 Guo Jinxian 2014-07-18 06:04:22 UTC
Case igt/pm_rpm/drm-resources-equal is able to reproduce this issue on latest -nightly(f326cd0fae7e74011d3c0a8bd2be2cd2e2a45407)
Comment 2 lu hua 2014-07-24 05:42:05 UTC
Created attachment 103375 [details]
dmesg
Comment 3 liulei 2014-07-30 08:33:13 UTC
==Bisect results==
----------------------------
Bisect shows: 13cf550448b58abf8f44f5d6a560f2d20871c965 is the first bad commit
commit 13cf550448b58abf8f44f5d6a560f2d20871c965
Author:     Dave Airlie <airlied@redhat.com>
AuthorDate: Wed Jun 18 11:29:35 2014 +1000
Commit:     Daniel Vetter <daniel.vetter@ffwll.ch>
CommitDate: Mon Jul 7 15:08:51 2014 +0200

    drm/i915: rework digital port IRQ handling (v2)

    The digital ports from Ironlake and up have the ability to distinguish
    between long and short HPD pulses. Displayport 1.1 only uses the short
    form to request link retraining usually, so we haven't really needed
    support for it until now.

    However with DP 1.2 MST we need to handle the short irqs on their
    own outside the modesetting locking the long hpd's involve. This
    patch adds the framework to distinguish between short/long to the
    current code base, to lay the basis for future DP 1.2 MST work.

    This should mean we get better bisectability in case of regression
    due to the new irq handling.

    v2: add GM45 support (untested, due to lack of hw)
Comment 4 Chris Wilson 2014-08-11 05:44:25 UTC
commit f68d697eaf3278200a7fc3c8b1d95d72837b84d8
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Mon Aug 4 07:15:09 2014 +0100

    drm/i915: only hook up hpd pulse for DP outputs
Comment 5 yaoming 2014-08-15 05:39:40 UTC
(In reply to comment #4)
> commit f68d697eaf3278200a7fc3c8b1d95d72837b84d8
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Mon Aug 4 07:15:09 2014 +0100
> 
>     drm/i915: only hook up hpd pulse for DP outputs

It also causes system hang.

[root@x-byt01 ~]# cd /GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests
[root@x-byt01 tests]# ./pm_rpm --run-subtest debugfs-forcewake-user
IGT-Version: 1.7-g5c7bcb1 (x86_64) (Linux: 3.16.0-rc4_ym_f68d69_20140815+ x86_64)
Runtime PM support: 1
PC8 residency support: 0
Comment 6 Guo Jinxian 2014-08-18 05:54:29 UTC
Verified on latest -nightly(186631131a9289dad22f51315d78b9b6ac5b425f)

root@x-byt06:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./pm_rpm --run-subtest debugfs-forcewake-user
IGT-Version: 1.7-g5c7bcb1 (x86_64) (Linux: 3.16.0_drm-intel-nightly_186631_20140818+ x86_64)
Runtime PM support: 1
PC8 residency support: 0
Subtest debugfs-forcewake-user: SUCCESS
Comment 7 Jari Tahvanainen 2016-10-19 11:08:28 UTC
Closing verified+fixed.

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.