Bug 99929 - [i915] Black/gray screen with kernel 4.9.3
Summary: [i915] Black/gray screen with kernel 4.9.3
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: XOrg git
Hardware: x86-64 (AMD64) Linux (All)
: highest critical
Assignee: Ville Syrjala
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: PatchSubmitted
Keywords: bisected, regression
Depends on:
Blocks:
 
Reported: 2017-02-23 18:55 UTC by Anders Blomdell
Modified: 2017-05-24 11:16 UTC (History)
2 users (show)

See Also:
i915 platform: SKL
i915 features: display/watermark


Attachments
dmesg of failed graphics (241.03 KB, text/plain)
2017-05-12 11:28 UTC, Anders Blomdell
no flags Details
dmesg of successful graphics (127.17 KB, text/plain)
2017-05-12 11:29 UTC, Anders Blomdell
no flags Details
opregion of failed graphics (8.00 KB, application/octet-stream)
2017-05-12 11:31 UTC, Anders Blomdell
no flags Details
opregion of successful graphics (8.00 KB, application/octet-stream)
2017-05-12 11:32 UTC, Anders Blomdell
no flags Details

Description Anders Blomdell 2017-02-23 18:55:43 UTC
For further info see https://bugzilla.redhat.com/show_bug.cgi?id=1398362


When booting 4.9.3 kernel , gdm starts with a black/gray screen. Some bisecting gave the results below.

I found that 4.9.3 kernel (from linux-stable) after reverting commit 0ce140d45a8398b501934ac289aef0eb7f47c596, the problem is fixed. Relevant output from the logs (4.9.3-1-g17a6527 is the one with the reverted patch):


Command line: BOOT_IMAGE=/vmlinuz-4.9.3-1-g17a6527+ root=/dev/mapper/vg0-root ro rd.auto=1

Command line: BOOT_IMAGE=/vmlinuz-4.9.3 root=/dev/mapper/vg0-root ro rd.auto=1
[drm:intel_cpu_fifo_underrun_irq_handler [i915]] *ERROR* CPU pipe A FIFO underrun
Comment 1 Jani Nikula 2017-02-27 13:00:27 UTC
(In reply to Anders Blomdell from comment #0)
> I found that 4.9.3 kernel (from linux-stable) after reverting commit
> 0ce140d45a8398b501934ac289aef0eb7f47c596, the problem is fixed. Relevant
> output from the logs (4.9.3-1-g17a6527 is the one with the reverted patch):

The bad commit is

commit 0ce140d45a8398b501934ac289aef0eb7f47c596
Author: Ville Syrjälä <ville.syrjala@linux.intel.com>
Date:   Tue Oct 11 20:52:47 2016 +0300

    drm/i915: Clean up DDI DDC/AUX CH sanitation
Comment 2 Ville Syrjala 2017-04-12 18:30:16 UTC
Please attach full dmesg with drm.debug=0xe for both the good and bad kernels. Also a copy of /sys/kernel/debug/dri/0/i915_opregion would be nice.
Comment 3 Jani Saarinen 2017-05-12 07:43:36 UTC
Hi, any comments from reporter, we are closing if no feedback received in a month.
Comment 4 Anders Blomdell 2017-05-12 11:28:00 UTC
Created attachment 131331 [details]
dmesg of failed graphics
Comment 5 Anders Blomdell 2017-05-12 11:29:06 UTC
Created attachment 131332 [details]
dmesg of successful graphics
Comment 6 Anders Blomdell 2017-05-12 11:31:36 UTC
Created attachment 131333 [details]
opregion of failed graphics
Comment 7 Anders Blomdell 2017-05-12 11:32:29 UTC
Created attachment 131334 [details]
opregion of successful graphics
Comment 8 Anders Blomdell 2017-05-12 11:35:01 UTC
Failed version is 4.8.15
Successful version is 4.9.10
Comment 9 Ville Syrjala 2017-05-12 12:16:31 UTC
I suspect this has been fixed by commit bb1d132935c2 ("drm/i915/vbt: split out defaults that are set when there is no VBT")

Can you confirm?
Comment 10 Anders Blomdell 2017-05-12 12:35:27 UTC
(In reply to Ville Syrjala from comment #9)
> I suspect this has been fixed by commit bb1d132935c2 ("drm/i915/vbt: split
> out defaults that are set when there is no VBT")
> 
> Can you confirm?

Which git-repo?
Comment 11 Ville Syrjala 2017-05-12 12:53:43 UTC
(In reply to Anders Blomdell from comment #10)
> (In reply to Ville Syrjala from comment #9)
> > I suspect this has been fixed by commit bb1d132935c2 ("drm/i915/vbt: split
> > out defaults that are set when there is no VBT")
> > 
> > Can you confirm?
> 
> Which git-repo?

It should be in Linus's tree by now.
Comment 12 Jani Saarinen 2017-05-22 10:04:42 UTC
Reporter, ca you report back?
Comment 13 Anders Blomdell 2017-05-22 14:33:43 UTC
commit bb1d132935c2 works!

Thanks!
Comment 14 Ricardo 2017-05-23 19:44:31 UTC
confirmed as resolved... need to test again once patch is incorporated
Comment 15 Jari Tahvanainen 2017-05-24 11:16:40 UTC
Closing since code is on drm-tip.

commit bb1d132935c2f87cd261eb559759fe49d5e5dc43
Author:     Jani Nikula <jani.nikula@intel.com>
AuthorDate: Fri Mar 10 15:27:58 2017 +0200
Commit:     Jani Nikula <jani.nikula@intel.com>
CommitDate: Tue Mar 14 10:49:13 2017 +0200
    drm/i915/vbt: split out defaults that are set when there is no VBT

AND

commit 9301397a63b3bf1090dffe846c6f1c8efa032236
Author:     Manasi Navare <manasi.d.navare@intel.com>
AuthorDate: Thu Apr 6 16:44:19 2017 +0300
Commit:     Jani Nikula <jani.nikula@intel.com>
CommitDate: Thu Apr 13 21:57:37 2017 +0300
    drm/i915: Implement Link Rate fallback on Link training failure


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.