| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Fri, 1 Feb 2008 11:56:23 -0800
From: Andrew Morton <akpm@...ux-foundation.org>
To: Yinghai Lu <Yinghai.Lu@....COM>
Cc: mingo@...e.hu, andi@...stfloor.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] make loglevel related commandline to early_param v2
On Fri, 01 Feb 2008 11:35:35 -0800
Yinghai Lu <Yinghai.Lu@....COM> wrote:
> [PATCH] make loglevel related commandline to early_param v2
>
> so we can use them for early console like
> console=uart8250,io,0x3f8,115200n8
> earlycon=uart8250,io,0x3f8,115200n8
> early_printk
>
> otherwise printk(KERN_DEBUG "debug msg") will not print out to console even
> "debug" command line is used.
>
> andi point out early_param need to return 0 instead 1.
>
> Ingo had another patch in x86.git to process ignore_loglevel
>
> Signed-off-by: Yinghai Lu <yinghai.lu@....com>
>
> diff --git a/init/main.c b/init/main.c
> index cb81ed1..3741bda 100644
> --- a/init/main.c
> +++ b/init/main.c
> @@ -232,30 +232,26 @@ EXPORT_SYMBOL(loops_per_jiffy);
>
> static int __init debug_kernel(char *str)
> {
> - if (*str)
> - return 0;
> console_loglevel = 10;
> - return 1;
> + return 0;
> }
>
> static int __init quiet_kernel(char *str)
> {
> - if (*str)
> - return 0;
> console_loglevel = 4;
> - return 1;
> + return 0;
> }
>
> -__setup("debug", debug_kernel);
> -__setup("quiet", quiet_kernel);
> +early_param("debug", debug_kernel);
> +early_param("quiet", quiet_kernel);
>
> static int __init loglevel(char *str)
> {
> get_option(&str, &console_loglevel);
> - return 1;
> + return 0;
> }
>
> -__setup("loglevel=", loglevel);
> +early_param("loglevel", loglevel);
>
> /*
> * Unknown boot options get handed to init, unless they look like
Your earlier patch converted ignore_loglevel but this one no longer does
so. Was that deliberate?
Below is the patch whcih I presently have queued. Is it OK?
init/main.c | 14 +++++---------
kernel/printk.c | 5 ++---
2 files changed, 7 insertions(+), 12 deletions(-)
diff -puN init/main.c~convert-loglevel-related-kernel-boot-parameters-to-early_param init/main.c
--- a/init/main.c~convert-loglevel-related-kernel-boot-parameters-to-early_param
+++ a/init/main.c
@@ -232,22 +232,18 @@ EXPORT_SYMBOL(loops_per_jiffy);
static int __init debug_kernel(char *str)
{
- if (*str)
- return 0;
console_loglevel = 10;
- return 1;
+ return 0;
}
static int __init quiet_kernel(char *str)
{
- if (*str)
- return 0;
console_loglevel = 4;
- return 1;
+ return 0;
}
-__setup("debug", debug_kernel);
-__setup("quiet", quiet_kernel);
+early_param("debug", debug_kernel);
+early_param("quiet", quiet_kernel);
static int __init loglevel(char *str)
{
@@ -255,7 +251,7 @@ static int __init loglevel(char *str)
return 1;
}
-__setup("loglevel=", loglevel);
+early_param("loglevel", loglevel);
/*
* Unknown boot options get handed to init, unless they look like
diff -puN kernel/printk.c~convert-loglevel-related-kernel-boot-parameters-to-early_param kernel/printk.c
--- a/kernel/printk.c~convert-loglevel-related-kernel-boot-parameters-to-early_param
+++ a/kernel/printk.c
@@ -447,11 +447,10 @@ static int __init ignore_loglevel_setup(
{
ignore_loglevel = 1;
printk(KERN_INFO "debug: ignoring loglevel setting.\n");
-
- return 1;
+ return 0;
}
-__setup("ignore_loglevel", ignore_loglevel_setup);
+early_param("ignore_loglevel", ignore_loglevel_setup);
/*
* Write out chars from start to end - 1 inclusive
_
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists