dynamic_debug: simplify dynamic_debug_init error exit
authorJim Cromie <jim.cromie@gmail.com>
Fri, 27 Apr 2012 20:30:38 +0000 (14:30 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 30 Apr 2012 20:25:39 +0000 (16:25 -0400)
We dont want errors while parsing ddebug_query to unload ddebug
tables, so set success after tables are loaded, and return 0 after
query parsing is done.

Simplify error handling code since its no longer used for success,
and change goto label to out_err to clarify this.

Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
Acked-by: Jason Baron <jbaron@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
lib/dynamic_debug.c

index 3b06f926d5b8e7417c77547cba89ff737d3e9ba3..66e0ec4d21f7ad0367a5d7f40e86d299a5901fd3 100644 (file)
@@ -984,7 +984,7 @@ static int __init dynamic_debug_init(void)
                if (strcmp(modname, iter->modname)) {
                        ret = ddebug_add_module(iter_start, n, modname);
                        if (ret)
-                               goto out_free;
+                               goto out_err;
                        n = 0;
                        modname = iter->modname;
                        iter_start = iter;
@@ -993,9 +993,11 @@ static int __init dynamic_debug_init(void)
        }
        ret = ddebug_add_module(iter_start, n, modname);
        if (ret)
-               goto out_free;
+               goto out_err;
 
-       /* ddebug_query boot param got passed -> set it up */
+       ddebug_init_success = 1;
+
+       /* apply ddebug_query boot param, dont unload tables on err */
        if (ddebug_setup_string[0] != '\0') {
                pr_warn("ddebug_query param name is deprecated,"
                        " change it to dyndbg\n");
@@ -1005,9 +1007,6 @@ static int __init dynamic_debug_init(void)
                                ddebug_setup_string);
                else
                        pr_info("%d changes by ddebug_query\n", ret);
-
-               /* keep tables even on ddebug_query parse error */
-               ret = 0;
        }
        /* now that ddebug tables are loaded, process all boot args
         * again to find and activate queries given in dyndbg params.
@@ -1021,12 +1020,10 @@ static int __init dynamic_debug_init(void)
        parse_args("dyndbg params", cmdline, NULL,
                   0, 0, 0, &ddebug_dyndbg_boot_param_cb);
        kfree(cmdline);
+       return 0;
 
-out_free:
-       if (ret)
-               ddebug_remove_all_tables();
-       else
-               ddebug_init_success = 1;
+out_err:
+       ddebug_remove_all_tables();
        return 0;
 }
 /* Allow early initialization for boot messages via boot param */