Re: [PATCH] signal: remove duplicate include in signal.c

From: Jan Kara
Date: Mon Nov 01 2021 - 08:19:37 EST


On Mon 01-11-21 07:43:38, cgel.zte@xxxxxxxxx wrote:
> From: Zhang Mingyu <zhang.mingyu@xxxxxxxxxx>
>
> 'asm/fpu/xstate.h' included in 'arch/x86/kernel/signal.c'
> is duplicated.It is also included on the 45 line.
>
> Reported-by: Zeal Robot <zealci@xxxxxxxxxx>
> Signed-off-by: Zhang Mingyu <zhang.mingyu@xxxxxxxxxx>

I guess you've sent this cleanup to a wrong set of people... You need to
send it to x86 maintainters. Thanks.

Honza

> ---
> arch/x86/kernel/signal.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/x86/kernel/signal.c b/arch/x86/kernel/signal.c
> index ec71e06ae364..fad9c4adcda6 100644
> --- a/arch/x86/kernel/signal.c
> +++ b/arch/x86/kernel/signal.c
> @@ -42,7 +42,6 @@
> #include <linux/compat.h>
> #include <asm/proto.h>
> #include <asm/ia32_unistd.h>
> -#include <asm/fpu/xstate.h>
> #endif /* CONFIG_X86_64 */
>
> #include <asm/syscall.h>
> --
> 2.25.1
>
--
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR