Re: [PATCH] powerpc64: Increase SIGSTKSZ and MINSIGSTKSZ

[Paul E Murphy <murphyp@linux.ibm.com>]

On 12/8/22 7:16 PM, Rajalakshmi Srinivasaraghavan via Libc-alpha wrote:
> This patch increases the value of SIGSTKSZ and MINSIGSTKSZ
> for powerpc64 similar to the kernel commit
> 2f82ec19757f58549467db568c56e7dfff8af283 to allow
> further expansion of the signal stack frame size.

This LGTM. The kernel's commit message is a little confusing to someone 
who didn't experience the original issue, but make sense once looking 
through the tree of referenced commits.

> ---
>   sysdeps/unix/sysv/linux/powerpc/bits/sigstack.h | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/sysdeps/unix/sysv/linux/powerpc/bits/sigstack.h b/sysdeps/unix/sysv/linux/powerpc/bits/sigstack.h
> index abc87cd7a6..4bff1fe1e7 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/bits/sigstack.h
> +++ b/sysdeps/unix/sysv/linux/powerpc/bits/sigstack.h
> @@ -23,10 +23,15 @@
>   # error "Never include this file directly.  Use <signal.h> instead"
>   #endif
> 
> +#ifdef __powerpc64__
> +#define MINSIGSTKSZ	8192
> +#define SIGSTKSZ	32768
> +#else
>   /* Minimum stack size for a signal handler.  */
>   #define MINSIGSTKSZ	4096
> 
>   /* System default stack size.  */
>   #define SIGSTKSZ	16384
> +#endif
> 
>   #endif /* bits/sigstack.h */