lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 13 Jun 2019 10:21:09 +0800 From: Guo Ren <guoren@...nel.org> To: Tobias Klauser <tklauser@...tanz.ch> Cc: linux-kernel@...r.kernel.org Subject: Re: [PATCH] csky: remove unsued thread_saved_pc and *_segments functions/macros Acked-by: Guo Ren <ren_guo@...ky.com> thread_saved_pc should be used in unwind stack and I'll give another patch to optimize the unwind flow for csky. On Wed, Jun 12, 2019 at 7:22 PM Tobias Klauser <tklauser@...tanz.ch> wrote: > > These are used nowhere in the tree (except for some architectures which > define them for their own use) and were already removed in: > > commit 6474924e2b5d ("arch: remove unused macro/function thread_saved_pc()") > commit c17c02040bf0 ("arch: remove unused *_segments() macros/functions") > > Remove them from arch/csky as well. > > Signed-off-by: Tobias Klauser <tklauser@...tanz.ch> > --- > arch/csky/include/asm/processor.h | 6 ------ > arch/csky/kernel/process.c | 10 ---------- > 2 files changed, 16 deletions(-) > > diff --git a/arch/csky/include/asm/processor.h b/arch/csky/include/asm/processor.h > index 21e0bd5293dd..464575156f0f 100644 > --- a/arch/csky/include/asm/processor.h > +++ b/arch/csky/include/asm/processor.h > @@ -83,12 +83,6 @@ static inline void release_thread(struct task_struct *dead_task) > > extern int kernel_thread(int (*fn)(void *), void *arg, unsigned long flags); > > -#define copy_segments(tsk, mm) do { } while (0) > -#define release_segments(mm) do { } while (0) > -#define forget_segments() do { } while (0) > - > -extern unsigned long thread_saved_pc(struct task_struct *tsk); > - > unsigned long get_wchan(struct task_struct *p); > > #define KSTK_EIP(tsk) (task_pt_regs(tsk)->pc) > diff --git a/arch/csky/kernel/process.c b/arch/csky/kernel/process.c > index e555740c0be5..adeb6b1bdb42 100644 > --- a/arch/csky/kernel/process.c > +++ b/arch/csky/kernel/process.c > @@ -24,16 +24,6 @@ asmlinkage void ret_from_kernel_thread(void); > */ > void flush_thread(void){} > > -/* > - * Return saved PC from a blocked thread > - */ > -unsigned long thread_saved_pc(struct task_struct *tsk) > -{ > - struct switch_stack *sw = (struct switch_stack *)tsk->thread.ksp; > - > - return sw->r15; > -} > - > int copy_thread(unsigned long clone_flags, > unsigned long usp, > unsigned long kthread_arg, > -- > 2.20.0 >
Powered by blists - more mailing lists