Bug 112086

Summary: [CI][RESUME]igt@i915_pm_rpm@modeset-non-lpsp - skip - Test requirement: setup_environment(), SKIP
Product: DRI Reporter: Lakshmi <lakshminarayana.vudum>
Component: IGTAssignee: Default DRI bug account <dri-devel>
Status: RESOLVED MOVED QA Contact:
Severity: normal    
Priority: not set CC: intel-gfx-bugs, tomi.p.sarvela
Version: DRI git   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: TGL i915 features: power/runtime PM

Description Lakshmi 2019-10-22 06:07:38 UTC
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/re-tgl-u/igt@i915_pm_rpm@modeset-non-lpsp.html
Runtime PM support: 1
PC8 residency support: 0
DMC: fw loaded: yes
i915_runtime_pm_status:
Runtime power status: enabled
GPU idle: yes
IRQs disabled: no
Usage count: 1
PCI device power state: D0 [0]
Wakeref last acquired:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   intel_dp_aux_xfer+0xd3/0x8e0 [i915]
   intel_dp_aux_transfer+0xa7/0x200 [i915]
   drm_dp_dpcd_access+0x76/0x110
   drm_dp_dpcd_read+0x29/0xc0
   intel_dp_get_dsc_sink_cap+0x54/0xe0 [i915]
   intel_dp_detect+0x1b9/0x4f0 [i915]
Wakeref last released:
   __untrack_all_wakerefs+0x67/0x70 [i915]
   untrack_all_intel_runtime_pm_wakerefs+0x3e/0x70 [i915]
   intel_runtime_suspend+0xa4/0x1c0 [i915]
   pci_pm_runtime_suspend+0x57/0x190
   __rpm_callback+0xb1/0x110
   rpm_callback+0x1a/0x70
   rpm_suspend+0xf6/0x620
   pm_runtime_work+0x6e/0xb0
Wakeref count: 91
Wakeref x11 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   intel_ddi_pre_enable+0x608/0xe40 [i915]
   intel_encoders_pre_enable.isra.51+0x61/0x80 [i915]
   haswell_crtc_enable+0x82/0x780 [i915]
   intel_update_crtc+0x5c/0x350 [i915]
   skl_commit_modeset_enables+0x25d/0x2a0 [i915]
   intel_atomic_commit_tail+0x26b/0x1540 [i915]
Wakeref x11 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   edp_panel_vdd_on+0xd5/0x210 [i915]
   intel_dp_aux_xfer+0xeb/0x8e0 [i915]
   intel_dp_aux_transfer+0xa7/0x200 [i915]
   drm_dp_dpcd_access+0x76/0x110
   drm_dp_dpcd_read+0x29/0xc0
   intel_dp_read_dpcd+0x38/0x120 [i915]
Wakeref x11 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   modeset_get_crtc_power_domains+0x120/0x140 [i915]
   intel_atomic_commit_tail+0xcf/0x1540 [i915]
   intel_atomic_commit+0x273/0x2f0 [i915]
   drm_client_modeset_commit_atomic+0x18b/0x210
   drm_client_modeset_commit_force+0x52/0x1a0
   drm_fb_helper_restore_fbdev_mode_unlocked+0x42/0x90
Wakeref x1 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   edp_panel_vdd_on+0xd5/0x210 [i915]
   intel_dp_aux_xfer+0xeb/0x8e0 [i915]
   intel_dp_aux_transfer+0xa7/0x200 [i915]
   drm_dp_dpcd_access+0x76/0x110
   drm_dp_dpcd_read+0x29/0xc0
   intel_dp_get_dsc_sink_cap+0x54/0xe0 [i915]
Wakeref x1 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   intel_ddi_pre_pll_enable+0x6e/0x120 [i915]
   intel_encoders_pre_pll_enable.isra.50+0x61/0x80 [i915]
   haswell_crtc_enable+0x5c/0x780 [i915]
   intel_update_crtc+0x5c/0x350 [i915]
   skl_commit_modeset_enables+0x1ca/0x2a0 [i915]
   intel_atomic_commit_tail+0x26b/0x1540 [i915]
Wakeref x1 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   intel_ddi_pre_enable+0x608/0xe40 [i915]
   intel_encoders_pre_enable.isra.51+0x61/0x80 [i915]
   haswell_crtc_enable+0x82/0x780 [i915]
   intel_update_crtc+0x5c/0x350 [i915]
   skl_commit_modeset_enables+0x1ca/0x2a0 [i915]
   intel_atomic_commit_tail+0x26b/0x1540 [i915]
Wakeref x1 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   edp_panel_vdd_on+0xd5/0x210 [i915]
   intel_dp_aux_xfer+0xeb/0x8e0 [i915]
   intel_dp_aux_transfer+0x124/0x200 [i915]
   drm_dp_dpcd_access+0x76/0x110
   drm_dp_dpcd_write+0x21/0x90
   intel_psr_disable_locked+0xed/0x180 [i915]
Wakeref x40 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   modeset_get_crtc_power_domains+0x120/0x140 [i915]
   intel_atomic_commit_tail+0xcf/0x1540 [i915]
   intel_atomic_commit+0x273/0x2f0 [i915]
   drm_atomic_helper_set_config+0x61/0x90
   drm_mode_setcrtc+0x18e/0x720
   drm_ioctl_kernel+0xa7/0xf0
