From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stafford Horne Subject: Re: [PATCH] kernel: exit: cleanup release_thread() Date: Sun, 21 Aug 2022 17:42:51 +0000 Message-ID: References: <20220819014406.32266-1-wangkefeng.wang@huawei.com> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc; bh=7Nrg8mvlqxFEJfzKh/tw2+dBWE8D/s98GD2ZnmZvMl4=; b=F0ptjTbxLv2NJaVf2gvBlBshDMfCokLJPPMsIV35UHBlxVECPPma43FPvPitouEYuo 8dgtnhsmPRFBvIO948wmtVOueVdGaHu/6efegZ+eBz+dquE2oOi9XMsurik6HL7oB6MX jV8pUVxH3f1RrwF0NH6pncq2tH6AwoMHlp65ZDTNCyiT6v1mar/Hou3sFmBJRov6RKag O1el0UDp9u/+GhEvUUZhnnL+z+SJXn9IfDfG4Dlp/Iuz3gqoHz6wEPHD+abHkg+f2q3v w/lO0XWXR94cHio73ZqXyXjLv1uGE7J2Xgag56CksB4nH92y765Uk+pyJbn+nJIZ5N3k uDSQ== Content-Disposition: inline In-Reply-To: <20220819014406.32266-1-wangkefeng.wang@huawei.com> List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Kefeng Wang Cc: Richard Henderson , Ivan Kokshaysky , Matt Turner , Vineet Gupta , Russell King , Catalin Marinas , Will Deacon , Guo Ren , Brian Cain , Huacai Chen , WANG Xuerui , Geert Uytterhoeven , Michal Simek , Thomas Bogendoerfer , Dinh Nguyen , Jonas Bonn , Stefan Kristiansson , "James E . J . Bottomley" , Helge Deller 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 > --- > 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