[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250324162432.723301-1-alex@alexrp.com>
Date: Mon, 24 Mar 2025 17:24:32 +0100
From: Alex Rønne Petersen <alex@...xrp.com>
To: musl@...ts.openwall.com
Cc: Alex Rønne Petersen <alex@...xrp.com>
Subject: [PATCH] powerpc, powerpc64, s390x: remove unsigned suffix on SA_* macros
I'm not really sure why this was ever here. It's not done for any other ports,
and AFAICT glibc doesn't do this either. It causes -Wsign-conversion warnings in
code that's completely fine, such as `act->sa_flags & SA_SIGINFO`.
---
arch/powerpc/bits/signal.h | 16 ++++++++--------
arch/powerpc64/bits/signal.h | 16 ++++++++--------
arch/s390x/bits/signal.h | 16 ++++++++--------
3 files changed, 24 insertions(+), 24 deletions(-)
diff --git a/arch/powerpc/bits/signal.h b/arch/powerpc/bits/signal.h
index c1bf3caf..de1af5c5 100644
--- a/arch/powerpc/bits/signal.h
+++ b/arch/powerpc/bits/signal.h
@@ -70,14 +70,14 @@ typedef struct __ucontext {
mcontext_t uc_mcontext;
} ucontext_t;
-#define SA_NOCLDSTOP 1U
-#define SA_NOCLDWAIT 2U
-#define SA_SIGINFO 4U
-#define SA_ONSTACK 0x08000000U
-#define SA_RESTART 0x10000000U
-#define SA_NODEFER 0x40000000U
-#define SA_RESETHAND 0x80000000U
-#define SA_RESTORER 0x04000000U
+#define SA_NOCLDSTOP 1
+#define SA_NOCLDWAIT 2
+#define SA_SIGINFO 4
+#define SA_ONSTACK 0x08000000
+#define SA_RESTART 0x10000000
+#define SA_NODEFER 0x40000000
+#define SA_RESETHAND 0x80000000
+#define SA_RESTORER 0x04000000
#endif
diff --git a/arch/powerpc64/bits/signal.h b/arch/powerpc64/bits/signal.h
index d5493b18..cf25d498 100644
--- a/arch/powerpc64/bits/signal.h
+++ b/arch/powerpc64/bits/signal.h
@@ -61,14 +61,14 @@ typedef struct __ucontext {
mcontext_t uc_mcontext;
} ucontext_t;
-#define SA_NOCLDSTOP 1U
-#define SA_NOCLDWAIT 2U
-#define SA_SIGINFO 4U
-#define SA_ONSTACK 0x08000000U
-#define SA_RESTART 0x10000000U
-#define SA_NODEFER 0x40000000U
-#define SA_RESETHAND 0x80000000U
-#define SA_RESTORER 0x04000000U
+#define SA_NOCLDSTOP 1
+#define SA_NOCLDWAIT 2
+#define SA_SIGINFO 4
+#define SA_ONSTACK 0x08000000
+#define SA_RESTART 0x10000000
+#define SA_NODEFER 0x40000000
+#define SA_RESETHAND 0x80000000
+#define SA_RESTORER 0x04000000
#endif
diff --git a/arch/s390x/bits/signal.h b/arch/s390x/bits/signal.h
index e5aca4ba..ba131e6b 100644
--- a/arch/s390x/bits/signal.h
+++ b/arch/s390x/bits/signal.h
@@ -72,14 +72,14 @@ typedef struct __ucontext {
sigset_t uc_sigmask;
} ucontext_t;
-#define SA_NOCLDSTOP 1U
-#define SA_NOCLDWAIT 2U
-#define SA_SIGINFO 4U
-#define SA_ONSTACK 0x08000000U
-#define SA_RESTART 0x10000000U
-#define SA_NODEFER 0x40000000U
-#define SA_RESETHAND 0x80000000U
-#define SA_RESTORER 0x04000000U
+#define SA_NOCLDSTOP 1
+#define SA_NOCLDWAIT 2
+#define SA_SIGINFO 4
+#define SA_ONSTACK 0x08000000
+#define SA_RESTART 0x10000000
+#define SA_NODEFER 0x40000000
+#define SA_RESETHAND 0x80000000
+#define SA_RESTORER 0x04000000
#endif
--
2.43.0
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
