@@ -233,11 +233,12 @@ static int __init __parse_crashkernel(char *cmdline,
unsigned long long system_ram,
unsigned long long *crash_size,
unsigned long long *crash_base,
- const char *name,
- const char *suffix)
+ int suffix_type)
{
char *first_colon, *first_space;
char *ck_cmdline;
+ const char *name = "crashkernel=";
+ const char *suffix = suffix_tbl[suffix_type];
BUG_ON(!crash_size || !crash_base);
*crash_size = 0;
@@ -275,8 +276,7 @@ int __init parse_crashkernel(char *cmdline,
unsigned long long *crash_size,
unsigned long long *crash_base)
{
- return __parse_crashkernel(cmdline, system_ram, crash_size, crash_base,
- "crashkernel=", NULL);
+ return __parse_crashkernel(cmdline, system_ram, crash_size, crash_base, SUFFIX_NULL);
}
int __init parse_crashkernel_high(char *cmdline,
@@ -284,8 +284,7 @@ int __init parse_crashkernel_high(char *cmdline,
unsigned long long *crash_size,
unsigned long long *crash_base)
{
- return __parse_crashkernel(cmdline, system_ram, crash_size, crash_base,
- "crashkernel=", suffix_tbl[SUFFIX_HIGH]);
+ return __parse_crashkernel(cmdline, system_ram, crash_size, crash_base, SUFFIX_HIGH);
}
int __init parse_crashkernel_low(char *cmdline,
@@ -293,8 +292,7 @@ int __init parse_crashkernel_low(char *cmdline,
unsigned long long *crash_size,
unsigned long long *crash_base)
{
- return __parse_crashkernel(cmdline, system_ram, crash_size, crash_base,
- "crashkernel=", suffix_tbl[SUFFIX_LOW]);
+ return __parse_crashkernel(cmdline, system_ram, crash_size, crash_base, SUFFIX_LOW);
}
/*
After commit adbc742bf786 ("x86, kdump: Change crashkernel_high/low= to crashkernel=,high/low"), all kdump bootup parameters start with "crashkernel=". Therefore, it is better for __parse_crashkernel() to use it directly than for the caller to pass it. So the parameter 'name' can be omitted. Similarly, we can pass the suffix type instead of the suffix name to avoid the global variable 'suffix_tbl' appearing in multiple places. There is no change in functionality, but it makes the code look a little more concise. Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> --- kernel/crash_core.c | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-)