diff options
author | Petri Latvala <petri.latvala@intel.com> | 2020-01-23 13:15:36 +0200 |
---|---|---|
committer | Petri Latvala <petri.latvala@intel.com> | 2020-01-23 14:52:49 +0200 |
commit | 50c31d6423d37633dae5d36c7aaea10162baa63d (patch) | |
tree | 803bfd4f293c8de3082818368554afad02b7a4f5 /runner | |
parent | b8d03fef6f135c27b3f45f820e62d40aac956471 (diff) |
runner: Don't check for taints when not configured for it
If someone wants to execute tests without aborting when tainted, they
get all their tests just straight up killed on the first taint without
actually aborting execution. Obey their wishes and keep running.
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'runner')
-rw-r--r-- | runner/executor.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/runner/executor.c b/runner/executor.c index 71483b74..0927d1fd 100644 --- a/runner/executor.c +++ b/runner/executor.c @@ -759,7 +759,7 @@ static int monitor_output(pid_t child, return -1; } - if (tainted(&taints)) /* cancel children after a kernel OOPS */ + if (settings->abort_mask & ABORT_TAINT && tainted(&taints)) /* cancel children after a kernel OOPS */ n = 0, intervals_left = 1; if (n == 0) { @@ -768,6 +768,7 @@ static int monitor_output(pid_t child, switch (killed) { case 0: + /* If abort_mask doesn't have taint set, taints is still 0 here */ if (!is_tainted(taints)) { show_kernel_task_state(); if (settings->log_level >= LOG_LEVEL_NORMAL) { |