[Intel-gfx] ✗ Fi.CI.IGT: warning for igt/core_suspend: Exercise igt_system_suspend_autoresume()

[Imre Deak]

On Mon, Oct 16, 2017 at 12:02:20PM +0100, Chris Wilson wrote:
> Quoting Imre Deak (2017-10-16 11:43:33)
> > On Mon, Oct 16, 2017 at 11:34:03AM +0100, Chris Wilson wrote:
> > > Quoting Patchwork (2017-10-16 11:30:46)
> > > > == Series Details ==
> > > > 
> > > > Series: igt/core_suspend: Exercise igt_system_suspend_autoresume()
> > > > URL   : https://patchwork.freedesktop.org/series/31986/
> > > > State : warning
> > > > 
> > > > == Summary ==
> > > > 
> > > > Test kms_frontbuffer_tracking:
> > > >         Subgroup fbc-rgb101010-draw-mmap-gtt:
> > > >                 skip       -> PASS       (shard-hsw)
> > > > Test core_suspend:
> > > >         Subgroup suspend-freze-processors:
> > > >                 notrun     -> INCOMPLETE (shard-hsw)
> > > >         Subgroup suspend-freze-core:
> > > >                 notrun     -> INCOMPLETE (shard-hsw)
> > > 
> > > Looks like we are not parsing the available debug modes correctly:
> > > 
> > > <4>[  487.523649] PM: Unsupported test mode for suspend to idle, please choose none/freezer/devices/platform.
> > > <6>[  487.523657] PM: suspend exit
> > > <6>[  487.523683] PM: suspend entry (s2idle)
> > > 
> > > i.e. neither processors or core are available, and repeats ad nauseam.
> > 
> > As I recall only the available suspend states can be queried, for that
> > you may want to use get_supported_suspend_states().

Ah, or just depending on igt_require() as-is works too.

> > As for the test
> > mode, maybe just hard-wire not use the invalid combinations?
> 
> 	$ cat pm_test 
> 	[none] core processors platform devices freezer
> 
> New feature? But seems like worth a try if those machines are not
> reporting core/processors.

They are.. On APL:

# cat pm_test
[none] core processors platform devices freezer
# echo processors > pm_test
# cat pm_test
none core [processors] platform devices freezer
# cat state
freeze mem disk
# echo freeze > state
bash: echo: write error: Resource temporarily unavailable
# dmesg | tail -3
[269134.709331] PM: suspend entry (s2idle)
[269134.709698] PM: Unsupported test mode for suspend to idle, please choose none/freezer/devices/platform.
[269134.710035] PM: suspend exit

--Imre