diff options
author | Rich Felker <dalias@aerifal.cx> | 2015-03-17 23:12:48 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2015-03-18 00:31:37 -0400 |
commit | d5a5045382315e36588ca225889baa36ed0ed38f (patch) | |
tree | b727e3ec85f49384a50f09568c62c0bb9410b6bd /arch | |
parent | 76fd01177a32d602bc5ee7bd7ba65316adbae611 (diff) | |
download | musl-d5a5045382315e36588ca225889baa36ed0ed38f.tar.gz musl-d5a5045382315e36588ca225889baa36ed0ed38f.tar.bz2 musl-d5a5045382315e36588ca225889baa36ed0ed38f.tar.xz musl-d5a5045382315e36588ca225889baa36ed0ed38f.zip |
fix MINSIGSTKSZ values for archs with large signal contexts
the previous values (2k min and 8k default) were too small for some
archs. aarch64 reserves 4k in the signal context for future extensions
and requires about 4.5k total, and powerpc reportedly uses over 2k.
the new minimums are chosen to fit the saved context and also allow a
minimal signal handler to run.
since the default (SIGSTKSZ) has always been 6k larger than the
minimum, it is also increased to maintain the 6k usable by the signal
handler. this happens to be able to store one pathname buffer and
should be sufficient for calling any function in libc that doesn't
involve conversion between floating point and decimal representations.
x86 (both 32-bit and 64-bit variants) may also need a larger minimum
(around 2.5k) in the future to support avx-512, but the values on
these archs are left alone for now pending further analysis.
the value for PTHREAD_STACK_MIN is not increased to match MINSIGSTKSZ
at this time. this is so as not to preclude applications from using
extremely small thread stacks when they know they will not be handling
signals. unfortunately cancellation and multi-threaded set*id() use
signals as an implementation detail and therefore require a stack
large enough for a signal context, so applications which use extremely
small thread stacks may still need to avoid using these features.
Diffstat (limited to 'arch')
-rw-r--r-- | arch/aarch64/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/arm/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/i386/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/microblaze/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/mips/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/or1k/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/powerpc/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/sh/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/x32/bits/signal.h | 5 | ||||
-rw-r--r-- | arch/x86_64/bits/signal.h | 5 |
10 files changed, 50 insertions, 0 deletions
diff --git a/arch/aarch64/bits/signal.h b/arch/aarch64/bits/signal.h index 430da95a..ef07cc9f 100644 --- a/arch/aarch64/bits/signal.h +++ b/arch/aarch64/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 6144 +#define SIGSTKSZ 12288 +#endif + #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE) typedef unsigned long greg_t; typedef unsigned long gregset_t[34]; diff --git a/arch/arm/bits/signal.h b/arch/arm/bits/signal.h index 048635fa..9d19eeb2 100644 --- a/arch/arm/bits/signal.h +++ b/arch/arm/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE) typedef int greg_t, gregset_t[18]; typedef struct sigcontext diff --git a/arch/i386/bits/signal.h b/arch/i386/bits/signal.h index adf0c075..1f9085a5 100644 --- a/arch/i386/bits/signal.h +++ b/arch/i386/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #ifdef _GNU_SOURCE #define REG_GS 0 #define REG_FS 1 diff --git a/arch/microblaze/bits/signal.h b/arch/microblaze/bits/signal.h index 01fa7bf5..4aa231a2 100644 --- a/arch/microblaze/bits/signal.h +++ b/arch/microblaze/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE) typedef unsigned long greg_t, gregset_t[38]; typedef struct sigcontext diff --git a/arch/mips/bits/signal.h b/arch/mips/bits/signal.h index 48bd1105..e2218427 100644 --- a/arch/mips/bits/signal.h +++ b/arch/mips/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE) typedef unsigned long long greg_t, gregset_t[32]; typedef struct { diff --git a/arch/or1k/bits/signal.h b/arch/or1k/bits/signal.h index c5e9c0cd..be576d1d 100644 --- a/arch/or1k/bits/signal.h +++ b/arch/or1k/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE) typedef unsigned long greg_t, gregset_t[34]; typedef struct sigcontext { diff --git a/arch/powerpc/bits/signal.h b/arch/powerpc/bits/signal.h index 3f9f810f..17a7354f 100644 --- a/arch/powerpc/bits/signal.h +++ b/arch/powerpc/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 4096 +#define SIGSTKSZ 10240 +#endif + #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE) typedef unsigned long greg_t, gregset_t[48]; diff --git a/arch/sh/bits/signal.h b/arch/sh/bits/signal.h index 41b94376..160311fa 100644 --- a/arch/sh/bits/signal.h +++ b/arch/sh/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #if defined(_GNU_SOURCE) || defined(_BSD_SOURCE) typedef int greg_t, gregset_t[16]; typedef int freg_t, fpregset_t[16]; diff --git a/arch/x32/bits/signal.h b/arch/x32/bits/signal.h index 2379f1ca..4c4adf31 100644 --- a/arch/x32/bits/signal.h +++ b/arch/x32/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #ifdef _GNU_SOURCE #define REG_R8 0 #define REG_R9 1 diff --git a/arch/x86_64/bits/signal.h b/arch/x86_64/bits/signal.h index b3a76165..e3c31417 100644 --- a/arch/x86_64/bits/signal.h +++ b/arch/x86_64/bits/signal.h @@ -1,6 +1,11 @@ #if defined(_POSIX_SOURCE) || defined(_POSIX_C_SOURCE) \ || defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#if defined(_XOPEN_SOURCE) || defined(_GNU_SOURCE) || defined(_BSD_SOURCE) +#define MINSIGSTKSZ 2048 +#define SIGSTKSZ 8192 +#endif + #ifdef _GNU_SOURCE #define REG_R8 0 #define REG_R9 1 |