Bug 107383

Summary: [CI][DRMTIP] igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&data)
Product: DRI Reporter: Martin Peres <martin.peres>
Component: DRM/IntelAssignee: Dhinakaran Pandiyan <dhinakaran.pandiyan>
Status: REOPENED --- QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: enhancement    
Priority: low CC: dhinakaran.pandiyan, intel-gfx-bugs, jose.souza
Version: XOrg git   
Hardware: Other   
OS: All   
Whiteboard: ReadyForDev
i915 platform: BDW, CFL i915 features: display/PSR

Comment 1 James Ausmus 2018-08-14 19:00:57 UTC
After discussion with DK, moving this to a Medium priority, as PSR isn't enabled by default, and this isn't blocking/affecting non-PSR IGT tests
Comment 2 Dhinakaran Pandiyan 2018-08-14 19:13:36 UTC
<7>[  480.340658] [drm:intel_dp_compute_config [i915]] PSR condition failed: PSR setup time (330 us) too long

This a test bug I'm aware of. The test should be a skip instead of fail as the driver does not enable PSR on this panel. Also, note that the issue is seen only with the panel on ICL.
Comment 3 Martin Peres 2018-08-28 14:03:35 UTC
Also seen on BDW: https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_94/fi-bdw-samus/igt@kms_psr@sprite_mmap_cpu.html

(kms_psr:1325) CRITICAL: Test assertion failure function main, file ../tests/kms_psr.c:454:
(kms_psr:1325) CRITICAL: Failed assertion: psr_wait_entry_if_enabled(&data)
Subtest sprite_mmap_cpu failed.


https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_95/fi-icl-u/igt@kms_psr@cursor_mmap_gtt.html

(kms_psr:2338) CRITICAL: Test assertion failure function main, file ../tests/kms_psr.c:464:
(kms_psr:2338) CRITICAL: Failed assertion: psr_wait_entry_if_enabled(&data)
Subtest cursor_mmap_gtt failed.
Comment 4 Martin Peres 2018-09-03 08:40:13 UTC
BDW also has a different variant: https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_93/fi-bdw-samus/igt@kms_psr@sprite_mmap_gtt.html

(kms_psr:1406) CRITICAL: Test assertion failure function run_test, file ../tests/kms_psr.c:247:
(kms_psr:1406) CRITICAL: Failed assertion: psr_wait_entry_if_enabled(data)
Subtest sprite_mmap_gtt failed.
Comment 5 Dhinakaran Pandiyan 2018-09-04 19:27:08 UTC
The ICL issue is different from the BDW.
The plan to address the ICL issue is to switch the eDP panel and as for BDW, we'll have to treat is a low priority for now.
Comment 6 Dhinakaran Pandiyan 2018-09-24 23:40:07 UTC
ICL now has a panel that works.
Comment 7 Martin Peres 2018-09-26 10:37:27 UTC
(In reply to Dhinakaran Pandiyan from comment #6)
> ICL now has a panel that works.

Thanks, re-purposing the bug for the BDW case only.
Comment 8 James Ausmus 2018-12-17 19:15:33 UTC
Marking as low/enhancement, as we don't yet support PSR by default on Gen8 platforms
Comment 9 Jose Roberto de Souza 2019-02-08 21:19:40 UTC
Also looks like we don't have anymore any BDW in CI with a PSR panel.
Comment 10 CI Bug Log 2019-03-04 08:38:04 UTC
A CI Bug Log filter associated to this bug has been updated:

{- PSR PSR2: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) -}
{+ PSR PSR2 fi-icl-y: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_235/fi-icl-y/igt@kms_psr@psr2_cursor_mmap_gtt.html
Comment 11 CI Bug Log 2019-03-15 09:54:22 UTC
A CI Bug Log filter associated to this bug has been updated:

{- PSR PSR2 fi-icl-y: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) -}
{+ PSR PSR2: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4886/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
* https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4886/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html
* https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4886/shard-iclb2/igt@kms_psr@psr2_primary_render.html
Comment 12 CI Bug Log 2019-03-20 10:34:39 UTC
A CI Bug Log filter associated to this bug has been updated:

{- PSR PSR2: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) -}
{+ PSR PSR2: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4889/shard-iclb2/igt@kms_psr@psr2_primary_blt.html
Comment 13 Lakshmi 2019-03-20 10:42:05 UTC
This issue is still occurring on ICL. Result of the test is fail, if Skip is what expected then need a fix in test?
ICL platform is added.
Comment 14 CI Bug Log 2019-03-20 13:27:20 UTC
A CI Bug Log filter associated to this bug has been updated:

{- PSR PSR2: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) -}
{+ PSR PSR2: igt@kms_psr* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4891/shard-iclb2/igt@kms_psr@psr2_suspend.html
* https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4891/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_gtt.html
Comment 15 Lakshmi 2019-03-21 08:07:30 UTC
Separate BUG 110215 created to track ICL.
Setting the priority back to low and enhancement.
Comment 16 CI Bug Log 2019-03-21 10:06:19 UTC
The CI Bug Log issue associated to this bug has been updated.

### New filters associated

* ICL: igt@kms_psr* - fail - Failed assertion: psr_wait_entry_if_enabled(&amp;data) / Failed assertion: psr_wait_entry_if_enabled(data) (No new failures associated)

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.