Wakeref x10 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   intel_ddi_pre_pll_enable+0x6e/0x120 [i915]
   intel_encoders_pre_pll_enable.isra.50+0x61/0x80 [i915]
   haswell_crtc_enable+0x5c/0x780 [i915]
   intel_update_crtc+0x5c/0x350 [i915]
   skl_commit_modeset_enables+0x25d/0x2a0 [i915]
   intel_atomic_commit_tail+0x26b/0x1540 [i915]
Wakeref x2 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   edp_panel_vdd_on+0xd5/0x210 [i915]
   intel_dp_aux_xfer+0xeb/0x8e0 [i915]
   intel_dp_aux_transfer+0x124/0x200 [i915]
   drm_dp_dpcd_access+0x76/0x110
   drm_dp_dpcd_write+0x21/0x90
   intel_dp_sink_dpms+0x4f/0x100 [i915]
Wakeref x1 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   intel_ddi_pre_enable+0x8bc/0xe40 [i915]
   intel_encoders_pre_enable.isra.51+0x61/0x80 [i915]
   haswell_crtc_enable+0x82/0x780 [i915]
   intel_update_crtc+0x5c/0x350 [i915]
   skl_commit_modeset_enables+0x25d/0x2a0 [i915]
   intel_atomic_commit_tail+0x26b/0x1540 [i915]
Wakeref x1 taken at:
   track_intel_runtime_pm_wakeref+0x14/0x90 [i915]
   intel_display_power_get+0x1f/0x60 [i915]
   intel_ddi_pre_pll_enable+0x111/0x120 [i915]
   intel_encoders_pre_pll_enable.isra.50+0x61/0x80 [i915]
   haswell_crtc_enable+0x5c/0x780 [i915]
   intel_update_crtc+0x5c/0x350 [i915]
   skl_commit_modeset_enables+0x25d/0x2a0 [i915]
   intel_atomic_commit_tail+0x26b/0x1540 [i915]
Test requirement not met in function __real_main1983, file ../tests/i915/i915_pm_rpm.c:1994:
Test requirement: setup_environment()
Subtest modeset-non-lpsp: SKIP
Comment 1 CI Bug Log 2019-10-22 06:08:49 UTC
The CI Bug Log issue associated to this bug has been updated.

### New filters associated

* TGL:  igt@i915_pm_rpm@modeset-non-lpsp - skip - Test requirement: setup_environment(), SKIP
  - https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/re-tgl-u/igt@i915_pm_rpm@modeset-non-lpsp.html
Comment 2 Don Hiatt 2019-10-23 17:29:45 UTC
I'm getting access denied when trying to look at the log: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/re-tgl-u/igt@i915_pm_rpm@modeset-non-lpsp.html
Comment 3 Lakshmi 2019-10-24 07:53:53 UTC
(In reply to Don Hiatt from comment #2)
> I'm getting access denied when trying to look at the log:
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/re-tgl-u/
> igt@i915_pm_rpm@modeset-non-lpsp.html

CC'ing Tomi.
Comment 4 Tomi Sarvela 2019-10-24 10:37:46 UTC
CI_DRM_7104 re-tgl-u run was messed up due to unnecessary manual intervention here the lab: it was four overlapping runs in one.

The reports have been re-run, and some of the visible results might have been changed as only one of the runs is considered valid.
Comment 5 CI Bug Log 2019-10-30 16:04:07 UTC
A CI Bug Log filter associated to this bug has been updated:

{- TGL:  igt@i915_pm_rpm@modeset-non-lpsp - skip - Test requirement: setup_environment(), SKIP -}
{+ TGL:  igt@i915_pm_rpm@modeset-non-lpsp|igt@i915_pm_rpm@gem-mmap-cpu - skip - Test requirement: setup_environment(), SKIP +}

New failures caught by the filter:

  * https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7181/re-tgl-u/igt@i915_pm_rpm@gem-mmap-cpu.html
Comment 6 Don Hiatt 2019-10-30 17:08:07 UTC
setup_environment() (tests/i915/i915_pm_rpm.c) requires that the DUT suspends, otherwise tests are skipped (as gem-mmap-cpu in the latest run).

I recently added some logging to IGT (see below) and it is showing that the DUT's pm_status is active but we are expecting it to be suspended. The test only allows 10 seconds to suspend before giving up. This appears to be just a race in the IGT where occasionally DUTs take longer that 10 seconds, hence the reason we only see this occasionally.

Since this is an IGT issue, I think the severity is medium. We should probably add some additional retries to close this race condition.


(i915_pm_rpm:910) igt_pm-WARNING: timeout: pm_status expected:suspended, got:active
Subtest gem-mmap-cpu: SKIP
Comment 7 Lakshmi 2019-10-31 07:02:01 UTC
(In reply to Don Hiatt from comment #6)
> setup_environment() (tests/i915/i915_pm_rpm.c) requires that the DUT
> suspends, otherwise tests are skipped (as gem-mmap-cpu in the latest run).
> 
> I recently added some logging to IGT (see below) and it is showing that the
> DUT's pm_status is active but we are expecting it to be suspended. The test
> only allows 10 seconds to suspend before giving up. This appears to be just
> a race in the IGT where occasionally DUTs take longer that 10 seconds, hence
> the reason we only see this occasionally.
> 
> Since this is an IGT issue, I think the severity is medium. We should
> probably add some additional retries to close this race condition.
> 
> 
> (i915_pm_rpm:910) igt_pm-WARNING: timeout: pm_status expected:suspended,
> got:active
> Subtest gem-mmap-cpu: SKIP

Changing the component to IGT.
Comment 8 Martin Peres 2019-11-12 07:42:06 UTC
-- 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/igt-gpu-tools/issues/35.

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.