[CI][RESUME] igt@i915_pm_rps@(waitboost|reset) - fail - Failed assertion: post_freqs[CUR] < post_freqs[MAX]

Submitted by Martin Peres @mupuf

Assigned to Sujaritha Sundaresan @sujaritha

Link to original bug (#111650)

Description

https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_365/fi-tgl-u/igt@i915_pm_rps@waitboost.html

Starting subtest: waitboost
(i915_pm_rps:993) CRITICAL: Test assertion failure function waitboost, file ../tests/i915/i915_pm_rps.c:608:
(i915_pm_rps:993) CRITICAL: Failed assertion: post_freqs[CUR] < post_freqs[MAX]
(i915_pm_rps:993) CRITICAL: error: 1100 >= 1100

added CI feature: power/runtime PM platform: TGL priority::medium severity::normal + 1 deleted label

CI Bug Log said:

The CI Bug Log issue associated to this bug has been updated.

New filters associated

* TGL: igt@i915_pm_rps@(waitboost|reset) - fail - Failed assertion: post_freqs[CUR] < post_freqs[MAX]
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_365/fi-tgl-u/igt@i915_pm_rps@waitboost.html
- https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14335/re-tgl-u/igt@i915_pm_rps@reset.html
- https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14335/re-tgl-u/igt@i915_pm_rps@waitboost.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_363/fi-tgl-u/igt@i915_pm_rps@waitboost.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_364/fi-tgl-u/igt@i915_pm_rps@reset.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_364/fi-tgl-u/igt@i915_pm_rps@waitboost.html

CI Bug Log said:

The CI Bug Log issue associated to this bug has been updated.

New filters associated

* TGL: igt@i915_pm_rps@min-max-config-loaded - fail - Failed assertion: freqs[MAX] <= freqs[CUR]
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_366/fi-tgl-u/igt@i915_pm_rps@min-max-config-loaded.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_365/fi-tgl-u/igt@i915_pm_rps@min-max-config-loaded.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_363/fi-tgl-u/igt@i915_pm_rps@min-max-config-loaded.html
- https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14335/re-tgl-u/igt@i915_pm_rps@min-max-config-loaded.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_364/fi-tgl-u/igt@i915_pm_rps@min-max-config-loaded.html

Sujaritha Sundaresan @sujaritha said:

Bug Assessment : VLK - 4701
 
• Reproduction rate : 100% (from CI Bug Log) from 6 runs.
• Last seen 2 days back on TGL systems.
• pm_rps@min-max-config-load – Subtest to verify the constraints with high load and to check if we can reach max.
• pm_rps@waitboost – Subtest to verify if we can achieve boost using gem_wait.
• pm_rps@reset – Subtest to test boost frequency after GPU reset.
• In this case, the issue seems to be the min-max-config-loaded test that is failing in turn causing the waitboost and reset tests to fails. During the min-max test, the loaded-check expects the current frequency to be greater than the max frequency. The tests fails since this condition is not met.
• This condition should be verified. I will continue to follow the issue.

LAKSHMINARAYANA VUDUM @l4kshmi said:

(In reply to sujaritha.sundaresan from comment 3)

Bug Assessment : VLK - 4701
 
• Reproduction rate : 100% (from CI Bug Log) from 6 runs.
• Last seen 2 days back on TGL systems.
• pm_rps@min-max-config-load – Subtest to verify the constraints with
high load and to check if we can reach max.
• pm_rps@waitboost – Subtest to verify if we can achieve boost using
gem_wait.
• pm_rps@reset – Subtest to test boost frequency after GPU reset.
• In this case, the issue seems to be the min-max-config-loaded test
that is failing in turn causing the waitboost and reset tests to fails.
During the min-max test, the loaded-check expects the current frequency to
be greater than the max frequency. The tests fails since this condition is
not met.
• This condition should be verified. I will continue to follow the issue.

Thanks for the initial assessment. Once you know the user impact, can you please set the severity of this bug? Currently severity is not set.

Chris Wilson @ickle closed a related bug:

*** Bug 111839 has been marked as a duplicate of this bug. ***

Chris Wilson @ickle said:

GPU clocks on tgl are currently snafu; the HW does not seem to honour our frequency requests.

CI Bug Log said:

The CI Bug Log issue associated to this bug has been updated.

New filters associated

* TGL: igt@i915_pm_rps@(min-max-config-idle|waitboost) - fail - Failed assertion: freqs[ACT] == freqs[RPn]
- https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3617/shard-tglb6/igt@i915_pm_rps@waitboost.html
- https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3617/shard-tglb8/igt@i915_pm_rps@min-max-config-idle.html
- https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5243/shard-tglb1/igt@i915_pm_rps@min-max-config-idle.html
- https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5243/shard-tglb2/igt@i915_pm_rps@waitboost.html

Sujaritha Sundaresan @sujaritha said:

Increasing the loaded wait timeout from 3000 to 3500 resolves this issue and allows the min-max-config-loaded, reset and waitboost tests to pass.

assigned to @sujaritha

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

TGL: igt@i915_pm_rps@(waitboost|reset) - fail - Failed assertion: post_freqs[CUR] < post_freqs[MAX] ICL TGL: igt@i915_pm_rps@(waitboost|reset) - fail - Failed assertion: post_freqs[CUR] < post_freqs[MAX]

New failures caught by the filter:

• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7612/shard-iclb8/igt@i915_pm_rps@reset.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-iclb4/igt@i915_pm_rps@reset.html

added platform: ICL label

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

ICL TGL: igt@i915_pm_rps@(waitboost|reset) - fail - Failed assertion: post_freqs[CUR] < post_freqs[MAX] EHL ICL TGL: igt@i915_pm_rps@(waitboost|reset) - fail - Failed assertion: post_freqs[CUR] < post_freqs[MAX]

New failures caught by the filter:

• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7928/re-ehl-1/igt@i915_pm_rps@reset.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7918/re-ehl-1/igt@i915_pm_rps@reset.html
• https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7918/re-ehl-1/igt@i915_pm_rps@waitboost.html

added platform: EHL label

added priority::highest severity::major labels and removed platform: EHL platform: TGL priority::medium severity::normal labels

added feature: power/GT label and removed feature: power/runtime PM label

added platform: TGL label and removed platform: ICL label

mentioned in issue #1744 (closed)

This has been narrowed down to the lack of RPS interrupts still absent in tgl B0, see live_rps_interrupt.

Hi @ickle, @sujaritha any updates to report here?

mentioned in issue #39 (closed)

commit 36d516be867ce3657401ab26b05a39a20f56fa13
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Apr 29 21:54:44 2020 +0100

    drm/i915/gt: Switch to manual evaluation of RPS

provides dynamic reclocking on tgl b0.

closed

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.