]> xenbits.xensource.com Git - people/andrewcoop/xen.git/commit
xen/arm: Remove most of the *_VIRT_END defines
authorJulien Grall <jgrall@amazon.com>
Fri, 29 Jul 2022 21:41:43 +0000 (22:41 +0100)
committerJulien Grall <julien@xen.org>
Fri, 29 Jul 2022 22:03:22 +0000 (23:03 +0100)
commit6dc9a1fe982f52c709ed03df8fdc6d58c4d96826
tree6498c0011ca19aa1e6ace39ad2fc194d93a381f5
parent062790aca6b1faea62c9ed2737c3791efb0d0f4c
xen/arm: Remove most of the *_VIRT_END defines

At the moment, *_VIRT_END may either point to the address after the end
or the last address of the region.

The lack of consistency make quite difficult to reason with them.

Furthermore, there is a risk of overflow in the case where the address
points past to the end. I am not aware of any cases, so this is only a
latent bug.

Start to solve the problem by removing all the *_VIRT_END exclusively used
by the Arm code and add *_VIRT_SIZE when it is not present.

Take the opportunity to rename BOOT_FDT_SLOT_SIZE to BOOT_FDT_VIRT_SIZE
for better consistency and use _AT(vaddr_t, ).

Also take the opportunity to fix the coding style of the comment touched
in mm.c.

Signed-off-by: Julien Grall <jgrall@amazon.com>
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
Reviewed-by: Luca Fancellu <luca.fancellu@arm.com>
Tested-By: Luca Fancellu <luca.fancellu@arm.com>
xen/arch/arm/include/asm/config.h
xen/arch/arm/livepatch.c
xen/arch/arm/mm.c