[PATCH] s390: fix SIGNAL_FRAMESIZE macro
Tuan Hoang
tmhoang at linux.ibm.com
Wed Mar 13 22:32:12 UTC 2019
Since __SIGNAL_FRAMESIZE is in private name space, it is better not to
use it directly. This macro has been stable on s390/s390x so it is safe
to use the integer value.
Signed-off-by: Tuan Hoang <tmhoang at linux.ibm.com>
---
linux/s390/arch_sigreturn.c | 2 +-
linux/s390/rt_sigframe.h | 8 +++++++-
linux/s390x/arch_sigreturn.c | 4 +---
3 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/linux/s390/arch_sigreturn.c b/linux/s390/arch_sigreturn.c
index 1d0f0e00..4a9ca8ec 100644
--- a/linux/s390/arch_sigreturn.c
+++ b/linux/s390/arch_sigreturn.c
@@ -6,7 +6,7 @@
*/
#ifndef SIGNAL_FRAMESIZE
-# define SIGNAL_FRAMESIZE __SIGNAL_FRAMESIZE
+# define SIGNAL_FRAMESIZE 96
#endif
#ifndef PTR_TYPE
# define PTR_TYPE unsigned long
diff --git a/linux/s390/rt_sigframe.h b/linux/s390/rt_sigframe.h
index e11573a4..101ca79c 100644
--- a/linux/s390/rt_sigframe.h
+++ b/linux/s390/rt_sigframe.h
@@ -10,8 +10,14 @@
# include <signal.h>
+#if defined __s390__
+# define __signal_framesize 96
+#elif defined __s390x__
+# define __signal_framesize 160
+#endif
+
typedef struct {
- uint8_t callee_used_stack[__SIGNAL_FRAMESIZE];
+ uint8_t callee_used_stack[__signal_framesize];
uint16_t svc_insn;
siginfo_t info;
ucontext_t uc;
diff --git a/linux/s390x/arch_sigreturn.c b/linux/s390x/arch_sigreturn.c
index 55930a35..2d75a8c6 100644
--- a/linux/s390x/arch_sigreturn.c
+++ b/linux/s390x/arch_sigreturn.c
@@ -7,9 +7,6 @@
#include <stdint.h>
-#define S390_SIGNAL_FRAMESIZE 96
-
-#define SIGNAL_FRAMESIZE S390_SIGNAL_FRAMESIZE
#define PTR_TYPE uint32_t
#define arch_sigreturn s390_arch_sigreturn
#include "s390/arch_sigreturn.c"
@@ -17,6 +14,7 @@
#undef PTR_TYPE
#undef SIGNAL_FRAMESIZE
+#define SIGNAL_FRAMESIZE 160
#define arch_sigreturn s390x_arch_sigreturn
#include "s390/arch_sigreturn.c"
#undef arch_sigreturn
--
2.20.1
More information about the Strace-devel
mailing list