]> git.baikalelectronics.ru Git - kernel.git/commitdiff
kernel: exit: cleanup release_thread()
authorKefeng Wang <wangkefeng.wang@huawei.com>
Fri, 19 Aug 2022 01:44:06 +0000 (09:44 +0800)
committerAndrew Morton <akpm@linux-foundation.org>
Mon, 12 Sep 2022 04:55:07 +0000 (21:55 -0700)
Only x86 has own release_thread(), introduce a new weak release_thread()
function to clean empty definitions in other ARCHs.

Link: https://lkml.kernel.org/r/20220819014406.32266-1-wangkefeng.wang@huawei.com
Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
Acked-by: Guo Ren <guoren@kernel.org> [csky]
Acked-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Brian Cain <bcain@quicinc.com>
Acked-by: Michael Ellerman <mpe@ellerman.id.au> [powerpc]
Acked-by: Stafford Horne <shorne@gmail.com> [openrisc]
Acked-by: Catalin Marinas <catalin.marinas@arm.com> [arm64]
Acked-by: Huacai Chen <chenhuacai@kernel.org> [LoongArch]
Cc: Alexander Gordeev <agordeev@linux.ibm.com>
Cc: Anton Ivanov <anton.ivanov@cambridgegreys.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Christian Borntraeger <borntraeger@linux.ibm.com>
Cc: Christophe Leroy <christophe.leroy@csgroup.eu>
Cc: Chris Zankel <chris@zankel.net>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Dinh Nguyen <dinguyen@kernel.org>
Cc: Guo Ren <guoren@kernel.org> [csky]
Cc: Heiko Carstens <hca@linux.ibm.com>
Cc: Helge Deller <deller@gmx.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Ivan Kokshaysky <ink@jurassic.park.msu.ru>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Johannes Berg <johannes@sipsolutions.net>
Cc: Jonas Bonn <jonas@southpole.se>
Cc: Matt Turner <mattst88@gmail.com>
Cc: Max Filippov <jcmvbkbc@gmail.com>
Cc: Michal Simek <monstr@monstr.eu>
Cc: Nicholas Piggin <npiggin@gmail.com>
Cc: Palmer Dabbelt <palmer@dabbelt.com>
Cc: Paul Walmsley <paul.walmsley@sifive.com>
Cc: Richard Henderson <richard.henderson@linaro.org>
Cc: Richard Weinberger <richard@nod.at>
Cc: Rich Felker <dalias@libc.org>
Cc: Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>
Cc: Sven Schnelle <svens@linux.ibm.com>
Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Vasily Gorbik <gor@linux.ibm.com>
Cc: Vineet Gupta <vgupta@kernel.org>
Cc: Will Deacon <will@kernel.org>
Cc: Xuerui Wang <kernel@xen0n.name>
Cc: Yoshinori Sato <ysato@users.osdn.me>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
33 files changed:
arch/alpha/include/asm/processor.h
arch/alpha/kernel/process.c
arch/arc/include/asm/processor.h
arch/arm/include/asm/processor.h
arch/arm/kernel/process.c
arch/arm64/include/asm/processor.h
arch/arm64/kernel/process.c
arch/csky/include/asm/processor.h
arch/hexagon/include/asm/processor.h
arch/hexagon/kernel/process.c
arch/ia64/include/asm/processor.h
arch/loongarch/include/asm/processor.h
arch/m68k/include/asm/processor.h
arch/microblaze/include/asm/processor.h
arch/mips/include/asm/processor.h
arch/nios2/include/asm/processor.h
arch/openrisc/include/asm/processor.h
arch/openrisc/kernel/process.c
arch/parisc/include/asm/processor.h
arch/parisc/kernel/process.c
arch/powerpc/include/asm/processor.h
arch/powerpc/kernel/process.c
arch/riscv/include/asm/processor.h
arch/s390/include/asm/processor.h
arch/sh/include/asm/processor_32.h
arch/sh/kernel/process_32.c
arch/sparc/include/asm/processor_32.h
arch/sparc/include/asm/processor_64.h
arch/um/include/asm/processor-generic.h
arch/x86/include/asm/processor.h
arch/xtensa/include/asm/processor.h
include/linux/sched/task.h
kernel/exit.c

