Bug 105540 - [CI] igt@kms_chamelium@*-edid-change-during-suspend - fail/dmesg-fail - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT)
Summary: [CI] igt@kms_chamelium@*-edid-change-during-suspend - fail/dmesg-fail - Faile...
Status: NEW
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other All
: medium normal
Assignee: Stanislav Lisovskiy
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: ReadyForDev
Keywords:
: 108757 111168 (view as bug list)
Depends on:
Blocks: 108757
  Show dependency treegraph
 
Reported: 2018-03-16 08:22 UTC by Marta Löfstedt
Modified: 2019-09-13 10:53 UTC (History)
4 users (show)

See Also:
i915 platform: CML, KBL, SKL
i915 features: display/Other


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marta Löfstedt 2018-03-16 08:22:40 UTC
https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_1/fi-kbl-7500u/igt@kms_chamelium@dp-edid-change-during-suspend.html
https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_1/fi-skl-6700k2/igt@kms_chamelium@hdmi-edid-change-during-suspend.html

(kms_chamelium:1537) CRITICAL: Test assertion failure function test_suspend_resume_edid_change, file ../tests/kms_chamelium.c:403:
(kms_chamelium:1537) CRITICAL: Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT)
(kms_chamelium:1537) CRITICAL: Last errno: 11, Resource temporarily unavailable
Comment 1 CI Bug Log 2019-01-31 14:49:55 UTC
A CI Bug Log filter associated to this bug has been updated:

{- fi-skl-guc fi-kbl-7567u: igt@kms_chamelium@(dp|hdmi)-edid-read - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) -}
{+ fi-skl-guc fi-kbl-7567u: igt@kms_chamelium@(dp|hdmi)-edid-read - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_202/fi-kbl-7567u/igt@kms_chamelium@hdmi-edid-change-during-suspend.html
Comment 2 Mun, Gwan-gyeong 2019-02-01 07:48:49 UTC
this patchset ( https://patchwork.freedesktop.org/series/49298/) implements to

dp-edid-change-during-hibernate
dp-edid-change-during-suspend
hdmp-edid-change-during-hibernate
hdmp-edid-change-during-suspend
Comment 3 CI Bug Log 2019-02-08 11:23:17 UTC
A CI Bug Log filter associated to this bug has been updated:

{- fi-skl-guc fi-kbl-7567u: igt@kms_chamelium@(dp|hdmi)-edid-read - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) -}
{+ fi-skl-guc fi-kbl-7567u: igt@kms_chamelium@(dp|hdmi)-edid-read - Fail/dmesg-fail Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_211/fi-kbl-7567u/igt@kms_chamelium@hdmi-edid-change-during-suspend.html
* https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_206/fi-kbl-7567u/igt@kms_chamelium@hdmi-edid-change-during-suspend.html
Comment 4 CI Bug Log 2019-02-28 15:19:15 UTC
A CI Bug Log filter associated to this bug has been updated:

{- fi-skl-guc fi-kbl-7567u: igt@kms_chamelium@(dp|hdmi)-edid-read - Fail/dmesg-fail Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) -}
{+ fi-skl-6700k2 fi-kbl-7567u fi-kbl-7500u: igt@kms_chamelium@(dp|hdmi)-edid-read - fail / dmesg-fail -Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) +}

 No new failures caught with the new filter
Comment 5 CI Bug Log 2019-03-27 08:36:10 UTC
A CI Bug Log filter associated to this bug has been updated:

{- fi-skl-6700k2 fi-kbl-7567u fi-kbl-7500u: igt@kms_chamelium@(dp|hdmi)-edid-read - fail / dmesg-fail -Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) -}
{+ CHAMELIUM: igt@kms_chamelium@(dp|hdmi)-edid-read - fail / dmesg-fail -Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) +}

 No new failures caught with the new filter
Comment 6 Martin Peres 2019-03-27 08:46:02 UTC
*** Bug 108757 has been marked as a duplicate of this bug. ***
Comment 7 CI Bug Log 2019-03-27 08:47:39 UTC
A CI Bug Log filter associated to this bug has been updated:

{- CHAMELIUM: igt@kms_chamelium@(dp|hdmi)-edid-read - fail / dmesg-fail -Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT) -}
{+ CHAMELIUM: igt@kms_chamelium@(dp|hdmi)-edid-read - fail / dmesg-fail -Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT), Last errno: 11, Resource temporarily unavailable +}

 No new failures caught with the new filter
Comment 8 CI Bug Log 2019-03-27 08:48:17 UTC
The CI Bug Log issue associated to this bug has been archived.

