drm/i915: Add detection of changing of edid on between suspend and resume

Submitted by Mun, Gwan-gyeong on Aug. 3, 2018, 4:34 p.m.

Details

Reviewer None
Submitted Aug. 3, 2018, 4:34 p.m.
Last Updated Aug. 9, 2018, 3:02 p.m.
Revision 3

Revisions

Patches download mbox

Tests

= CI Bug Log - changes from CI_DRM_4613 -> Patchwork_9848 =

== Summary - FAILURE ==

  Serious unknown changes coming with Patchwork_9848 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_9848, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/47680/revisions/1/mbox/

== Possible new issues ==

  Here are the unknown changes that may have been introduced in Patchwork_9848:

  === IGT changes ===

    ==== Possible regressions ====

    igt@drv_hangman@error-state-basic:
      fi-pnv-d510:        PASS -> DMESG-WARN
      fi-blb-e6850:       PASS -> DMESG-WARN
      fi-bwr-2160:        PASS -> DMESG-WARN

    igt@gem_exec_suspend@basic-s3:
      fi-ilk-650:         PASS -> DMESG-WARN
      fi-skl-guc:         PASS -> DMESG-WARN
      fi-pnv-d510:        PASS -> INCOMPLETE
      fi-hsw-4770:        PASS -> DMESG-WARN
      fi-bxt-j4205:       PASS -> DMESG-WARN
      fi-ivb-3770:        PASS -> DMESG-WARN
      fi-skl-6700k2:      PASS -> DMESG-WARN
      fi-blb-e6850:       PASS -> INCOMPLETE
      fi-hsw-4770r:       PASS -> DMESG-WARN

    igt@gem_exec_suspend@basic-s4-devices:
      fi-hsw-4770:        PASS -> INCOMPLETE
      fi-ilk-650:         PASS -> INCOMPLETE
      fi-hsw-4770r:       PASS -> INCOMPLETE

    igt@kms_addfb_basic@invalid-get-prop:
      fi-bwr-2160:        PASS -> INCOMPLETE

    
