diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2012-06-08 01:28:13 -0400 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2012-06-08 01:28:13 -0400 |
commit | ae82fdb1406ad41d68f07027fe31f2d35ba22a90 (patch) | |
tree | dced5ed8a4afeeab7e3c3610d0272f2f18106f33 /init/main.c | |
parent | f8f5701bdaf9134b1f90e5044a82c66324d2073f (diff) |
module_param: stop double-calling parameters.
Commit 026cee0086fe1df4cf74691cf273062cc769617d "params:
<level>_initcall-like kernel parameters" set old-style module
parameters to level 0. And we call those level 0 calls where we used
to, early in start_kernel().
We also loop through the initcall levels and call the levelled
module_params before the corresponding initcall. Unfortunately level
0 is early_init(), so we call the standard module_param calls twice.
(Turns out most things don't care, but at least ubi.mtd does).
Change the level to -1 for standard module_param calls.
Reported-by: Benoît Thébaudeau <benoit.thebaudeau@advansee.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Cc: stable@kernel.org
Diffstat (limited to 'init/main.c')
-rw-r--r-- | init/main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/init/main.c b/init/main.c index 1ca6b32c4828..37e12098eac1 100644 --- a/init/main.c +++ b/init/main.c | |||
@@ -508,7 +508,7 @@ asmlinkage void __init start_kernel(void) | |||
508 | parse_early_param(); | 508 | parse_early_param(); |
509 | parse_args("Booting kernel", static_command_line, __start___param, | 509 | parse_args("Booting kernel", static_command_line, __start___param, |
510 | __stop___param - __start___param, | 510 | __stop___param - __start___param, |
511 | 0, 0, &unknown_bootoption); | 511 | -1, -1, &unknown_bootoption); |
512 | 512 | ||
513 | jump_label_init(); | 513 | jump_label_init(); |
514 | 514 | ||