On 3 Jul 2025, at 2:06, Aboorva Devarajan wrote: > From: Donet Tom <[email protected]> > > In main(), the high address is stored in hptr, but for mark_range(), > the address passed is ptr, not hptr. Fixed this by changing ptr[i] to > hptr[i] in mark_range() function call. > > Fixes: b2a79f62133a ("selftests/mm: virtual_address_range: unmap chunks after > validation") > Signed-off-by: Donet Tom <[email protected]> > Signed-off-by: Aboorva Devarajan <[email protected]> > --- > tools/testing/selftests/mm/virtual_address_range.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/mm/virtual_address_range.c > b/tools/testing/selftests/mm/virtual_address_range.c > index 169dbd692bf5..e24c36a39f22 100644 > --- a/tools/testing/selftests/mm/virtual_address_range.c > +++ b/tools/testing/selftests/mm/virtual_address_range.c > @@ -227,7 +227,7 @@ int main(int argc, char *argv[]) > if (hptr[i] == MAP_FAILED) > break; > > - mark_range(ptr[i], MAP_CHUNK_SIZE); > + mark_range(hptr[i], MAP_CHUNK_SIZE); > validate_addr(hptr[i], 1); > } > hchunks = i;
It looks like it was a copy-paste error. Thank you for fixing it. Reviewed-by: Zi Yan <[email protected]> Best Regards, Yan, Zi
