Commit dd0c5d04 authored by Fuad Tabba's avatar Fuad Tabba Committed by Marc Zyngier
Browse files

KVM: arm64: selftests: Fix incorrect rounding in page_align() 



The implementation of `page_align()` in `processor.c` calculates
alignment incorrectly for values that are already aligned. Specifically,
`(v + vm->page_size) & ~(vm->page_size - 1)` aligns to the *next* page
boundary even if `v` is already page-aligned, potentially wasting a page
of memory.

Fix the calculation to use standard alignment logic: `(v + vm->page_size
- 1) & ~(vm->page_size - 1)`.

Fixes: 7a6629ef ("kvm: selftests: add virt mem support for aarch64")
Reviewed-by: default avatarAndrew Jones <andrew.jones@linux.dev>
Signed-off-by: default avatarFuad Tabba <tabba@google.com>
Link: https://patch.msgid.link/20260109082218.3236580-3-tabba@google.com


Signed-off-by: default avatarMarc Zyngier <maz@kernel.org>
parent 7e03d07d

tools/testing/selftests/kvm/lib/arm64/processor.c

+1 −1
Original line number Diff line number Diff line
@@ -23,7 +23,7 @@ static vm_vaddr_t exception_handlers; 


static uint64_t page_align(struct kvm_vm *vm, uint64_t v)

{

	return (v + vm->page_size) & ~(vm->page_size - 1);

	return (v + vm->page_size - 1) & ~(vm->page_size - 1);

}



static uint64_t pgd_index(struct kvm_vm *vm, vm_vaddr_t gva)