[PATCH] kexec: Fix callback parameter types

Benjamin Kay benjamin at benkay.net
Tue Jul 29 10:18:11 PDT 2014


Thanks, this was causing kexec to refuse to load a crash kernel on my i686 
system.

On Tuesday, July 29, 2014 15:51:03 Visa Hankala wrote:
> Commit 4362bfaca4d8d2a4380532880f0d08c2bf14184e changed
> kexec_iomem_for_each_line() to pass 'base' and 'length' parameters of
> the callback function as unsigned long long integers instead of unsigned
> longs. Some of the functions were not updated. This patch corrects the
> remaining instances.
> 
> Signed-off-by: Visa Hankala <visa at hankala.org>
> ---
>  kexec/arch/i386/crashdump-x86.c |    4 ++--
>  kexec/arch/sh/crashdump-sh.c    |    4 ++--
>  kexec/arch/sh/kexec-sh.c        |    4 ++--
>  3 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/kexec/arch/i386/crashdump-x86.c
> b/kexec/arch/i386/crashdump-x86.c index 77bdad5..82bf239 100644
> --- a/kexec/arch/i386/crashdump-x86.c
> +++ b/kexec/arch/i386/crashdump-x86.c
> @@ -1041,8 +1041,8 @@ int get_max_crash_kernel_limit(uint64_t *start,
> uint64_t *end)
> 
>  static int crashkernel_mem_callback(void *UNUSED(data), int nr,
>                                            char *UNUSED(str),
> -                                          unsigned long base,
> -                                          unsigned long length)
> +                                          unsigned long long base,
> +                                          unsigned long long length)
>  {
>  	if (nr >= CRASH_RESERVED_MEM_NR)
>  		return 1;
> diff --git a/kexec/arch/sh/crashdump-sh.c b/kexec/arch/sh/crashdump-sh.c
> index fe11b17..9e6af6b 100644
> --- a/kexec/arch/sh/crashdump-sh.c
> +++ b/kexec/arch/sh/crashdump-sh.c
> @@ -34,8 +34,8 @@ static struct memory_range
> crash_memory_range[CRASH_MAX_MEMORY_RANGES]; static int crash_sh_range_nr;
>  static int crash_sh_memory_range_callback(void *UNUSED(data), int
> UNUSED(nr), char *str,
> -					  unsigned long base,
> -					  unsigned long length)
> +					  unsigned long long base,
> +					  unsigned long long length)
>  {
> 
>  	struct memory_range *range = crash_memory_range;
> diff --git a/kexec/arch/sh/kexec-sh.c b/kexec/arch/sh/kexec-sh.c
> index aea5dfb..ce341c8 100644
> --- a/kexec/arch/sh/kexec-sh.c
> +++ b/kexec/arch/sh/kexec-sh.c
> @@ -24,8 +24,8 @@ static struct memory_range
> memory_range[MAX_MEMORY_RANGES];
> 
>  static int kexec_sh_memory_range_callback(void *UNUSED(data), int nr,
>  					  char *UNUSED(str),
> -					  unsigned long base,
> -					  unsigned long length)
> +					  unsigned long long base,
> +					  unsigned long long length)
>  {
>  	if (nr < MAX_MEMORY_RANGES) {
>  		memory_range[nr].start = base;
> 
> _______________________________________________
> kexec mailing list
> kexec at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/kexec




More information about the kexec mailing list