Bug 26395 - [drm:edid_is_valid] *ERROR* Raw EDID:
Summary: [drm:edid_is_valid] *ERROR* Raw EDID:
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: XOrg git
Hardware: x86 (IA32) Linux (All)
: low normal
Assignee: ykzhao
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-02 11:47 UTC by Bartosz Fenski
Modified: 2017-02-06 12:10 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
dmesg output (53.10 KB, text/plain)
2010-02-02 11:47 UTC, Bartosz Fenski
no flags Details
vbios.dump (64.00 KB, application/octet-stream)
2010-02-02 11:49 UTC, Bartosz Fenski
no flags Details
dmesg with drm.debug=0x04 (74.23 KB, text/plain)
2010-02-08 00:05 UTC, Bartosz Fenski
no flags Details
Ignore the LVDS edid when it is unavailable or invalid (2.00 KB, patch)
2010-02-08 23:12 UTC, ykzhao
no flags Details | Splinter Review
try the debug patch that the LVDS edid when it is unavailable or invalid (1.97 KB, patch)
2010-02-22 00:31 UTC, ykzhao
no flags Details | Splinter Review

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.