[tip:sched/urgent] sched/core: Remove pointless printout in sched_show_task()

[tip-bot for Linus Torvalds]

Commit-ID:  8243d5597793b5e85143c9a935e1b971c59740a9
Gitweb: http://git.kernel.org/tip/8243d5597793b5e85143c9a935e1b971c59740a9
Author: Linus Torvalds 
AuthorDate: Tue, 1 Nov 2016 17:47:18 -0600
Committer:  Ingo Molnar 
CommitDate: Thu, 3 Nov 2016 07:31:34 +0100

sched/core: Remove pointless printout in sched_show_task()

In sched_show_task() we print out a useless hex number, not even a
symbol, and there's a big question mark whether this even makes sense
anyway, I suspect we should just remove it all.

Signed-off-by: Linus Torvalds 
Acked-by: Andy Lutomirski 
Cc: Peter Zijlstra 
Cc: Tetsuo Handa 
Cc: Thomas Gleixner 
Cc: b...@alien8.de
Cc: brge...@gmail.com
Cc: j...@thejh.net
Cc: keesc...@chromium.org
Cc: linux-...@vger.kernel.org
Cc: tycho.ander...@canonical.com
Link: 
http://lkml.kernel.org/r/ca+55afzphurpfzavu4z6moy7zmimcwpuudyu8bj9z0j+s8x...@mail.gmail.com
Signed-off-by: Ingo Molnar 
---
 kernel/sched/core.c | 9 -
 1 file changed, 9 deletions(-)

diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 9abf66b..154fd68 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -5198,17 +5198,8 @@ void sched_show_task(struct task_struct *p)
state = __ffs(state) + 1;
printk(KERN_INFO "%-15.15s %c", p->comm,
state < sizeof(stat_nam) - 1 ? stat_nam[state] : '?');
-#if BITS_PER_LONG == 32
-   if (state == TASK_RUNNING)
-   printk(KERN_CONT " running  ");
-   else
-   printk(KERN_CONT " %08lx ", thread_saved_pc(p));
-#else
if (state == TASK_RUNNING)
printk(KERN_CONT "  running task");
-   else
-   printk(KERN_CONT " %016lx ", thread_saved_pc(p));
-#endif
 #ifdef CONFIG_DEBUG_STACK_USAGE
free = stack_not_used(p);
 #endif

[tip:sched/urgent] sched/core: Remove pointless printout in sched_show_task()

[tip-bot for Linus Torvalds]

Commit-ID:  8243d5597793b5e85143c9a935e1b971c59740a9
Gitweb: http://git.kernel.org/tip/8243d5597793b5e85143c9a935e1b971c59740a9
Author: Linus Torvalds 
AuthorDate: Tue, 1 Nov 2016 17:47:18 -0600
Committer:  Ingo Molnar 
CommitDate: Thu, 3 Nov 2016 07:31:34 +0100

sched/core: Remove pointless printout in sched_show_task()

In sched_show_task() we print out a useless hex number, not even a
symbol, and there's a big question mark whether this even makes sense
anyway, I suspect we should just remove it all.

Signed-off-by: Linus Torvalds 
Acked-by: Andy Lutomirski 
Cc: Peter Zijlstra 
Cc: Tetsuo Handa 
Cc: Thomas Gleixner 
Cc: b...@alien8.de
Cc: brge...@gmail.com
Cc: j...@thejh.net
Cc: keesc...@chromium.org
Cc: linux-...@vger.kernel.org
Cc: tycho.ander...@canonical.com
Link: 
http://lkml.kernel.org/r/ca+55afzphurpfzavu4z6moy7zmimcwpuudyu8bj9z0j+s8x...@mail.gmail.com
Signed-off-by: Ingo Molnar 
---
 kernel/sched/core.c | 9 -
 1 file changed, 9 deletions(-)

diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 9abf66b..154fd68 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -5198,17 +5198,8 @@ void sched_show_task(struct task_struct *p)
state = __ffs(state) + 1;
printk(KERN_INFO "%-15.15s %c", p->comm,
state < sizeof(stat_nam) - 1 ? stat_nam[state] : '?');
-#if BITS_PER_LONG == 32
-   if (state == TASK_RUNNING)
-   printk(KERN_CONT " running  ");
-   else
-   printk(KERN_CONT " %08lx ", thread_saved_pc(p));
-#else
if (state == TASK_RUNNING)
printk(KERN_CONT "  running task");
-   else
-   printk(KERN_CONT " %016lx ", thread_saved_pc(p));
-#endif
 #ifdef CONFIG_DEBUG_STACK_USAGE
free = stack_not_used(p);
 #endif