lkml.org 
[lkml]   [2002]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] use 1ULL instead of 1UL in kernel/signal.c

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/

\
 
 \ /
  Last update: 2005-03-22 13:30    [W:0.052 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site