index 43e234c518b1c05bf92a3fa452240a5ab6d67a09..714abe494e5fdc5996be1fd46d7b217916678902 100644 (file)
@@ -36,8 +36,6 @@ extern void start_thread(struct pt_regs *, unsigned long, unsigned long);
 
 /* Free all resources held by a thread. */
 struct task_struct;
-extern void release_thread(struct task_struct *);
-
 unsigned long __get_wchan(struct task_struct *p);
 
 #define KSTK_EIP(tsk) (task_pt_regs(tsk)->pc)
index e2e25f8b5e76c7285ec07c704b8c4887b5f3718b..dbf1bc5e2ad25d3441acdc04ed90200abc93d57e 100644 (file)
@@ -225,11 +225,6 @@ flush_thread(void)
        current_thread_info()->pcb.unique = 0;
 }
 
-void
-release_thread(struct task_struct *dead_task)
-{
-}
-
 /*
  * Copy architecture-specific thread state
  */
index 54db9d7bb562d3f9acb697a89610568076b9b4f9..fb844fce1ab671552e4dc3a7dd0384da4c6e6abf 100644 (file)
@@ -43,9 +43,6 @@ struct task_struct;
 #define task_pt_regs(p) \
        ((struct pt_regs *)(THREAD_SIZE + (void *)task_stack_page(p)) - 1)
 
-/* Free all resources held by a thread */
-#define release_thread(thread) do { } while (0)
-
 /*
  * A lot of busy-wait loops in SMP are based off of non-volatile data otherwise
  * get optimised away by gcc
index bdc35c0e8dfb9482ff20759d5373c15cbc95d86b..326864f79d18fe035a1e3b499b7185f8ca53caa7 100644 (file)
@@ -81,9 +81,6 @@ static inline void arch_thread_struct_whitelist(unsigned long *offset,
 /* Forward declaration, a strange C thing */
 struct task_struct;
 
-/* Free all resources held by a thread. */
-extern void release_thread(struct task_struct *);
-
 unsigned long __get_wchan(struct task_struct *p);
 
 #define task_pt_regs(p) \
index 3d9cace638840137622a8c3f4e37209c74b99071..712d3e6d9be905ffe369353c4d79c6743fb792aa 100644 (file)
@@ -232,10 +232,6 @@ void flush_thread(void)
        thread_notify(THREAD_NOTIFY_FLUSH, thread);
 }
 
-void release_thread(struct task_struct *dead_task)
-{
-}
-
 asmlinkage void ret_from_fork(void) __asm__("ret_from_fork");
 
 int copy_thread(struct task_struct *p, const struct kernel_clone_args *args)
index 86eb0bfe3b380a562299216e8862a58edf2ca9b7..4cfb4cd1d47593363e997222088f9ecf26f9b676 100644 (file)
@@ -323,9 +323,6 @@ static inline bool is_ttbr1_addr(unsigned long addr)
 /* Forward declaration, a strange C thing */
 struct task_struct;
 
-/* Free all resources held by a thread. */
-extern void release_thread(struct task_struct *);
-
 unsigned long __get_wchan(struct task_struct *p);
 
 void update_sctlr_el1(u64 sctlr);
index 92bcc1768f0b997b761771ba2dad659a0fe16953..9015f49c206efad37e131d3ea8635c3497cfa81a 100644 (file)
@@ -279,10 +279,6 @@ void flush_thread(void)
        flush_tagged_addr_state();
 }
 
