From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9C7FD10EC1C for ; Mon, 28 Mar 2022 10:07:51 +0000 (UTC) Date: Mon, 28 Mar 2022 11:52:06 +0200 From: Mauro Carvalho Chehab To: Petri Latvala Message-ID: <20220328115206.6f6a029b@maurocar-mobl2> In-Reply-To: <20220328091329.16009-1-petri.latvala@intel.com> References: <20220328091329.16009-1-petri.latvala@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [igt-dev] [PATCH i-g-t] runner: Fix handling of per-test-timeout List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On Mon, 28 Mar 2022 12:13:29 +0300 Petri Latvala 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 > Cc: Mauro Carvalho Chehab Reviewed-by: Mauro Carvalho Chehab > --- > 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);