New failures matching the above filters will not be associated to this bug anymore.
Comment 9 Easwar Hariharan 2019-03-27 19:01:34 UTC
(In reply to CI Bug Log from comment #8)
> The CI Bug Log issue associated to this bug has been archived.
> 
> New failures matching the above filters will not be associated to this bug
> anymore.

Does this mean this is not reproducible anymore?
Comment 10 Mun, Gwan-gyeong 2019-04-11 15:49:20 UTC
new patch series updated: https://patchwork.freedesktop.org/series/59352/ 

the patch series addressed  review comments from Maarten.
Comment 11 Mun, Gwan-gyeong 2019-04-21 09:21:54 UTC
Comment addressed patch updated: https://patchwork.freedesktop.org/patch/301078/?series=59691&rev=3
Comment 12 Jani Saarinen 2019-06-03 15:05:24 UTC
gg, do you have any updates on this, where this got stuck? This still seen eg.
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6179/re-icl-u/igt@kms_chamelium@dp-edid-change-during-suspend.html
Comment 13 Stanislav Lisovskiy 2019-06-20 10:33:03 UTC
There are some problems, like i915_hotplug_work_func was actually not calling drm_helper_hpd_irq_event(dev) which we rely on, but drm_kms_helper_hotplug_event(dev) which I guess is wrong as drm_helper_hpd_irq_event itself calls drm_kms_helper_hotplug_event after doing some checks and those patches rely on that drm_helper_hpd_irq_event is called.

Also the edid blob ptr doesn't get updated during probe after suspend, because driver still uses the cached version( I checked that), which makes it impossible to check that the edid actually got changed because the pointer and blob is the same!

To fix this need to call drm_edid_duplicate in drm_helper_hpd_irq_event before doing probing(drm_helper_probe_detect) which will modify connector->edid_blob_ptr and copy it to another blob then compare to the new one.

Currently I'm implementing those fixes.
Comment 14 emersion 2019-09-10 11:31:23 UTC
*** Bug 111168 has been marked as a duplicate of this bug. ***
Comment 15 CI Bug Log 2019-09-10 11:34:21 UTC
The CI Bug Log issue associated to this bug has been updated.

### New filters associated

* fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_365/fi-cml-u2/igt@kms_chamelium@hdmi-aspect-ratio.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_365/fi-cml-u2/igt@kms_chamelium@hdmi-crc-planes-random.html
Comment 16 CI Bug Log 2019-09-11 14:03:09 UTC
A CI Bug Log filter associated to this bug has been updated:

{- fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host -}
{+ fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host +}

New failures caught by the filter:

  * https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_366/fi-cml-u2/igt@kms_chamelium@dp-link-status.html
Comment 17 CI Bug Log 2019-09-11 14:03:58 UTC
A CI Bug Log filter associated to this bug has been updated:

{- fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host -}
{+ fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host +}

New failures caught by the filter:

  * https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_366/fi-cml-u2/igt@kms_chamelium@hdmi-audio-edid.html
  * https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_366/fi-cml-u2/igt@kms_chamelium@dp-mode-timings.html
  * https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_366/fi-cml-u2/igt@kms_chamelium@hdmi-mode-timings.html
Comment 18 CI Bug Log 2019-09-13 07:20:46 UTC
A CI Bug Log filter associated to this bug has been updated:

{- fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host -}
{+ fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host +}

New failures caught by the filter:

  * https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_367/fi-cml-u2/igt@kms_chamelium@hdmi-audio.html
  * https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_367/fi-cml-u2/igt@kms_chamelium@dp-audio-edid.html
Comment 19 Stanislav Lisovskiy 2019-09-13 08:29:10 UTC
(In reply to CI Bug Log from comment #15)
> The CI Bug Log issue associated to this bug has been updated.
> 
> ### New filters associated
> 
> * fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl
> failed to execute the HTTP POST transaction, explaining:  Failed to connect
> to 192.168.1.226 port 9992: No route to host
>   -
> https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_365/fi-cml-u2/
> igt@kms_chamelium@hdmi-aspect-ratio.html
>   -
> https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_365/fi-cml-u2/
> igt@kms_chamelium@hdmi-crc-planes-random.html

         /\
         ||
         ||
That filter and error have nothing to do with edid change during suspend and missing hotplug, I guess we need another bug for that, otherwise we get one more junkyard :)
Comment 20 emersion 2019-09-13 08:47:49 UTC
(In reply to Stanislav Lisovskiy from comment #19)
> That filter and error have nothing to do with edid change during suspend and
> missing hotplug, I guess we need another bug for that, otherwise we get one
> more junkyard :)

Indeed!

It seems like the Chamelium board's network was stuck. I rebooted it, I can reach it again from fi-cml-u2.

(The "no route to host" filters should still be removed from this bug.)
Comment 21 CI Bug Log 2019-09-13 10:53:47 UTC
The CI Bug Log issue associated to this bug has been updated.

### Removed filters

* fi-cml-u2: igt@kms_chamelium@* - fail - Chamelium RPC call failed: libcurl failed to execute the HTTP POST transaction, explaining:  Failed to connect to 192.168.1.226 port 9992: No route to host (added on 3 hours ago)


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.