diff options
author | Alexander Gordeev <agordeev@linux.ibm.com> | 2023-04-03 08:44:37 +0200 |
---|---|---|
committer | Vasily Gorbik <gor@linux.ibm.com> | 2023-04-19 17:24:15 +0200 |
commit | 86295cb4530dfbc2315ca5997502dad03748e5dd (patch) | |
tree | 7e320c9751874ef6f2960b5f3bb2c649f850a0a0 | |
parent | 7a04d491edf4766e7724671355b1ab27cae81a4a (diff) | |
download | linux-86295cb4530dfbc2315ca5997502dad03748e5dd.tar.gz linux-86295cb4530dfbc2315ca5997502dad03748e5dd.tar.bz2 linux-86295cb4530dfbc2315ca5997502dad03748e5dd.zip |
s390/kdump: cleanup do_start_kdump() prototype and usage
Avoid unnecessary run-time and compile-time type
conversions of do_start_kdump() function return
value and parameter.
Signed-off-by: Alexander Gordeev <agordeev@linux.ibm.com>
Reviewed-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
-rw-r--r-- | arch/s390/kernel/machine_kexec.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/arch/s390/kernel/machine_kexec.c b/arch/s390/kernel/machine_kexec.c index eb473fc835b2..3e0fd01a192a 100644 --- a/arch/s390/kernel/machine_kexec.c +++ b/arch/s390/kernel/machine_kexec.c @@ -111,9 +111,8 @@ static noinline void __machine_kdump(void *image) store_status(__do_machine_kdump, image); } -static unsigned long do_start_kdump(unsigned long addr) +static int do_start_kdump(struct kimage *image) { - struct kimage *image = (struct kimage *) addr; int (*start_kdump)(int) = (void *)image->start; int rc; @@ -134,8 +133,8 @@ static bool kdump_csum_valid(struct kimage *image) int rc; preempt_disable(); - rc = call_on_stack(1, S390_lowcore.nodat_stack, unsigned long, do_start_kdump, - unsigned long, (unsigned long)image); + rc = call_on_stack(1, S390_lowcore.nodat_stack, int, do_start_kdump, + struct kimage *, image); preempt_enable(); return rc == 0; #else |