Re: [4.4-rc1][Regression] drm/i915: Check live status before reading edid

From: Jindal, Sonika
Date: Fri Feb 26 2016 - 01:09:03 EST

On 2/26/2016 12:11 AM, Joseph Salisbury wrote:
On 02/24/2016 10:53 PM, Jindal, Sonika wrote:
Hi Joe,

Yes, first thing to try is to increase the tries.
We testing with 300 retries, but the second monitor still did not show
up. However, it did show up in lspci.

Can you please point me to the bug and provide more details like platform, monitor, cable.
The bug is at: . All the hardware details should
be in the bug report. The cable is a single link dvi-d cable.
Unfortunately the bug reporter does not have a dual link cable to test.
If you need any additional info, we can ask the bug reporter.
If this is with single link cable, the issue could be the same.
As Ville suggested for the other issue to use video=HDMI-A-1:e as command line argument, can you please give it a try?
The logs shared in the bug doesn't have drm logs enabled, so couldnt get much out of it.
Which platform is this?

Alternatively you can add something like following in intel_hdmi_detect to make it ignore the live status checks.

@@ -1419,7 +1419,7 @@ intel_hdmi_detect(struct drm_connector *connector, bool force)

+ live_status = live_status | force;
if (intel_hdmi_set_edid(connector, live_status)) {
struct intel_hdmi *intel_hdmi = intel_attached_hdmi(connector);

Are you referring to the same issue as Oleksandr reported where a single link dvi/hdmi cable didnât work and dual link worked?
I'm not sure if this is the exact issue or not. I'll review the other
thread and compare.


-----Original Message-----
From: Joseph Salisbury [mailto:joseph.salisbury@xxxxxxxxxxxxx]
Sent: Thursday, February 25, 2016 3:09 AM
To: Jindal, Sonika <sonika.jindal@xxxxxxxxx>
Cc: Sharma, Shashank <shashank.sharma@xxxxxxxxx>; Vivi, Rodrigo <rodrigo.vivi@xxxxxxxxx>; Daniel Vetter <daniel.vetter@xxxxxxxx>; Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx>; David Airlie <airlied@xxxxxxxx>; intel-gfx <intel-gfx@xxxxxxxxxxxxxxxxxxxxx>; dri-devel <dri-devel@xxxxxxxxxxxxxxxxxxxxx>; LKML <linux-kernel@xxxxxxxxxxxxxxx>
Subject: [4.4-rc1][Regression] drm/i915: Check live status before reading edid

Hi Sonika,

A kernel bug report was opened against Ubuntu [0]. After a kernel bisect, it was found that reverting the following commit resolved this bug:

commit 237ed86c693d8a8e4db476976aeb30df4deac74b
Author: Sonika Jindal <sonika.jindal@xxxxxxxxx>
Date: Tue Sep 15 09:44:20 2015 +0530

drm/i915: Check live status before reading edid

The regression was introduced as of v4.4-rc1.

I was hoping to get your feedback, since you are the patch author. Do think increasing the number of tries in intel_hdmi_detect() is worth trying? Do you think gathering any additional data will help diagnose this issue, or would it be best to submit a revert request?