Bug 107383

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

<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.

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.

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.

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.

ICL now has a panel that works.

(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.

Marking as low/enhancement, as we don't yet support PSR by default on Gen8 platforms

Also looks like we don't have anymore any BDW in CI with a PSR panel.

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(&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(&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

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(&data) / Failed assertion: psr_wait_entry_if_enabled(data) -}
{+ PSR PSR2: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&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

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(&data) / Failed assertion: psr_wait_entry_if_enabled(data) -}
{+ PSR PSR2: igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&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

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.

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(&data) / Failed assertion: psr_wait_entry_if_enabled(data) -}
{+ PSR PSR2: igt@kms_psr* - fail - Failed assertion: psr_wait_entry_if_enabled(&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

Separate BUG 110215 created to track ICL.
Setting the priority back to low and enhancement.

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(&data) / Failed assertion: psr_wait_entry_if_enabled(data) (No new failures associated)

-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/drm/intel/issues/132.