2008-07-03 20:12:27

by Hiroshi Shimamoto

Subject: [PATCH] x86_32: remove redundant KERN_INFO

From: Hiroshi Shimamoto <[email protected]>

This printk has a KERN_ facility level in the format string.

Signed-off-by: Hiroshi Shimamoto <[email protected]>
---
arch/x86/kernel/signal_32.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/x86/kernel/signal_32.c b/arch/x86/kernel/signal_32.c
index d923736..d633d80 100644
--- a/arch/x86/kernel/signal_32.c
+++ b/arch/x86/kernel/signal_32.c
@@ -212,7 +212,7 @@ asmlinkage unsigned long sys_sigreturn(unsigned long __unused)

badframe:
if (show_unhandled_signals && printk_ratelimit()) {
- printk(KERN_INFO "%s%s[%d] bad frame in sigreturn frame:"
+ printk("%s%s[%d] bad frame in sigreturn frame:"
"%p ip:%lx sp:%lx oeax:%lx",
task_pid_nr(current) > 1 ? KERN_INFO : KERN_EMERG,
current->comm, task_pid_nr(current), frame, regs->ip,
--
1.5.4.1

2008-07-18 17:01:42

by Ingo Molnar

Subject: Re: [PATCH] x86_32: remove redundant KERN_INFO

* Hiroshi Shimamoto <[email protected]> wrote:

> From: Hiroshi Shimamoto <[email protected]>
>
> This printk has a KERN_ facility level in the format string.

applied to tip/x86/cleanups, thanks.

Ingo