== Known issues ==

  Here are the changes found in Patchwork_9848 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@drv_selftest@live_workarounds:
      {fi-cfl-8109u}:     PASS -> DMESG-FAIL (fdo#107292)

    igt@gem_exec_suspend@basic-s4-devices:
      fi-skl-6700k2:      PASS -> INCOMPLETE (k.org#199541, fdo#105524, fdo#104108)
      fi-bxt-j4205:       PASS -> INCOMPLETE (fdo#103927)
      fi-ivb-3770:        PASS -> INCOMPLETE (fdo#106220)
      fi-skl-guc:         PASS -> INCOMPLETE (fdo#106693, fdo#104108)

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
      fi-snb-2520m:       PASS -> INCOMPLETE (fdo#103713)

    
    ==== Possible fixes ====

    igt@drv_selftest@live_workarounds:
      fi-bdw-5557u:       DMESG-FAIL (fdo#107292) -> PASS

    
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
  fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
  fdo#104108 https://bugs.freedesktop.org/show_bug.cgi?id=104108
  fdo#105524 https://bugs.freedesktop.org/show_bug.cgi?id=105524
  fdo#106220 https://bugs.freedesktop.org/show_bug.cgi?id=106220
  fdo#106693 https://bugs.freedesktop.org/show_bug.cgi?id=106693
  fdo#107292 https://bugs.freedesktop.org/show_bug.cgi?id=107292
  k.org#199541 https://bugzilla.kernel.org/show_bug.cgi?id=199541


== Participating hosts (52 -> 48) ==

  Additional (1): fi-glk-j4005 
  Missing    (5): fi-ctg-p8600 fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-hsw-4200u 


== Build changes ==

    * Linux: CI_DRM_4613 -> Patchwork_9848

  CI_DRM_4613: e3dd88c4ffdcda9f36bdb43cbdd7eaa605002055 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4587: 5d78c73d871525ec9caecd88ad7d9abe36637314 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_9848: 3d56929672784c517c804405c6f4d6a9bab546a4 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

3d5692967278 drm/i915: Add detection of changing of edid on between suspend and resume
$ dim checkpatch origin/drm-tip
3d5692967278 drm/i915: Add detection of changing of edid on between suspend and resume
-:7: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#7: 
The hotplug detection routine of i915 uses drm_helper_hpd_irq_event(). This helper

-:89: WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#89: FILE: drivers/gpu/drm/i915/intel_hotplug.c:566:
+		 * as "connector_status_connected". */

total: 0 errors, 2 warnings, 0 checks, 92 lines checked
$ dim sparse origin/drm-tip
Commit: drm/i915: Add detection of changing of edid on between suspend and resume
Okay!

Patches download mbox

Tests

= CI Bug Log - changes from CI_DRM_4636 -> Patchwork_9899 =

== Summary - FAILURE ==

  Serious unknown changes coming with Patchwork_9899 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_9899, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/47680/revisions/2/mbox/

== Possible new issues ==

  Here are the unknown changes that may have been introduced in Patchwork_9899:

  === IGT changes ===

    ==== Possible regressions ====

    igt@drv_module_reload@basic-reload-inject:
      fi-bsw-n3050:       PASS -> DMESG-WARN

    igt@gem_exec_suspend@basic-s3:
      {fi-kbl-soraka}:    NOTRUN -> INCOMPLETE

    
== Known issues ==

  Here are the changes found in Patchwork_9899 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@drv_module_reload@basic-reload-inject:
      fi-hsw-4770r:       PASS -> DMESG-WARN (fdo#107425)

    igt@drv_selftest@live_hangcheck:
      fi-kbl-7560u:       NOTRUN -> DMESG-FAIL (fdo#106947, fdo#106560)

    igt@drv_selftest@live_workarounds:
      fi-skl-6700hq:      PASS -> DMESG-FAIL (fdo#107292)

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
      {fi-byt-clapper}:   PASS -> FAIL (fdo#107362, fdo#103191)

    igt@prime_vgem@basic-fence-flip:
      fi-ilk-650:         PASS -> FAIL (fdo#104008)

    
    ==== Warnings ====

    {igt@kms_psr@primary_page_flip}:
      fi-cnl-psr:         DMESG-WARN (fdo#107372) -> DMESG-FAIL (fdo#107372)

    
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
  fdo#104008 https://bugs.freedesktop.org/show_bug.cgi?id=104008
  fdo#106560 https://bugs.freedesktop.org/show_bug.cgi?id=106560
  fdo#106947 https://bugs.freedesktop.org/show_bug.cgi?id=106947
  fdo#107292 https://bugs.freedesktop.org/show_bug.cgi?id=107292
  fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
  fdo#107372 https://bugs.freedesktop.org/show_bug.cgi?id=107372
  fdo#107425 https://bugs.freedesktop.org/show_bug.cgi?id=107425


== Participating hosts (50 -> 47) ==

  Additional (2): fi-kbl-soraka fi-kbl-7560u 
  Missing    (5): fi-skl-guc fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-hsw-4200u 


== Build changes ==

    * Linux: CI_DRM_4636 -> Patchwork_9899

  CI_DRM_4636: 084bb2fb549650b6da80976c9bc594779ce342b4 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4590: e6ddaca7a8ea9d3d27f0ecaa36b357cc02e2df3b @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_9899: f7f78bf703682167942c440f7cc9cf19e86c95cd @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

f7f78bf70368 drm/i915: Add detection of changing of edid on between suspend and resume
$ dim checkpatch origin/drm-tip
93959254664e drm/i915: Add detection of changing of edid on between suspend and resume
-:8: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#8: 
helper can detect changing of status of connector, but it can not detect changing

-:96: WARNING:LINE_SPACING: Missing a blank line after declarations
#96: FILE: drivers/gpu/drm/i915/intel_hotplug.c:571:
+			struct edid *cur_edid = to_intel_connector(connector)->detect_edid;
+			if (!old_edid || !cur_edid)

total: 0 errors, 2 warnings, 0 checks, 95 lines checked
$ dim sparse origin/drm-tip
Commit: drm/i915: Add detection of changing of edid on between suspend and resume
Okay!

Patches download mbox

Tests

= CI Bug Log - changes from CI_DRM_4639 -> Patchwork_9909 =

== Summary - FAILURE ==

  Serious unknown changes coming with Patchwork_9909 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_9909, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/47680/revisions/3/mbox/

== Possible new issues ==

  Here are the unknown changes that may have been introduced in Patchwork_9909:

  === IGT changes ===

    ==== Possible regressions ====

    igt@drv_module_reload@basic-reload-inject:
      fi-byt-j1900:       PASS -> DMESG-WARN
      fi-bsw-n3050:       PASS -> DMESG-WARN

    
== Known issues ==

  Here are the changes found in Patchwork_9909 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@debugfs_test@read_all_entries:
      fi-snb-2520m:       PASS -> INCOMPLETE (fdo#103713)

    igt@drv_selftest@live_coherency:
      fi-gdg-551:         NOTRUN -> DMESG-FAIL (fdo#107164)

    igt@drv_selftest@live_requests:
      {fi-bsw-kefka}:     PASS -> INCOMPLETE (fdo#105876)

    igt@drv_selftest@live_workarounds:
      fi-whl-u:           PASS -> DMESG-FAIL (fdo#107292)

    igt@kms_frontbuffer_tracking@basic:
      {fi-byt-clapper}:   PASS -> FAIL (fdo#103167)

    {igt@kms_psr@primary_mmap_gtt}:
      fi-cnl-psr:         PASS -> DMESG-WARN (fdo#107372)

    
    ==== Possible fixes ====

    {igt@amdgpu/amd_prime@amd-to-i915}:
      fi-bxt-j4205:       INCOMPLETE (fdo#103927) -> SKIP

    {igt@amdgpu/amd_prime@i915-to-amd}:
      fi-bxt-j4205:       DMESG-FAIL -> SKIP

    igt@kms_pipe_crc_basic@read-crc-pipe-a:
      {fi-byt-clapper}:   FAIL (fdo#107362) -> PASS

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
      fi-bxt-dsi:         INCOMPLETE (fdo#103927) -> PASS

    
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
  fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
  fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
  fdo#105876 https://bugs.freedesktop.org/show_bug.cgi?id=105876
  fdo#107164 https://bugs.freedesktop.org/show_bug.cgi?id=107164
  fdo#107292 https://bugs.freedesktop.org/show_bug.cgi?id=107292
  fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
  fdo#107372 https://bugs.freedesktop.org/show_bug.cgi?id=107372


== Participating hosts (53 -> 49) ==

  Additional (1): fi-gdg-551 
  Missing    (5): fi-ctg-p8600 fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-hsw-4200u 


== Build changes ==

    * Linux: CI_DRM_4639 -> Patchwork_9909

  CI_DRM_4639: da34c4841d4bd5098cef0bd3ddaeed1ee3eb3103 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4591: 6cb3d7dbe5831a7b2b5b7a4638d8a8b7ac624f5f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_9909: b5655d96db430792733958ab2a84e12a85f3584d @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

b5655d96db43 drm/i915: Add detection of changing of edid on between suspend and resume
$ dim checkpatch origin/drm-tip
8c821c599a13 drm/i915: Add detection of changing of edid on between suspend and resume
-:15: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#15: 
 3) unplug 1)'s display device and plug the other display device to a connector

total: 0 errors, 1 warnings, 0 checks, 96 lines checked
$ dim sparse origin/drm-tip
Commit: drm/i915: Add detection of changing of edid on between suspend and resume
Okay!