Summary: | [BYT Bisected]igt/kms_mmio_vs_cs_flip/setcrtc_vs_cs_flip causes system hang | ||||||
---|---|---|---|---|---|---|---|
Product: | DRI | Reporter: | Guo Jinxian <jinxianx.guo> | ||||
Component: | DRM/Intel | Assignee: | Dave Airlie <airlied> | ||||
Status: | CLOSED FIXED | QA Contact: | Intel GFX Bugs mailing list <intel-gfx-bugs> | ||||
Severity: | critical | ||||||
Priority: | highest | CC: | intel-gfx-bugs, ming.yao | ||||
Version: | unspecified | ||||||
Hardware: | Other | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
i915 platform: | i915 features: | ||||||
Attachments: |
|
Description
Guo Jinxian
2014-07-11 03:13:39 UTC
Created attachment 103376 [details]
dmesg
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) Signed-off-by: Dave Airlie <airlied@redhat.com> Reviewed-by: Todd Previte <tprevite@gmail.com> [danvet: Fix conflicts in i915_irq.c with Oscar Mateo's irq handling race fixes and a trivial one in intel_drv.h with the psr code.] Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> :040000 040000 7be50985e66c24e64d4de3cccbdcd521ef59903b 8e9cf7373e1c96e78989fd0514e8da8debaad1c4 M drivers The commit unable to revert. It's parents commit(564ed191f5d816d24501664296991ec70327e2bc) unable to causes system hang. 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 (In reply to comment #3) > 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. Verified on latest -nightly(186631131a9289dad22f51315d78b9b6ac5b425f) root@x-byt06:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./kms_mmio_vs_cs_flip --run-subtest setcrtc_vs_cs_flip IGT-Version: 1.7-g5c7bcb1 (x86_64) (Linux: 3.16.0_drm-intel-nightly_186631_20140818+ x86_64) Subtest setcrtc_vs_cs_flip: SUCCESS 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.