Re: [igt-dev] [PATCH i-g-t] runner: Fix handling of per-test-timeout

[Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>]

On Mon, 28 Mar 2022 12:13:29 +0300
Petri Latvala <petri.latvala@intel.com> wrote:

> Instead of stopping execution on resume-init success, stop on
> resume-init failure like intended.
> 
> Fixes: 4b88a9253443 ("runner: check if it has root permissions")
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Reviewed-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> ---
>  runner/executor.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/runner/executor.c b/runner/executor.c
> index b4efc4fd..6e6ca9cc 100644
> --- a/runner/executor.c
> +++ b/runner/executor.c
> @@ -2043,7 +2043,7 @@ bool execute(struct execute_state *state,
>  			}
>  			close(sigfd);
>  			close(testdirfd);
> -			if (initialize_execute_state_from_resume(resdirfd, state, settings, job_list))
> +			if (!initialize_execute_state_from_resume(resdirfd, state, settings, job_list))
>  				return false;
>  			state->time_left = time_left;
>  			return execute(state, settings, job_list);