Hi Pingfan,

On Tue, 18 Jan 2022 15:48:09 +0800
Pingfan Liu <pi...@redhat.com> wrote:

> After kernel commit 7bc1a0f9e176 ("arm64: mm: use single quantity to
> represent the PA to VA translation"), phys_offset can be negative if
> running 52-bits kernel on 48-bits hardware.
> 
> So changing phys_offset from unsigned to signed.
> 
> Signed-off-by: Pingfan Liu <pi...@redhat.com>
> Cc: Kairui Song <kas...@tencent.com>
> Cc: Simon Horman <ho...@verge.net.au>
> Cc: Philipp Rudo <pr...@redhat.com>
> To: kexec@lists.infradead.org
> ---
>  kexec/arch/arm64/kexec-arm64.c | 12 ++++++------
>  kexec/arch/arm64/kexec-arm64.h |  2 +-
>  util_lib/elf_info.c            |  2 +-
>  util_lib/include/elf_info.h    |  2 +-
>  4 files changed, 9 insertions(+), 9 deletions(-)
> 

[...]

> diff --git a/kexec/arch/arm64/kexec-arm64.h b/kexec/arch/arm64/kexec-arm64.h
> index ed447ac..1844b67 100644
> --- a/kexec/arch/arm64/kexec-arm64.h
> +++ b/kexec/arch/arm64/kexec-arm64.h
> @@ -58,7 +58,7 @@ extern off_t initrd_size;
>   */
>  
>  struct arm64_mem {
> -     uint64_t phys_offset;
> +     long phys_offset;

I think this one should be int64_t as well.

Other than that
Reviewed-by: Philipp Rudo <pr...@redhat.com>

>       uint64_t text_offset;
>       uint64_t image_size;
>       uint64_t vp_offset;


_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

Reply via email to