From: Simon Marchi <simon.marchi@polymtl.ca>
Date: Mon, 10 May 2021 16:13:36 +0000 (-0400)
Subject: gdb, gdbserver: make status_to_str display the signal name
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=31aceaef1cba910c84b81c78064d226fd10d8976;p=binutils-gdb.git

gdb, gdbserver: make status_to_str display the signal name

I was looking at some "set debug lin-lwp" logs, and saw that a thread
received the "Child exited" signal.  It took me a moment to realize that
this was SIGCHLD.  I then thought that it would be nice for
status_to_str to show the signal name (SIGCHLD) in addition to the
description "Child exited", since people are much more used to referring
to signals using their names.

Fortunately, libiberty contains a handy function to get the signal name
from the signal number, strsigno, use that.

The output of "set debug lin-lwp" now looks like:

    [linux-nat] linux_nat_wait_1: waitpid 1209631 received SIGTRAP - Trace/breakpoint trap (stopped)

gdb/ChangeLog:

	* nat/linux-waitpid.c (status_to_str): Show signal name.

Change-Id: I8ad9b1e744dd64461fd87b08d5c29f9ef97c4691
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 8fa69802341..52d5faac8a9 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2021-05-10  Simon Marchi  <simon.marchi@polymtl.ca>
+
+	* nat/linux-waitpid.c (status_to_str): Show signal name.
+
 2021-05-09  Andrew Burgess  <andrew.burgess@embecosm.com>
 
 	* python/py-breakpoint.c (pybp_debug): New static global.
diff --git a/gdb/nat/linux-waitpid.c b/gdb/nat/linux-waitpid.c
index f205df7a246..99489040fb0 100644
--- a/gdb/nat/linux-waitpid.c
+++ b/gdb/nat/linux-waitpid.c
@@ -32,14 +32,16 @@ status_to_str (int status)
   if (WIFSTOPPED (status))
     {
       if (WSTOPSIG (status) == SYSCALL_SIGTRAP)
-	return string_printf ("%s (stopped at syscall)",
-			      strsignal (SIGTRAP));
+	return string_printf ("%s - %s (stopped at syscall)",
+			      strsigno (SIGTRAP), strsignal (SIGTRAP));
       else
-	return string_printf ("%s (stopped)",
+	return string_printf ("%s - %s (stopped)",
+			      strsigno (WSTOPSIG (status)),
 			      strsignal (WSTOPSIG (status)));
     }
   else if (WIFSIGNALED (status))
-    return string_printf ("%s (terminated)",
+    return string_printf ("%s - %s (terminated)",
+			  strsigno (WTERMSIG (status)),
 			  strsignal (WTERMSIG (status)));
   else
     return string_printf ("%d (exited)", WEXITSTATUS (status));