diff options
| author | Petri Latvala <petri.latvala@intel.com> | 2022-10-31 12:24:34 +0200 |
|---|---|---|
| committer | Petri Latvala <petri.latvala@intel.com> | 2022-10-31 17:46:44 +0200 |
| commit | 372c56225e12578a7a4a6bcc5b79eb40b643fcde (patch) | |
| tree | 758b2e3b1acff3051d75ea6e51b2da516b45f54a /runner | |
| parent | c8122e4412ef0b29191a4fb796d7b6ba9b340150 (diff) | |
runner: Disable socket communications for now
Currently, having i915 CI systems test the socket communications when
testing is impossible. If a test hangs completely, the incomplete set
of results are processed into results.json with an older IGT version
from a previous post-merge build, which is unable to parse socket
communications. Disable them for now.
This commit will get reverted after suitable testing has been possible
to execute.
v2: Rename the enable var, explain its presence further in a comment
(Kamil)
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Diffstat (limited to 'runner')
| -rw-r--r-- | runner/executor.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/runner/executor.c b/runner/executor.c index 669e85241..a79a34e0e 100644 --- a/runner/executor.c +++ b/runner/executor.c @@ -1659,7 +1659,14 @@ static int execute_next_entry(struct execute_state *state, sigprocmask(SIG_UNBLOCK, sigmask, NULL); - if (socketfd >= 0 && !getenv("IGT_RUNNER_DISABLE_SOCKET_COMMUNICATION")) { + if (socketfd >= 0 && !getenv("IGT_RUNNER_DISABLE_SOCKET_COMMUNICATION") && + getenv("IGT_RUNNER_ENABLE_SOCKET_COMMUNICATION_FIXME")) { + /* + * The variable for enabling socket comms is + * temporary. Do not use unless you really + * know what you're doing. Requiring it will + * go away later. + */ snprintf(envstring, sizeof(envstring), "%d", socketfd); setenv("IGT_RUNNER_SOCKET_FD", envstring, 1); } |