[CI][DRMTIP] igt@kms_psr@* - fail - Failed assertion: psr_wait_entry_if_enabled(&data)

Submitted by Martin Peres @mupuf

Assigned to Dhinakaran Pandiyan @dpandiya

Link to original bug (#107383)

Description

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

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

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

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

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

added CI Enhancement feature: display/PSR platform: BDW platform: CFL priority::low + 1 deleted label

James Ausmus said:

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

Dhinakaran Pandiyan @dpandiya said:

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

Martin Peres @mupuf said:

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.

Martin Peres @mupuf said:

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.

Dhinakaran Pandiyan @dpandiya said:

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.

Dhinakaran Pandiyan @dpandiya said:

ICL now has a panel that works.

Martin Peres @mupuf said:

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

James Ausmus said:

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

José Roberto de Souza @zehortigoza said:

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

CI Bug Log said:

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

CI Bug Log said:

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

CI Bug Log said:

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

LAKSHMINARAYANA VUDUM @l4kshmi said:

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.

CI Bug Log said:

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

LAKSHMINARAYANA VUDUM @l4kshmi said:

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

CI Bug Log said:

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)

assigned to @dpandiya

added platform: ICL platform: RKL labels

A CI Bug Log filter associated to this bug has been updated by Lakshmi Vudum:

Description: ICL RKL: igt@kms_psr* - fail - Failed assertion: psr_wait_entry_if_enabled(&data) / Failed assertion: psr_wait_entry_if_enabled(data)

Equivalent query: runconfig_tag IS IN ["DRM-TIP"] AND (machine_name IS IN ["fi-icl-u2", "shard-iclb", "shard-iclb1", "shard-iclb2", "shard-iclb3", "shard-iclb4", "shard-iclb5", "shard-iclb6", "fi-icl-u3", "shard-iclb7", "fi-icl-y", "shard-iclb8", "fi-icl-guc", "fi-icl-dsi", "re-icl-u", "fi-icl-u4", "fi-icl-1065g7", "pig-icl-1065g7", "re-rkl-1", "re-rkl-cham", "fi-icl-u"] OR machine_tag IS IN ["ICL"]) AND ((testsuite_name = "IGT" AND test_name IS IN ["igt@kms_psr_sink_crc@sprite_mmap_gtt_waiting", "igt@kms_psr_sink_crc@primary_mmap_cpu", "igt@kms_psr_sink_crc@cursor_plane_onoff", "igt@kms_psr_sink_crc@sprite_render", "igt@kms_psr_sink_crc@sprite_mmap_cpu", "igt@kms_psr_sink_crc@psr_drrs", "igt@kms_psr_sink_crc@primary_page_flip", "igt@kms_psr_sink_crc@primary_blt", "igt@kms_psr@basic", "igt@kms_psr@cursor_blt", "igt@kms_psr@dpms", "igt@kms_psr@sprite_blt", "igt@kms_psr@sprite_mmap_cpu", "igt@kms_psr_sink_crc@cursor_plane_move", "igt@kms_psr@sprite_plane_move", "igt@kms_psr@cursor_plane_move", "igt@kms_psr_sink_crc@dpms_off_psr_active", "igt@kms_psr@no_drrs", "igt@kms_psr@cursor_mmap_cpu", "igt@kms_psr@primary_blt", "igt@kms_psr@primary_mmap_gtt", "igt@kms_psr_sink_crc@dpms_off_psr_exit", "igt@kms_psr@sprite_mmap_gtt", "igt@kms_psr@primary_render", "igt@kms_psr@cursor_mmap_gtt", "igt@kms_psr@primary_mmap_cpu", "igt@kms_psr@primary_page_flip", "igt@kms_psr_sink_crc@sprite_mmap_gtt", "igt@kms_psr@cursor_plane_onoff", "igt@kms_psr@cursor_render", "igt@kms_psr@sprite_render", "igt@kms_psr_sink_crc@basic", "igt@kms_psr_sink_crc@dpms", "igt@kms_psr_sink_crc@no_drrs", "igt@kms_psr_sink_crc@suspend", "igt@kms_psr_sink_crc@cursor_render", "igt@kms_psr_sink_crc@psr_basic", "igt@kms_psr_sink_crc@suspend_psr_exit", "igt@kms_psr_sink_crc@sprite_blt", "igt@kms_psr@psr2_cursor_plane_move", "igt@kms_psr@psr2_primary_mmap_gtt", "igt@kms_psr@psr2_sprite_plane_onoff", "igt@kms_psr@psr2_primary_page_flip", "igt@kms_psr_sink_crc@cursor_mmap_cpu", "igt@kms_psr@psr2_sprite_render", "igt@kms_psr@psr2_sprite_plane_move", "igt@kms_psr@psr2_sprite_mmap_gtt", "igt@kms_psr@psr2_primary_render", "igt@kms_psr@psr2_suspend", "igt@kms_psr@psr2_basic", "igt@kms_psr@psr2_sprite_mmap_cpu", "igt@kms_psr@psr2_cursor_render", "igt@kms_psr@psr2_sprite_blt", "igt@kms_psr@psr2_primary_blt", "igt@kms_psr@psr2_cursor_mmap_cpu", "igt@kms_psr@psr2_cursor_blt", "igt@kms_psr@psr2_no_drrs", "igt@kms_psr@psr2_cursor_plane_onoff", "igt@kms_psr@psr2_dpms", "igt@kms_psr@psr2_cursor_mmap_gtt", "igt@kms_psr_sink_crc@cursor_blt", "igt@kms_psr_sink_crc@cursor_mmap_gtt", "igt@kms_psr@psr2_primary_mmap_cpu", "igt@kms_psr_sink_crc@sprite_plane_move", "igt@kms_psr_sink_crc@suspend_psr_active", "igt@kms_psr@sprite_plane_onoff", "igt@kms_psr_sink_crc@primary_render", "igt@kms_psr_sink_crc@cursor_mmap_gtt_waiting", "igt@kms_psr_sink_crc@primary_mmap_gtt_waiting", "igt@kms_psr@suspend", "igt@kms_psr_sink_crc@sprite_plane_onoff", "igt@kms_psr_sink_crc@primary_mmap_gtt"])) AND ((testsuite_name = "IGT" AND status_name IS IN ["fail", "dmesg-fail"])) AND stderr ~= 'Failed assertion: psr_wait_entry_if_enabled\(&?data\)'

New failures caught by the filter:

• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_basic.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_cursor_blt.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_cursor_mmap_cpu.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_cursor_mmap_gtt.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_cursor_plane_move.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_cursor_plane_onoff.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_cursor_render.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_dpms.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_no_drrs.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_primary_blt.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_primary_mmap_cpu.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_primary_mmap_gtt.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_primary_page_flip.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_primary_render.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_sprite_blt.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_sprite_mmap_cpu.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_sprite_mmap_gtt.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_sprite_plane_move.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_sprite_plane_onoff.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_sprite_render.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8925/re-rkl-1/igt@kms_psr@psr2_suspend.html