Hi Pingfan,
On Tue, 18 Jan 2022 15:48:09 +0800
Pingfan Liu <[email protected]> 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 <[email protected]>
> Cc: Kairui Song <[email protected]>
> Cc: Simon Horman <[email protected]>
> Cc: Philipp Rudo <[email protected]>
> To: [email protected]
> ---
> 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 <[email protected]>
> uint64_t text_offset;
> uint64_t image_size;
> uint64_t vp_offset;
_______________________________________________
kexec mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/kexec
