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

From: Matthew Wilcox (willy@debian.org)
Date: Tue Oct 22 2002 - 16:27:19 EST

On PA-RISC we have 36 signals defined for hpux compatibility. So M()
and T() in kernel/signal.c try to do (1UL << 33) which is garbage on 32-bit
architectures. How do people feel about this patch?

--- kernel/signal.c 21 Oct 2002 03:46:14 -0000 1.5
+++ kernel/signal.c 22 Oct 2002 21:25:29 -0000
@@ -96,7 +96,7 @@ int max_queued_signals = 1024;
 #define M_SIGEMT 0
 #endif
 
-#define M(sig) (1UL << (sig))
+#define M(sig) (1ULL << (sig))
 
 #define SIG_USER_SPECIFIC_MASK (\
        M(SIGILL) | M(SIGTRAP) | M(SIGABRT) | M(SIGBUS) | \
@@ -132,7 +132,7 @@ int max_queued_signals = 1024;
         M(SIGXCPU) | M(SIGXFSZ) | M_SIGEMT )
 
 #define T(sig, mask) \
- ((1UL << (sig)) & mask)
+ ((1ULL << (sig)) & mask)
 
 #define sig_user_specific(sig) \
                (((sig) < SIGRTMIN) && T(sig, SIG_USER_SPECIFIC_MASK))

(patch copied & pasted; whitespace is munged. This is an indication I
expect to get some pushback on this issue ;-) 

-- 
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:00 EST