2022-08-21 17:44:37

by Stafford Horne

[permalink] [raw]
Subject: Re: [PATCH] kernel: exit: cleanup release_thread()

On Fri, Aug 19, 2022 at 09:44:06AM +0800, Kefeng Wang wrote:
> Only x86 has own release_thread(), introduce a new weak
> release_thread() function to clean empty definitions in
> other ARCHs.
>
> Signed-off-by: Kefeng Wang <[email protected]>
> ---

> arch/openrisc/include/asm/processor.h | 1 -
> arch/openrisc/kernel/process.c | 4 ----

> diff --git a/arch/openrisc/include/asm/processor.h b/arch/openrisc/include/asm/processor.h
> index aa1699c18add..ed9efb430afa 100644
> --- a/arch/openrisc/include/asm/processor.h
> +++ b/arch/openrisc/include/asm/processor.h
> @@ -72,7 +72,6 @@ struct thread_struct {
>
>
> void start_thread(struct pt_regs *regs, unsigned long nip, unsigned long sp);
> -void release_thread(struct task_struct *);
> unsigned long __get_wchan(struct task_struct *p);
>
> #define cpu_relax() barrier()
> diff --git a/arch/openrisc/kernel/process.c b/arch/openrisc/kernel/process.c
> index 52dc983ddeba..f94b5ec06786 100644
> --- a/arch/openrisc/kernel/process.c
> +++ b/arch/openrisc/kernel/process.c
> @@ -125,10 +125,6 @@ void show_regs(struct pt_regs *regs)
> show_registers(regs);
> }
>
> -void release_thread(struct task_struct *dead_task)
> -{
> -}
> -
> /*
> * Copy the thread-specific (arch specific) info from the current
> * process to the new one p

For OpenRISC bits.

Acked-by: Stafford Horne <[email protected]>