Re: [PATCH v2 3/3] linux-user: Remove TARGET_SIGSTKSZ

[Laurent Vivier]

Le 26/11/2021 à 03:23, Song Gao a écrit :
> TARGET_SIGSTKSZ is not used, we should remove it.
>
> Signed-off-by: Song Gao <gaosong@loongson.cn>
> ---
>   linux-user/alpha/target_signal.h  | 1 -
>   linux-user/generic/signal.h       | 1 -
>   linux-user/hppa/target_signal.h   | 1 -
>   linux-user/mips/target_signal.h   | 1 -
>   linux-user/mips64/target_signal.h | 1 -
>   linux-user/sparc/target_signal.h  | 1 -
>   6 files changed, 6 deletions(-)
>
> diff --git a/linux-user/alpha/target_signal.h b/linux-user/alpha/target_signal.h
> index 0b6a39d..bbb06e5 100644
> --- a/linux-user/alpha/target_signal.h
> +++ b/linux-user/alpha/target_signal.h
> @@ -62,7 +62,6 @@ typedef struct target_sigaltstack {
>   #define TARGET_SA_SIGINFO       0x00000040
>   
>   #define TARGET_MINSIGSTKSZ	4096
> -#define TARGET_SIGSTKSZ                16384
>   
>   /* From <asm/gentrap.h>.  */
>   #define TARGET_GEN_INTOVF      -1      /* integer overflow */
> diff --git a/linux-user/generic/signal.h b/linux-user/generic/signal.h
> index ceaf8a8..6fd05b7 100644
> --- a/linux-user/generic/signal.h
> +++ b/linux-user/generic/signal.h
> @@ -69,7 +69,6 @@ typedef struct target_sigaltstack {
>   #define TARGET_SS_DISABLE 2
>   
>   #define TARGET_MINSIGSTKSZ     2048
> -#define TARGET_SIGSTKSZ        8192
>   
>   /* bit-flags */
>   #define TARGET_SS_AUTODISARM (1U << 31) /* disable sas during sighandling */
> diff --git a/linux-user/hppa/target_signal.h b/linux-user/hppa/target_signal.h
> index d558119..af6c2fc 100644
> --- a/linux-user/hppa/target_signal.h
> +++ b/linux-user/hppa/target_signal.h
> @@ -64,7 +64,6 @@ typedef struct target_sigaltstack {
>   #define TARGET_SA_NOCLDWAIT     0x00000080
>   
>   #define TARGET_MINSIGSTKSZ	2048
> -#define TARGET_SIGSTKSZ                8192
>   
>   /* bit-flags */
>   #define TARGET_SS_AUTODISARM (1U << 31) /* disable sas during sighandling */
> diff --git a/linux-user/mips/target_signal.h b/linux-user/mips/target_signal.h
> index 780a4dd..fa542c1 100644
> --- a/linux-user/mips/target_signal.h
> +++ b/linux-user/mips/target_signal.h
> @@ -67,7 +67,6 @@ typedef struct target_sigaltstack {
>   #define TARGET_SA_RESTORER      0x04000000      /* Only for O32 */
>   
>   #define TARGET_MINSIGSTKSZ    2048
> -#define TARGET_SIGSTKSZ       8192
>   
>   #if defined(TARGET_ABI_MIPSO32)
>   /* compare linux/arch/mips/kernel/signal.c:setup_frame() */
> diff --git a/linux-user/mips64/target_signal.h 
> b/linux-user/mips64/target_signal.h
> index 275e9b7..b05098f 100644
> --- a/linux-user/mips64/target_signal.h
> +++ b/linux-user/mips64/target_signal.h
> @@ -65,7 +65,6 @@ typedef struct target_sigaltstack {
>   #define TARGET_SA_RESETHAND     0x80000000
>   
>   #define TARGET_MINSIGSTKSZ    2048
> -#define TARGET_SIGSTKSZ       8192
>   
>   /* bit-flags */
>   #define TARGET_SS_AUTODISARM (1U << 31) /* disable sas during sighandling */
> diff --git a/linux-user/sparc/target_signal.h b/linux-user/sparc/target_signal.h
> index e661ddd..87757f0 100644
> --- a/linux-user/sparc/target_signal.h
> +++ b/linux-user/sparc/target_signal.h
> @@ -65,7 +65,6 @@ typedef struct target_sigaltstack {
>   #define TARGET_ARCH_HAS_KA_RESTORER 1
>   
>   #define TARGET_MINSIGSTKSZ	4096
> -#define TARGET_SIGSTKSZ                16384
>   
>   #ifdef TARGET_ABI32
>   #define TARGET_ARCH_HAS_SETUP_FRAME

Applied to my linux-user-for-7.0 branch.

Thanks,
Laurent