Bug 194770

Summary: [DRM][PATCH] constant stream of EDID/Connector messages from DRM
Product: Base System Reporter: Larry Rosenman <ler>
Component: kernAssignee: freebsd-bugs (Nobody) <bugs>
Status: Closed FIXED    
Severity: Affects Only Me CC: dumbbell
Priority: ---    
Version: CURRENT   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch to move the messages under DRM_DEBUG_KMS
none
one more message to move to debug none

Description Larry Rosenman freebsd_committer freebsd_triage 2014-11-02 19:54:22 UTC
Created attachment 148959 [details]
patch to move the messages under DRM_DEBUG_KMS

Getting a constant stream of:
Oct 31 06:53:25 borg kernel: error: [drm:pid0:drm_do_get_edid] *ERROR* DVI-I-1:
+EDID block 0 invalid.
Oct 31 06:53:25 borg kernel: error: [drm:pid0:radeon_dvi_detect] *ERROR*
+DVI-I-1: probed a monitor but no|invalid EDID
Oct 31 06:53:25 borg kernel: error: [drm:pid0:drm_do_get_edid] *ERROR* DVI-I-1:
+EDID block 0 invalid.
Oct 31 06:53:25 borg kernel: error: [drm:pid0:radeon_dvi_detect] *ERROR*
+DVI-I-1: probed a monitor but no|invalid EDID
Oct 31 06:53:25 borg kernel: error: [drm:pid0:drm_do_get_edid] *ERROR* DVI-I-1:
+EDID block 0 invalid.

dumbbell@ suggested commenting them out, but I think we should patch it so that it's under debug. 

That's what this patch does.
Comment 1 Larry Rosenman freebsd_committer freebsd_triage 2014-11-15 04:17:01 UTC
Created attachment 149429 [details]
one more message to move to debug

All but this message have been handled....
Comment 2 Larry Rosenman freebsd_committer freebsd_triage 2014-11-15 16:54:01 UTC
Jean-Sebastien,
   can you add this?
Comment 3 commit-hook freebsd_committer freebsd_triage 2014-11-16 17:54:07 UTC
A commit references this bug:

Author: dumbbell
Date: Sun Nov 16 17:53:48 UTC 2014
New revision: 274587
URL: https://svnweb.freebsd.org/changeset/base/274587

Log:
  drm/radeon: Lower priority of a message related to invalid EDID

  Like in r259717, the prority goes from "error" to "debug" to avoid
  spamming logs when the connectors are polled.

  PR:		194770
  Submitted by:	Larry Rosenman <ler@lerctr.org>
  MFC after:	1 week

Changes:
  head/sys/dev/drm2/radeon/radeon_connectors.c
Comment 4 Jean-Sébastien Pédron freebsd_committer freebsd_triage 2014-11-16 18:13:27 UTC
Committed, thank you Larry!
Comment 5 commit-hook freebsd_committer freebsd_triage 2014-11-22 17:20:25 UTC
A commit references this bug:

Author: dumbbell
Date: Sat Nov 22 17:19:40 UTC 2014
New revision: 274862
URL: https://svnweb.freebsd.org/changeset/base/274862

Log:
  drm: Lower priority of three messages related to invalid EDID

  Like in r259717, the prority goes from "error" to "debug" to avoid
  spamming logs when the connectors are polled.

  PR:		194770
  Submitted by:	Larry Rosenman <ler@lerctr.org>
  MFC of:		r273962, r274587

Changes:
_U  stable/10/
  stable/10/sys/dev/drm2/drm_edid.c
  stable/10/sys/dev/drm2/radeon/radeon_connectors.c