[PATCH] sh: use common signalent.h

Denys Vlasenko dvlasenk at redhat.com
Fri Feb 20 16:24:59 UTC 2009


On Fri, 2009-02-13 at 17:29 -0500, Mike Frysinger wrote:
> The sh port uses the same signal list as the common Linux one, so drop the
> arch specific copy.


Applying, thanks!

What to do with ia64? It also has the same signal assignments,
and I believe SIGRT_nn are handled by strace itself, so
current mess is not necessary:

--- signalent.h 2000-02-03 23:01:30.000000000 +0100
+++ ../signalent.h      2001-08-03 23:53:51.000000000 +0200
@@ -30,35 +30,4 @@
        "SIGIO",        /* 29 */
        "SIGPWR",       /* 30 */
        "SIGSYS",       /* 31 */
-       "SIGRT0",       /* 32 */
-       "SIGRT1",       /* 33 */
-       "SIGRT2",       /* 34 */
-       "SIGRT3",       /* 35 */
-       "SIGRT4",       /* 36 */
-       "SIGRT5",       /* 37 */
-       "SIGRT6",       /* 38 */
-       "SIGRT7",       /* 39 */
-       "SIGRT8",       /* 40 */
-       "SIGRT9",       /* 41 */
-       "SIGRT10",      /* 42 */
-       "SIGRT11",      /* 43 */
-       "SIGRT12",      /* 44 */
-       "SIGRT13",      /* 45 */
-       "SIGRT14",      /* 46 */
-       "SIGRT15",      /* 47 */
-       "SIGRT16",      /* 48 */
-       "SIGRT17",      /* 49 */
-       "SIGRT18",      /* 50 */
-       "SIGRT19",      /* 51 */
-       "SIGRT20",      /* 52 */
-       "SIGRT21",      /* 53 */
-       "SIGRT22",      /* 54 */
-       "SIGRT23",      /* 55 */
-       "SIGRT24",      /* 56 */
-       "SIGRT25",      /* 57 */
-       "SIGRT26",      /* 58 */
-       "SIGRT27",      /* 59 */
-       "SIGRT28",      /* 60 */
-       "SIGRT29",      /* 61 */
-       "SIGRT30",      /* 62 */
-       "SIGRT31",      /* 63 */
+       "SIGRTMIN",     /* 32 */

(it also misses SIGRT31 which does exist).


And powerpc, s390, s390x have the only difference:

--- signalent.h 2003-01-14 08:53:30.000000000 +0100
+++ ../signalent.h      2001-08-03 23:53:51.000000000 +0200
@@ -29,5 +29,5 @@
        "SIGWINCH",     /* 28 */
        "SIGIO",        /* 29 */
        "SIGPWR",       /* 30 */
-       "SIGUNUSED",    /* 31 */
+       "SIGSYS",       /* 31 */
        "SIGRTMIN",     /* 32 */

I wonder whether this is just a mistake?
--
vda






More information about the Strace-devel mailing list