Bug 26395

Summary: [drm:edid_is_valid] *ERROR* Raw EDID:
Product: DRI Reporter: Bartosz Fenski <fenio>
Component: DRM/IntelAssignee: ykzhao <yakui.zhao>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: low    
Version: XOrg git   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
dmesg output
none
vbios.dump
none
dmesg with drm.debug=0x04
none
Ignore the LVDS edid when it is unavailable or invalid
none
try the debug patch that the LVDS edid when it is unavailable or invalid none

Description Bartosz Fenski 2010-02-02 11:47:06 UTC
Created attachment 33012 [details]
dmesg output

I found similar error in this thread: http://marc.info/?l=linux-kernel&m=125506581323966&w=2

So I'm filling wanted outputs as attachments. Hope that helps.
Comment 1 Bartosz Fenski 2010-02-02 11:49:27 UTC
Created attachment 33013 [details]
vbios.dump
Comment 2 Jesse Barnes 2010-02-05 14:58:01 UTC
Not sure what the bug is here, reassigning to Yakui.  Invalid EDIDs can be due to bad timing on the kernel side, flakey hardware or cables, or a real bad EDID in the monitor.
Comment 3 Bartosz Fenski 2010-02-06 03:06:59 UTC
It's notebook Sony Vaio VGN-C2S_L, so I suppose it's bad EDID, and not cables ;)
Comment 4 ykzhao 2010-02-07 19:09:32 UTC
(In reply to comment #3)
> It's notebook Sony Vaio VGN-C2S_L, so I suppose it's bad EDID, and not cables
> ;)

Will you please add the boot option of "drm.debug=0x04" and attach the output of dmesg?

Thanks.
   Yakui
> 

Comment 5 Bartosz Fenski 2010-02-08 00:05:07 UTC
Created attachment 33144 [details]
dmesg with drm.debug=0x04

Here goes dmesg with drm.debug=0x04 option.
Comment 6 ykzhao 2010-02-08 23:12:03 UTC
Created attachment 33189 [details] [review]
Ignore the LVDS edid when it is unavailable or invalid

Will you please try the attached patch and see whether it frequently reports the following similar message?
     >i915 0000:00:02.0: LVDS-1: EDID invalid.

thanks.
Comment 7 Bartosz Fenski 2010-02-09 02:30:48 UTC
Now it only reports:

[   23.903220] [drm] Initialized drm 1.1.0 20060810
[   23.908682] i915: no symbol version for module_layout
[   23.932410] i915: no symbol version for module_layout

(no symbol version probably due to my way of compilation).

Does this ignoring LVDS can make some problems or is it safe?
Comment 8 Bartosz Fenski 2010-02-09 09:32:13 UTC
Since that change my X.org eats many CPU, and everything is working extremely slow. 

At least I'm finally able to run Enemy-territory game, but it works so slow, that I have to try to select proper option to be able to exit from the game.
Before that patch I wasn't able to even run it (signal 11).

Seems there's something wrong with hardware acceleration after that patch.

Is there something else I can provide you to resolve these issues?

regards
fEnIo
Comment 9 ykzhao 2010-02-22 00:31:08 UTC
Created attachment 33484 [details] [review]
try the debug patch that the LVDS edid when it is unavailable or invalid

will you please try the updated patch and see whether it can work for you?

thanks.
Comment 10 ykzhao 2010-02-22 00:53:51 UTC
(In reply to comment #8)
> Since that change my X.org eats many CPU, and everything is working extremely
> slow. 
> 
> At least I'm finally able to run Enemy-territory game, but it works so slow,
> that I have to try to select proper option to be able to exit from the game.
> Before that patch I wasn't able to even run it (signal 11).

It seems that the slow issue is not related with this bug. And from the log it seems that the xorg will query the status of every possible outputs periodically.

Can you provide the info of "cat /proc/interrupts"?

Thanks.
   
> 
> Seems there's something wrong with hardware acceleration after that patch.
> 
> Is there something else I can provide you to resolve these issues?
> 
> regards
> fEnIo
> 

Comment 11 ykzhao 2010-04-11 01:01:51 UTC
Now the corresponding patch is already shipped. And after the patch is applied, it can avoid the warning message of "drm:edid_is_valid" perodically.

commit bfac4d6725baacbfc085c38e231b8582a1b8f62b
Author: Zhao Yakui <yakui.zhao@intel.com>
Date:   Wed Apr 7 17:11:22 2010 +0800
 
    drm/i915: Ignore LVDS EDID when it is unavailabe or invalid

So this bug will be marked as resolved.

Thanks.
Comment 12 Jari Tahvanainen 2017-02-06 12:10:46 UTC
Closing (>6 years) old Resolved+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.