Messages in this thread Patch in this message | | | Date | Thu, 22 Nov 2018 23:03:57 -0800 | From | tip-bot for Yafang Shao <> | Subject | [tip:x86/cleanups] x86/process: Avoid unnecessary NULL check in get_wchan() |
| |
Commit-ID: 6e662ae7bce6db602f79e57791f5fb887fb7d371 Gitweb: https://git.kernel.org/tip/6e662ae7bce6db602f79e57791f5fb887fb7d371 Author: Yafang Shao <laoar.shao@gmail.com> AuthorDate: Wed, 21 Nov 2018 19:12:14 +0800 Committer: Ingo Molnar <mingo@kernel.org> CommitDate: Fri, 23 Nov 2018 07:58:23 +0100
x86/process: Avoid unnecessary NULL check in get_wchan()
Task 'p' is always guaranteed to be non-NULL, because the only call sites are in fs/proc/ which all guarantee a non-NULL task pointer.
[ mingo: Improved the changelog. ] Signed-off-by: Yafang Shao <laoar.shao@gmail.com> Cc: Andy Lutomirski <luto@amacapital.net> Cc: Borislav Petkov <bp@alien8.de> Cc: Brian Gerst <brgerst@gmail.com> Cc: Denys Vlasenko <dvlasenk@redhat.com> Cc: H. Peter Anvin <hpa@zytor.com> Cc: Linus Torvalds <torvalds@linux-foundation.org> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Thomas Gleixner <tglx@linutronix.de> Link: http://lkml.kernel.org/r/1542798734-12532-1-git-send-email-laoar.shao@gmail.com Signed-off-by: Ingo Molnar <mingo@kernel.org> --- arch/x86/kernel/process.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/kernel/process.c b/arch/x86/kernel/process.c index c93fcfdf1673..3c3ee8982577 100644 --- a/arch/x86/kernel/process.c +++ b/arch/x86/kernel/process.c @@ -730,7 +730,7 @@ unsigned long get_wchan(struct task_struct *p) unsigned long start, bottom, top, sp, fp, ip, ret = 0; int count = 0; - if (!p || p == current || p->state == TASK_RUNNING) + if (p == current || p->state == TASK_RUNNING) return 0; if (!try_get_task_stack(p))
| |