Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d2b60881e28072109601c373abd1085499ccfef0
Commit:     d2b60881e28072109601c373abd1085499ccfef0
Parent:     d133ccbdc30c7f86459519cec1126d6473762b10
Author:     Satyam Sharma <[EMAIL PROTECTED]>
AuthorDate: Fri Aug 10 15:29:47 2007 -0700
Committer:  David S. Miller <[EMAIL PROTECTED]>
CommitDate: Wed Oct 10 16:48:03 2007 -0700

    [NET] netconsole: Simplify boot/module option setup logic
    
    Based upon initial work by Keiichi Kii <[EMAIL PROTECTED]>.
    
    Presently, boot/module parameters are set up quite differently for the case 
of
    built-in netconsole (__setup() -> obsolete_checksetup() ->
    netpoll_parse_options() -> strlen(config) == 0 in init_netconsole()) vs
    modular netconsole (module_param_string() -> string copied to the config
    variable -> strlen(config) != 0 init_netconsole() -> 
netpoll_parse_options()).
    
    This patch makes both of them similar by doing exactly the equivalent of a
    module_param_string() in option_setup() also -- just copying the param 
string
    passed from the kernel command line into "config" variable.  So,
    strlen(config) != 0 in both cases, and netpoll_parse_options() is always
    called from init_netconsole(), thus making the setup logic for both cases
    similar.
    
    Now, option_setup() is only ever called / used for the built-in case, so we
    put it inside a #ifndef MODULE, otherwise gcc will complain about
    option_setup() being "defined but not used".  Also, the "configured" 
variable
    is redundant with this patch and hence removed.
    
    Signed-off-by: Satyam Sharma <[EMAIL PROTECTED]>
    Signed-off-by: Keiichi Kii <[EMAIL PROTECTED]>
    Acked-by: Matt Mackall <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
---
 drivers/net/netconsole.c |   27 ++++++++++++++-------------
 1 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c
index 2c2aef1..e56aa6c 100644
--- a/drivers/net/netconsole.c
+++ b/drivers/net/netconsole.c
@@ -53,6 +53,15 @@ static char config[MAX_PARAM_LENGTH];
 module_param_string(netconsole, config, MAX_PARAM_LENGTH, 0);
 MODULE_PARM_DESC(netconsole, " [EMAIL 
PROTECTED]/[dev],[tgt-port]@<tgt-ip>/[tgt-macaddr]\n");
 
+#ifndef        MODULE
+static int __init option_setup(char *opt)
+{
+       strlcpy(config, opt, MAX_PARAM_LENGTH);
+       return 1;
+}
+__setup("netconsole=", option_setup);
+#endif /* MODULE */
+
 static struct netpoll np = {
        .name           = "netconsole",
        .dev_name       = "eth0",
@@ -60,7 +69,6 @@ static struct netpoll np = {
        .remote_port    = 6666,
        .remote_mac     = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff},
 };
-static int configured;
 
 static void write_msg(struct console *con, const char *msg, unsigned int len)
 {
@@ -85,26 +93,19 @@ static struct console netconsole = {
        .write  = write_msg,
 };
 
-static int __init option_setup(char *opt)
-{
-       configured = !netpoll_parse_options(&np, opt);
-       return 1;
-}
-
-__setup("netconsole=", option_setup);
-
 static int __init init_netconsole(void)
 {
        int err = 0;
 
-       if (strnlen(config, MAX_PARAM_LENGTH))
-               option_setup(config);
-
-       if (!configured) {
+       if (!strnlen(config, MAX_PARAM_LENGTH)) {
                printk(KERN_INFO "netconsole: not configured, aborting\n");
                goto out;
        }
 
+       err = netpoll_parse_options(&np, config);
+       if (err)
+               goto out;
+
        err = netpoll_setup(&np);
        if (err)
                goto out;
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to