[igt-dev] [RFC i-g-t 09/11] Add test to verify session key renewed after suspend-resume cycle

[Teres Alexis, Alan Previn]

Right - got it. Thanks.
...alan

-----Original Message-----
From: Chris Wilson <chris at chris-wilson.co.uk> 
Sent: Wednesday, March 3, 2021 4:00 PM
To: Teres Alexis, Alan Previn <alan.previn.teres.alexis at intel.com>; igt-dev at lists.freedesktop.org
Cc: Teres Alexis, Alan Previn <alan.previn.teres.alexis at intel.com>
Subject: Re: [igt-dev] [RFC i-g-t 09/11] Add test to verify session key renewed after suspend-resume cycle

Quoting Alan Previn (2021-03-02 22:53:32)
> +static void trigger_powermgt_suspend_cycle(int i915,
> +       struct powermgt_data *pm)
> +{
> +       igt_pm_enable_sata_link_power_management();
> +
> +
> +       igt_system_suspend_autoresume(SUSPEND_STATE_MEM, 
> + SUSPEND_TEST_DEVICES);
> +
> +       igt_wait_for_pm_status(IGT_RUNTIME_PM_STATUS_SUSPENDED);
> +       gem_test_engine(i915, I915_EXEC_RENDER);
> +       while 
> + ((!igt_wait_for_pm_status(IGT_RUNTIME_PM_STATUS_ACTIVE))) {

With an autosuspend delay of 0, this is not guaranteed to ever report ACTIVE.

Just drop everything after wait_for_pm_status(SUSPENDED) as clearly using the protected rendercopy must wake the device.
-Chris