Bug 107587 - [CI][BAT] igt@pm_rpm@module-reload - fail - Could not unload i915
Summary: [CI][BAT] igt@pm_rpm@module-reload - fail - Could not unload i915
Status: CLOSED WORKSFORME
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-16 07:09 UTC by Tomi Sarvela
Modified: 2018-09-07 15:42 UTC (History)
1 user (show)

See Also:
i915 platform: BXT, GLK, HSW, KBL
i915 features: power/runtime PM

Attachments

Description Tomi Sarvela 2018-08-16 07:09:26 UTC 
igt@pm_rpm@module-reload fails with

(pm_rpm:1774) igt_kmod-WARNING: Could not unload i915
(pm_rpm:1774) igt_kmod-WARNING: Could not load i915
(pm_rpm:1774) CRITICAL: Test assertion failure function main, file ../tests/pm_rpm.c:2039:
(pm_rpm:1774) CRITICAL: Failed assertion: igt_i915_driver_load("disable_display=1") == 0
(pm_rpm:1774) CRITICAL: error: 99 != 0
Subtest module-reload failed.

https://intel-gfx-ci.01.org/tree/drm-tip/igt@pm_rpm@module-reload.html

This test has been disabled from CI for now.
Comment 1 Chris Wilson 2018-08-16 09:02:25 UTC 
Fwiw,

https://patchwork.freedesktop.org/series/48189/

should fix the wakeref leak, however the test was removed from the full run in

commit f5c1b2b7081f3057c325a1afba2013e7ff5b299f (upstream/master, origin/master, origin/HEAD)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Thu Aug 16 07:04:59 2018 +0100

    intel-ci: Exclude pm_rpm module-reload from full testing
    
    The pm_rpm module-reload exists to exercise a rpm wakeref leak, and
    affects the random selection of tests run after it. Similar to the
    normal module-reload tests, care must be taken in its execution to avoid
    causing spurious failures elsewhere.
    
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Acked-by: Petri Latvala <petri.latvala@intel.com>

but left in BAT.
Comment 2 Martin Peres 2018-09-07 15:42:00 UTC 
(In reply to Chris Wilson from comment #1)
> Fwiw,
> 
> https://patchwork.freedesktop.org/series/48189/
> 
> should fix the wakeref leak, however the test was removed from the full run
> in
> 
> commit f5c1b2b7081f3057c325a1afba2013e7ff5b299f (upstream/master,
> origin/master, origin/HEAD)
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Thu Aug 16 07:04:59 2018 +0100
> 
>     intel-ci: Exclude pm_rpm module-reload from full testing
>     
>     The pm_rpm module-reload exists to exercise a rpm wakeref leak, and
>     affects the random selection of tests run after it. Similar to the
>     normal module-reload tests, care must be taken in its execution to avoid
>     causing spurious failures elsewhere.
>     
>     Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>     Acked-by: Petri Latvala <petri.latvala@intel.com>
> 
> but left in BAT.

Thanks, it definitely fixed it!

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.