public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] runner: Show more elements of the signaler's argv[]
@ 2019-09-23 11:20 Chris Wilson
  2019-09-23 12:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Chris Wilson @ 2019-09-23 11:20 UTC (permalink / raw)
  To: igt-dev; +Cc: Petri Latvala

/proc/$pid/cmdline is the entire argv[] including NUL-terminator.
Replace the NULs with spaces so we get a better idea of who the
signaler was, as often it is a subprocess (such as a child of sudo,
or worse java).

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Petri Latvala <petri.latvala@intel.com>
---
 runner/executor.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/runner/executor.c b/runner/executor.c
index f7183293d..91e8a7b20 100644
--- a/runner/executor.c
+++ b/runner/executor.c
@@ -625,7 +625,12 @@ static const char *get_cmdline(pid_t pid, char *buf, size_t len)
 	if (len < 0)
 		return "unknown";
 
-	buf[len] = '\0';
+	/* cmdline is the whole argv[], completed with NUL-terminators */
+	for (size_t i = 0; i < len; i++)
+		if (buf[i] == '\0')
+			buf[i] = ' ';
+
+	buf[len] = '\0'; /* but make sure that we return a valid string! */
 	return buf;
 }
 
@@ -907,7 +912,7 @@ static int monitor_output(pid_t child,
 			} else {
 				/* We're dying, so we're taking them with us */
 				if (settings->log_level >= LOG_LEVEL_NORMAL) {
-					char comm[80];
+					char comm[120];
 
 					outf("Abort requested by %s [%d] via %s, terminating children\n",
 					     get_cmdline(siginfo.ssi_pid, comm, sizeof(comm)),
-- 
2.23.0

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-09-23 18:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-09-23 11:20 [igt-dev] [PATCH i-g-t] runner: Show more elements of the signaler's argv[] Chris Wilson
2019-09-23 12:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2019-09-23 13:08 ` [igt-dev] [PATCH i-g-t] " Petri Latvala
2019-09-23 18:52 ` [igt-dev] ✗ Fi.CI.IGT: failure for " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox