Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1422669AbXAESst (ORCPT ); Fri, 5 Jan 2007 13:48:49 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1422666AbXAESsT (ORCPT ); Fri, 5 Jan 2007 13:48:19 -0500 Received: from saraswathi.solana.com ([198.99.130.12]:34876 "EHLO saraswathi.solana.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1422669AbXAESrl (ORCPT ); Fri, 5 Jan 2007 13:47:41 -0500 Message-Id: <200701051842.l05IgEUd004622@ccure.user-mode-linux.org> X-Mailer: exmh version 2.7.2 01/07/2005 with nmh-1.0.4 To: akpm@osdl.org cc: linux-kernel@vger.kernel.org, user-mode-linux-devel@lists.sourceforge.net Subject: [PATCH 4/9] UML - Mostly const a structure Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Fri, 05 Jan 2007 13:42:14 -0500 From: Jeff Dike Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4739 Lines: 120 The chan_opts structure is mostly const, and needs no locking. Comment the lack of locking on the one field that can change. Make all the other fields const. It turned out that console_open_chan didn't use its chan_opts argument, so that is deleted from the function and its callers. Signed-off-by: Jeff Dike -- arch/um/drivers/chan_kern.c | 5 ++--- arch/um/drivers/ssl.c | 3 ++- arch/um/drivers/stdio_console.c | 3 ++- arch/um/include/chan_kern.h | 3 +-- arch/um/include/chan_user.h | 8 ++++---- 5 files changed, 11 insertions(+), 11 deletions(-) Index: linux-2.6.18-mm/arch/um/drivers/chan_kern.c =================================================================== --- linux-2.6.18-mm.orig/arch/um/drivers/chan_kern.c 2006-12-29 18:21:46.000000000 -0500 +++ linux-2.6.18-mm/arch/um/drivers/chan_kern.c 2007-01-01 13:20:04.000000000 -0500 @@ -354,8 +354,7 @@ int console_write_chan(struct list_head return ret; } -int console_open_chan(struct line *line, struct console *co, - const struct chan_opts *opts) +int console_open_chan(struct line *line, struct console *co) { int err; @@ -363,7 +362,7 @@ int console_open_chan(struct line *line, if(err) return err; - printk("Console initialized on /dev/%s%d\n",co->name,co->index); + printk("Console initialized on /dev/%s%d\n", co->name, co->index); return 0; } Index: linux-2.6.18-mm/arch/um/drivers/ssl.c =================================================================== --- linux-2.6.18-mm.orig/arch/um/drivers/ssl.c 2007-01-01 12:18:42.000000000 -0500 +++ linux-2.6.18-mm/arch/um/drivers/ssl.c 2007-01-01 13:20:25.000000000 -0500 @@ -38,6 +38,7 @@ static void ssl_announce(char *dev_name, dev_name); } +/* Almost const, except that xterm_title may be changed in an initcall */ static struct chan_opts opts = { .announce = ssl_announce, .xterm_title = "Serial Line #%d", @@ -171,7 +172,7 @@ static int ssl_console_setup(struct cons { struct line *line = &serial_lines[co->index]; - return console_open_chan(line, co, &opts); + return console_open_chan(line, co); } static struct console ssl_cons = { Index: linux-2.6.18-mm/arch/um/include/chan_kern.h =================================================================== --- linux-2.6.18-mm.orig/arch/um/include/chan_kern.h 2006-12-29 17:26:54.000000000 -0500 +++ linux-2.6.18-mm/arch/um/include/chan_kern.h 2007-01-01 13:20:20.000000000 -0500 @@ -36,8 +36,7 @@ extern int write_chan(struct list_head * int write_irq); extern int console_write_chan(struct list_head *chans, const char *buf, int len); -extern int console_open_chan(struct line *line, struct console *co, - const struct chan_opts *opts); +extern int console_open_chan(struct line *line, struct console *co); extern void deactivate_chan(struct list_head *chans, int irq); extern void reactivate_chan(struct list_head *chans, int irq); extern void chan_enable_winch(struct list_head *chans, struct tty_struct *tty); Index: linux-2.6.18-mm/arch/um/drivers/stdio_console.c =================================================================== --- linux-2.6.18-mm.orig/arch/um/drivers/stdio_console.c 2007-01-01 13:21:28.000000000 -0500 +++ linux-2.6.18-mm/arch/um/drivers/stdio_console.c 2007-01-01 13:21:48.000000000 -0500 @@ -42,6 +42,7 @@ void stdio_announce(char *dev_name, int dev_name); } +/* Almost const, except that xterm_title may be changed in an initcall */ static struct chan_opts opts = { .announce = stdio_announce, .xterm_title = "Virtual Console #%d", @@ -145,7 +146,7 @@ static int uml_console_setup(struct cons { struct line *line = &vts[co->index]; - return console_open_chan(line, co, &opts); + return console_open_chan(line, co); } static struct console stdiocons = { Index: linux-2.6.18-mm/arch/um/include/chan_user.h =================================================================== --- linux-2.6.18-mm.orig/arch/um/include/chan_user.h 2007-01-01 11:32:21.000000000 -0500 +++ linux-2.6.18-mm/arch/um/include/chan_user.h 2007-01-01 13:24:50.000000000 -0500 @@ -9,11 +9,11 @@ #include "init.h" struct chan_opts { - void (*announce)(char *dev_name, int dev); + void (*const announce)(char *dev_name, int dev); char *xterm_title; - int raw; - unsigned long tramp_stack; - int in_kernel; + const int raw; + const unsigned long tramp_stack; + const int in_kernel; }; enum chan_init_pri { INIT_STATIC, INIT_ALL, INIT_ONE }; - 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/