Bug 106766

Summary: [CI] igt@kms_chamelium@(dp|hdmi)-edid-read - fail - Failed assertion: edid_blob_id != 0
Product: DRI Reporter: Martin Peres <martin.peres>
Component: DRM/IntelAssignee: Intel GFX Bugs mailing list <intel-gfx-bugs>
Status: RESOLVED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: major    
Priority: high CC: intel-gfx-bugs
Version: XOrg git   
Hardware: Other   
OS: All   
Whiteboard: ReadyForDev
i915 platform: ALL i915 features: display/Other

Description Martin Peres 2018-06-01 07:49:23 UTC
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_4267/fi-kbl-7500u/igt@kms_chamelium@dp-edid-read.html

(kms_chamelium:3744) CRITICAL: Test assertion failure function test_edid_read, file ../tests/kms_chamelium.c:274:
(kms_chamelium:3744) CRITICAL: Failed assertion: edid_blob = drmModeGetPropertyBlob(data->drm_fd, edid_blob_id)
(kms_chamelium:3744) CRITICAL: Last errno: 2, No such file or directory
Subtest dp-edid-read failed.
Comment 1 Martin Peres 2018-06-01 07:50:19 UTC
Maybe I would need to create a display/Chamelium feature.
Comment 2 emersion 2019-08-16 13:53:57 UTC
This bug used to be hit very rarely (every few months) and only on fi-kbl-7500u.

However it's now also hit on ICL and CML, every week or so.

User impact: screen not detected when plugging it in.

Added a new check to make sure the test fails because of a NULL blob: https://patchwork.freedesktop.org/series/65315/
Comment 3 emersion 2019-08-16 13:55:19 UTC
By "not detected" I mean "appears as not detected from a user PoV". Of course DRM detects it (since it's CONNECTED). However userspace will probably choke on the EDID being unset.
Comment 4 CI Bug Log 2019-08-19 09:53:34 UTC
A CI Bug Log filter associated to this bug has been updated:

{- KBL: igt@kms_chamelium@dp-edid-read - fail - Failed assertion: edid_blob = drmModeGetPropertyBlob(data-&gt;drm_fd, edid_blob_id) -}
{+ KBL: igt@kms_chamelium@dp-edid-read - fail - Failed assertion: edid_blob = drmModeGetPropertyBlob(data-&gt;drm_fd, edid_blob_id) +}


  No new failures caught with the new filter
Comment 5 emersion 2019-08-19 09:54:53 UTC
commit 0bf74af39ede5fa26195e4e7da590b3d0b729469
Author: Simon Ser <simon.ser@intel.com>
Date:   Fri Aug 16 16:47:13 2019 +0300

    tests/kms_chamelium: check whether EDID blob is non-NULL before trying to read it
    
    test_edid_read sets an EDID on a port, plugs it and reads it back. Sometimes
    retrieving the blob fails. Check whether the blob is set at all before trying
    to get it. This makes it obvious it's not drmModeGetPropertyBlob failing for
    another reason.
    
    Signed-off-by: Simon Ser <simon.ser@intel.com>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106766
    Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Comment 6 emersion 2019-08-20 08:35:07 UTC
*** Bug 111437 has been marked as a duplicate of this bug. ***
Comment 7 emersion 2019-08-26 11:49:04 UTC
*** Bug 111474 has been marked as a duplicate of this bug. ***
Comment 8 CI Bug Log 2019-08-26 13:53:30 UTC
A CI Bug Log filter associated to this bug has been updated:

{- KBL: igt@kms_chamelium@dp-edid-read - fail - Failed assertion: edid_blob = drmModeGetPropertyBlob(data-&gt;drm_fd, edid_blob_id) -}
{+ KBL: igt@kms_chamelium@dp-edid-read - fail - Failed assertion: edid_blob = drmModeGetPropertyBlob(data-&gt;drm_fd, edid_blob_id) +}


  No new failures caught with the new filter
Comment 9 CI Bug Log 2019-08-26 13:54:03 UTC
A CI Bug Log filter associated to this bug has been updated:

{- KBL: igt@kms_chamelium@dp-edid-read - fail - Failed assertion: edid_blob = drmModeGetPropertyBlob(data-&gt;drm_fd, edid_blob_id) -}
{+ KBL: igt@kms_chamelium@dp-edid-read - fail - Failed assertion: edid_blob_id != 0 +}


  No new failures caught with the new filter
Comment 10 Lakshmi 2019-08-26 14:24:02 UTC
(In reply to emersion from comment #2)
> This bug used to be hit very rarely (every few months) and only on
> fi-kbl-7500u.
> 
> However it's now also hit on ICL and CML, every week or so.
> 
> User impact: screen not detected when plugging it in.
> 
> Added a new check to make sure the test fails because of a NULL blob:
> https://patchwork.freedesktop.org/series/65315/

Based on the impact bumping the severity and priority.
Comment 11 Jani Saarinen 2019-09-23 07:24:39 UTC
ASs this was merged, resolving to get CI feedback.
Comment 12 Lakshmi 2019-09-24 12:57:54 UTC
This issue is used to appear once in 125.2 CI_DRM runs. Last seen on CI_DRM_6673 (1 month, 2 weeks old).
Also this issue occurred only once on IGT runs GT_5174 (2 weeks, 1 day old). So, we have a wait before we close this issue. Let's come back after a month or two and see if can close.

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.