Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756023AbZIKTaI (ORCPT ); Fri, 11 Sep 2009 15:30:08 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755689AbZIKTaI (ORCPT ); Fri, 11 Sep 2009 15:30:08 -0400 Received: from smtp.zeugmasystems.com ([70.79.96.174]:8803 "EHLO zeugmasystems.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752179AbZIKTaH convert rfc822-to-8bit (ORCPT ); Fri, 11 Sep 2009 15:30:07 -0400 X-MimeOLE: Produced By Microsoft Exchange V6.5 Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Subject: [PATCH] cleanup legacy console_print function Date: Fri, 11 Sep 2009 12:29:59 -0700 Message-ID: X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH] cleanup legacy console_print function Thread-Index: AconTY4JWUpFDwijSlGlsPdrRWZqxwAENfCwAu3I6RA= References: <20090825172640.af8b9f58.randy.dunlap@oracle.com> From: "Anirban Sinha" To: "Linus Torvalds" , , "Randy Dunlap" Cc: "Anirban Sinha" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4347 Lines: 161 Hi Linus: >>Looks ok to me, but I'm not going to apply it in late -rc, since it >>doesn't really fix any bug. Feel free to re-send after 2.6.31 is out, >or >>try to get it into one of the trees to be merged later. >> > >Thanks Linus. I will resend the patch once 2.6.31 is out of the door. Here's the patch again: Subject: cleanup legacy console_print function console_print() is an old legacy interface mostly unused in the entire kernel tree. It's best to clean up its existing use and let developers usew their own implementation of it as they feel fit. Signed-off-by: Anirban Sinha --- arch/ia64/kernel/head.S | 1 + arch/ia64/kernel/head.h | 1 + arch/ia64/kernel/process.c | 7 +++++++ drivers/char/serial167.c | 5 ++--- include/linux/dtlk.h | 6 +++--- include/linux/tty.h | 4 ---- kernel/printk.c | 6 ------ 7 files changed, 14 insertions(+), 16 deletions(-) diff --git a/arch/ia64/kernel/head.S b/arch/ia64/kernel/head.S index 23f846d..e6c5c3d 100644 --- a/arch/ia64/kernel/head.S +++ b/arch/ia64/kernel/head.S @@ -34,6 +34,7 @@ #include #include #include +#include "head.h" #ifdef CONFIG_HOTPLUG_CPU #define SAL_PSR_BITS_TO_SET \ diff --git a/arch/ia64/kernel/head.h b/arch/ia64/kernel/head.h new file mode 100644 index 0000000..2e2ac68 --- /dev/null +++ b/arch/ia64/kernel/head.h @@ -0,0 +1 @@ +extern void console_print(const char *s); diff --git a/arch/ia64/kernel/process.c b/arch/ia64/kernel/process.c index 5d7c0e5..c2bb591 100644 --- a/arch/ia64/kernel/process.c +++ b/arch/ia64/kernel/process.c @@ -161,6 +161,13 @@ show_regs (struct pt_regs *regs) show_stack(NULL, NULL); } +/* local support for deprecated console_print */ +void +console_print(const char *s) +{ + printk(KERN_EMERG "%s", s); +} + void do_notify_resume_user(sigset_t *unused, struct sigscratch *scr, long in_syscall) { diff --git a/drivers/char/serial167.c b/drivers/char/serial167.c index 51e7a46..5942a9d 100644 --- a/drivers/char/serial167.c +++ b/drivers/char/serial167.c @@ -171,7 +171,6 @@ static int startup(struct cyclades_port *); static void cy_throttle(struct tty_struct *); static void cy_unthrottle(struct tty_struct *); static void config_setup(struct cyclades_port *); -extern void console_print(const char *); #ifdef CYCLOM_SHOW_STATUS static void show_status(int); #endif @@ -245,7 +244,7 @@ void SP(char *data) { unsigned long flags; local_irq_save(flags); - console_print(data); + printk(KERN_EMERG "%s", data); local_irq_restore(flags); } @@ -255,7 +254,7 @@ void CP(char data) unsigned long flags; local_irq_save(flags); scrn[0] = data; - console_print(scrn); + printk(KERN_EMERG "%c", scrn); local_irq_restore(flags); } /* CP */ diff --git a/include/linux/dtlk.h b/include/linux/dtlk.h index 2896d90..4f93c45 100644 --- a/include/linux/dtlk.h +++ b/include/linux/dtlk.h @@ -4,15 +4,15 @@ { \ if(dtlk_trace) \ { \ - console_print(text); \ - console_print("\n"); \ + printk(KERN_EMERG "%s", text); \ + printk(KERN_EMERG "\n"); \ } \ } #define TRACE_CHR(chr) \ { \ if(dtlk_trace) \ - console_print(chr); \ + printk(KERN_EMERG "%c", chr); \ } \ #endif diff --git a/include/linux/tty.h b/include/linux/tty.h index e8c6c91..ba191e3 100644 --- a/include/linux/tty.h +++ b/include/linux/tty.h @@ -517,10 +517,6 @@ extern void serial_console_init(void); extern int pcxe_open(struct tty_struct *tty, struct file *filp); -/* printk.c */ - -extern void console_print(const char *); - /* vt.c */ extern int vt_ioctl(struct tty_struct *tty, struct file *file, diff --git a/kernel/printk.c b/kernel/printk.c index b4d97b5..32e15ce 100644 --- a/kernel/printk.c +++ b/kernel/printk.c @@ -1060,12 +1060,6 @@ void __sched console_conditional_schedule(void) } EXPORT_SYMBOL(console_conditional_schedule); -void console_print(const char *s) -{ - printk(KERN_EMERG "%s", s); -} -EXPORT_SYMBOL(console_print); - void console_unblank(void) { struct console *c; -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/