-void release_thread(struct task_struct *dead_task)
-{
-}
-
 void arch_release_task_struct(struct task_struct *tsk)
 {
        fpsimd_release_task(tsk);
index 9638206bc44f74cf62a40d56998493c709c6451f..63ad71fab30d7bbe5af9a42b7581e3c486554a41 100644 (file)
@@ -69,11 +69,6 @@ do {                                                                 \
 /* Forward declaration, a strange C thing */
 struct task_struct;
 
-/* Free all resources held by a thread. */
-static inline void release_thread(struct task_struct *dead_task)
-{
-}
-
 /* Prepare to copy thread state - unlazy all lazy status */
 #define prepare_to_copy(tsk)    do { } while (0)
 
index 615f7e49968e619626f423b10af853f2a80ddbf1..0cd39c2cdf8f7a4073a3cca54a4847935414724b 100644 (file)
@@ -60,10 +60,6 @@ struct thread_struct {
 #define KSTK_EIP(tsk) (pt_elr(task_pt_regs(tsk)))
 #define KSTK_ESP(tsk) (pt_psp(task_pt_regs(tsk)))
 
-/*  Free all resources held by a thread; defined in process.c  */
-extern void release_thread(struct task_struct *dead_task);
-
-/* Get wait channel for task P.  */
 extern unsigned long __get_wchan(struct task_struct *p);
 
 /*  The following stuff is pretty HEXAGON specific.  */
index f0552f98a7bae116a1d5fb133adffc78695c8b62..e15eeaebd78530e90eace2211582a1bb2bf1f32b 100644 (file)
@@ -112,13 +112,6 @@ int copy_thread(struct task_struct *p, const struct kernel_clone_args *args)
        return 0;
 }
 
-/*
- * Release any architecture-specific resources locked by thread
- */
-void release_thread(struct task_struct *dead_task)
-{
-}
-
 /*
  * Some archs flush debug and FPU info here
  */
index 757c2f6d8d4b8128eec011f7f6ba20dcc91f2ac7..d1978e0040548fbacd2f63bb7aca1bbf08068d66 100644 (file)
@@ -318,13 +318,6 @@ struct thread_struct {
 struct mm_struct;
 struct task_struct;
 
-/*
- * Free all resources held by a thread. This is called after the
- * parent of DEAD_TASK has collected the exit status of the task via
- * wait().
- */
-#define release_thread(dead_task)
-
 /* Get wait channel for task P.  */
 extern unsigned long __get_wchan (struct task_struct *p);
 
index 1c4b4308378d43424aa31d4df2b7c9cd239747f5..6954dc5d24e9df599b184f63487cd8a1b05a250f 100644 (file)
@@ -176,9 +176,6 @@ struct thread_struct {
 
 struct task_struct;
 
-/* Free all resources held by a thread. */
-#define release_thread(thread) do { } while (0)
-
 enum idle_boot_override {IDLE_NO_OVERRIDE = 0, IDLE_HALT, IDLE_NOMWAIT, IDLE_POLL};
 
 extern unsigned long           boot_option_idle_override;
index d86b4009880b4e790c4abeb4cda8fbc38be7e238..7a2da780830b8f628a8cbaa25b5333cbc36a8e50 100644 (file)
@@ -145,11 +145,6 @@ static inline void start_thread(struct pt_regs * regs, unsigned long pc,
 /* Forward declaration, a strange C thing */
 struct task_struct;
 
-/* Free all resources held by a thread. */
-static inline void release_thread(struct task_struct *dead_task)
-{
-}
-
 unsigned long __get_wchan(struct task_struct *p);
 void show_registers(struct pt_regs *regs);
 
index 7e9e92670df3340f2acd27f8563f3f6493a5de39..4e193c7550dfa21cc7fb5cb8b55c22a0b0f00030 100644 (file)
@@ -63,11 +63,6 @@ struct thread_struct {
        .pgdir = swapper_pg_dir, \
 }
 
-/* Free all resources held by a thread. */
-static inline void release_thread(struct task_struct *dead_task)
-{
-}
-
 unsigned long __get_wchan(struct task_struct *p);
 
 /* The size allocated for kernel stacks. This _must_ be a power of two! */
index 4bb24579d12e43ff119c48e91b0023cafc140bf9..3fde1ff72bd167e8ce322a2a73dd5612c9b5423b 100644 (file)
@@ -344,9 +344,6 @@ struct thread_struct {
 
 struct task_struct;
 
-/* Free all resources held by a thread. */
-#define release_thread(thread) do { } while(0)
-
 /*
  * Do necessary setup to start up a newly executed thread.
  */
index b8125dfbcad2d6cbe42239aeb3b2afa77bf6a79b..8916d93d5c2d0ba37dae84bf47979c3a21131838 100644 (file)
@@ -64,11 +64,6 @@ extern void start_thread(struct pt_regs *regs, unsigned long pc,
 
 struct task_struct;
 
-/* Free all resources held by a thread. */
-static inline void release_thread(struct task_struct *dead_task)
-{
-}
-
 extern unsigned long __get_wchan(struct task_struct *p);
 
 #define task_pt_regs(p) \
index aa1699c18add85ee7ec81918ba0f4b0129b2ce1a..ed9efb430afa17c9b655ef84789ebd7ee7b47aa2 100644 (file)
@@ -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()
index 52dc983ddeba30c6e978aa683e256be746da7139..f94b5ec06786e9ed10a5b3b80c61d0696558536f 100644 (file)
@@ -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
index 4621ceb51314702a066b241bc6c8a36ec6e5b261..a608970b249af7fbbf3cef99a22fe9549f6c0c22 100644 (file)
@@ -266,9 +266,6 @@ on downward growing arches, it looks like this:
 
 struct mm_struct;
 
-/* Free all resources held by a thread. */
-extern void release_thread(struct task_struct *);
-
 extern unsigned long __get_wchan(struct task_struct *p);
 
 #define KSTK_EIP(tsk)  ((tsk)->thread.regs.iaoq[0])
index 7c37e09c92da6fe0e6e5cbd3ced4604d55a1024d..3db0e97e6c066d0ef6bf7c0c568556bf7965b4e6 100644 (file)
@@ -146,10 +146,6 @@ void flush_thread(void)
        */
 }
 
-void release_thread(struct task_struct *dead_task)
-{
-}
-
 /*
  * Idle thread support
  *
index fdfaae194ddd5c17a978425db9501916ac02f3b8..92e332415d02c2a94b0cb82afc7b2bf401204025 100644 (file)
@@ -75,7 +75,6 @@ extern int _chrp_type;
 
 struct task_struct;
 void start_thread(struct pt_regs *regs, unsigned long fdptr, unsigned long sp);
-void release_thread(struct task_struct *);
 
 #define TS_FPR(i) fp_state.fpr[i][TS_FPROFFSET]
 #define TS_CKFPR(i) ckfp_state.fpr[i][TS_FPROFFSET]
index 0fbda89cd1bb53b43024953469d9c39490dfeb5b..991cda25b9a9e074d02d9891f76dc44ecdc60772 100644 (file)
@@ -1655,11 +1655,6 @@ EXPORT_SYMBOL_GPL(set_thread_tidr);
 
 #endif /* CONFIG_PPC64 */
 
-void
-release_thread(struct task_struct *t)
-{
-}
-
 /*
  * this gets called so that we can store coprocessor state into memory and
  * copy the current task into the new thread.
index 19eedd4af4cdee092b54ef60e5c4d7270bba44a0..94a0590c69710b54e2e1f7258bd6692858ce5f90 100644 (file)
@@ -65,11 +65,6 @@ static inline void arch_thread_struct_whitelist(unsigned long *offset,
 extern void start_thread(struct pt_regs *regs,
                        unsigned long pc, unsigned long sp);
 
-/* Free all resources held by a thread. */
-static inline void release_thread(struct task_struct *dead_task)
-{
-}
-
 extern unsigned long __get_wchan(struct task_struct *p);
 
 
index bd66f8e349492f2e0b70f51a43a40732151fd703..c52fe651eebab698458ffd3979734f8afb7d769d 100644 (file)
@@ -186,9 +186,6 @@ struct pt_regs;
 void show_registers(struct pt_regs *regs);
 void show_cacheinfo(struct seq_file *m);
 
-/* Free all resources held by a thread. */
-static inline void release_thread(struct task_struct *tsk) { }
-
 /* Free guarded storage control block */
 void guarded_storage_release(struct task_struct *tsk);
 void gs_load_bc_cb(struct pt_regs *regs);
index 45240ec6b85a43de127afb13b786c1142fcdb178..27aebf1e75a2003003cffaf82c43d46b3e206357 100644 (file)
@@ -127,9 +127,6 @@ struct task_struct;
 
 extern void start_thread(struct pt_regs *regs, unsigned long new_pc, unsigned long new_sp);
 
-/* Free all resources held by a thread. */
-extern void release_thread(struct task_struct *);
-
 /*
  * FPU lazy state save handling.
  */
index a808843375e713eb5e7c037079e3cb1d2a503b69..92b6649d492952ce6943ed2c535337cfd27508ca 100644 (file)
@@ -84,11 +84,6 @@ void flush_thread(void)
 #endif
 }
 
-void release_thread(struct task_struct *dead_task)
-{
-       /* do nothing */
-}
-
 asmlinkage void ret_from_fork(void);
 asmlinkage void ret_from_kernel_thread(void);
 
index b26c35336b51d180a9485297dd785c49d773a0cd..ba8b70ffec085feb17de9050f37de98e0039f7c3 100644 (file)
@@ -80,9 +80,6 @@ static inline void start_thread(struct pt_regs * regs, unsigned long pc,
                             : "memory");
 }
 
-/* Free all resources held by a thread. */
-#define release_thread(tsk)            do { } while(0)
-
 unsigned long __get_wchan(struct task_struct *);
 
 #define task_pt_regs(tsk) ((tsk)->thread.kregs)
index 89850dff6b033da352da7c39b58a066e4bca3df6..2667f35d5ea5678c64cc99ded20af2b797754967 100644 (file)
@@ -176,9 +176,6 @@ do { \
        regs->tstate &= ~TSTATE_PEF;    \
 } while (0)
 
-/* Free all resources held by a thread. */
-#define release_thread(tsk)            do { } while (0)
-
 unsigned long __get_wchan(struct task_struct *task);
 
 #define task_pt_regs(tsk) (task_thread_info(tsk)->kregs)
index d0fc1862da957dea0c5c0655e441c6a2885e05d8..bb5f06480da95237ff37a5b58394522f40d031cf 100644 (file)
@@ -55,10 +55,6 @@ struct thread_struct {
        .request                = { 0 } \
 }
 
-static inline void release_thread(struct task_struct *task)
-{
-}
-
 /*
  * User space process size: 3GB (default).
  */
index 356308c73951400d44e5b285de5ce0181d988ed0..67c9d73b31faa1590c0aa4abd72da7d5b20ff0ed 100644 (file)
@@ -587,9 +587,6 @@ static inline void load_sp0(unsigned long sp0)
 
 #endif /* CONFIG_PARAVIRT_XXL */
 
-/* Free all resources held by a thread. */
-extern void release_thread(struct task_struct *);
-
 unsigned long __get_wchan(struct task_struct *p);
 
 /*
index 76bc63127c66ef02403d4b7d5c67bd28a0d10456..5abde43c570cd358372a8da8fc6cee8032a10863 100644 (file)
@@ -221,9 +221,6 @@ struct thread_struct {
 struct task_struct;
 struct mm_struct;
 
-/* Free all resources held by a thread. */
-#define release_thread(thread) do { } while(0)
-
 extern unsigned long __get_wchan(struct task_struct *p);
 
 #define KSTK_EIP(tsk)          (task_pt_regs(tsk)->pc)
index 81cab4b01edcb1bb9f40edf2e88de83b207e40f9..d6c48163c6defba399fc5e0588d1ce855418c97e 100644 (file)
@@ -127,6 +127,9 @@ static inline void put_task_struct_many(struct task_struct *t, int nr)
 
 void put_task_struct_rcu_user(struct task_struct *task);
 
+/* Free all architecture-specific resources held by a thread. */
+void release_thread(struct task_struct *dead_task);
+
 #ifdef CONFIG_ARCH_WANTS_DYNAMIC_TASK_STRUCT
 extern int arch_task_struct_size __read_mostly;
 #else
index 84021b24f79e3d1d2dc11fe70bf69cf620c7fc85..f4b7b058f4e6922dadf5fde57515270d564bc93f 100644 (file)
@@ -183,6 +183,10 @@ void put_task_struct_rcu_user(struct task_struct *task)
                call_rcu(&task->rcu, delayed_put_task_struct);
 }
 
+void __weak release_thread(struct task_struct *dead_task)
+{
+}
+
 void release_task(struct task_struct *p)
 {
        struct task_struct *leader;