Re: [PATCH] use 1ULL instead of 1UL in kernel/signal.c

From: Matthew Wilcox (willy@debian.org)
Date: Wed Oct 23 2002 - 08:17:12 EST

On Wed, Oct 23, 2002 at 12:17:12AM +0100, Alan Cox wrote:
> Care to move the define into include/asm-foo then ?

How about this instead? All other arches define SIGRTMIN to be 32, so
this only affects PA.

Index: kernel/signal.c
===================================================================
RCS file: /var/cvs/linux-2.5/kernel/signal.c,v
retrieving revision 1.1.2.4
diff -u -p -r1.1.2.4 signal.c
--- kernel/signal.c 21 Oct 2002 03:07:20 -0000 1.1.2.4
+++ kernel/signal.c 23 Oct 2002 13:14:33 -0000
@@ -96,7 +96,12 @@ int max_queued_signals = 1024;
 #define M_SIGEMT 0
 #endif
 
+#if SIGRTMIN > 32
+#define M(sig) (1ULL << (sig))
+#else
 #define M(sig) (1UL << (sig))
+#endif
+#define T(sig, mask) (M(sig) & mask)
 
 #define SIG_USER_SPECIFIC_MASK (\
         M(SIGILL) | M(SIGTRAP) | M(SIGABRT) | M(SIGBUS) | \
@@ -130,9 +135,6 @@ int max_queued_signals = 1024;
         M(SIGQUIT) | M(SIGILL) | M(SIGTRAP) | M(SIGABRT) | \
         M(SIGFPE) | M(SIGSEGV) | M(SIGBUS) | M(SIGSYS) | \
         M(SIGXCPU) | M(SIGXFSZ) | M_SIGEMT )
-
-#define T(sig, mask) \
- ((1UL << (sig)) & mask)
 
 #define sig_user_specific(sig) \
                 (((sig) < SIGRTMIN) && T(sig, SIG_USER_SPECIFIC_MASK)) 

-- 
Revolutions do not require corporate support.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

This archive was generated by hypermail 2b29 : Wed Oct 23 2002 - 22:01:03 EST