FYI: I reviewed and pushed patches 1-4 of this series, thanks!
On Tue, 2017-06-27 at 13:53 +0300, Paul Kocialkowski wrote:
> This stores the autoresume delay provided via
> igt_set_autoresume_delay
> for use during suspend via rtc wake scheduling. This delay was
> previously only used for pm_test suspendm while the function suggests
> it should be applied to all autoresume cases.
>
> There is also definitely a use case for configuring the rtc wake
> delay,
> so this implements it.
>
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@xxxxxxxxxxxxxxx>
> ---
> lib/igt_aux.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/lib/igt_aux.c b/lib/igt_aux.c
> index eb563f72..882dba06 100644
> --- a/lib/igt_aux.c
> +++ b/lib/igt_aux.c
> @@ -680,6 +680,8 @@ void igt_cleanup_aperture_trashers(void)
> free(trash_bos);
> }
>
> +static int autoresume_delay;
> +
> static const char *suspend_state_name[] = {
> [SUSPEND_STATE_FREEZE] = "freeze",
> [SUSPEND_STATE_STANDBY] = "standby",
> @@ -746,7 +748,10 @@ static void suspend_via_rtcwake(enum
> igt_suspend_state state)
>
> igt_assert(state < SUSPEND_STATE_NUM);
>
> - delay = state == SUSPEND_STATE_DISK ? 30 : 15;
> + if (autoresume_delay)
> + delay = autoresume_delay;
> + else
> + delay = state == SUSPEND_STATE_DISK ? 30 : 15;
>
> /*
> * Skip if rtcwake would fail for a reason not related to
> the kernel's
> @@ -889,6 +894,8 @@ void igt_set_autoresume_delay(int delay_secs)
> igt_require(write(delay_fd, delay_str, strlen(delay_str)));
>
> close(delay_fd);
> +
> + autoresume_delay = delay_secs;
> }
>
> /**
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
