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

Petri Latvala petri.latvala at intel.com
Mon Mar 28 09:13:29 UTC 2022


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 at intel.com>
Cc: Mauro Carvalho Chehab <mchehab at 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);
-- 
2.30.2



More information about the igt-dev mailing list