On 08/07/2021 21.09, Alex Bennée wrote:
The check_aligned_anonymous_unfixed_mmaps and
check_aligned_anonymous_unfixed_colliding_mmaps do a lot of mmap's and
copying of data. This is especially unfriendly to targets like hexagon
which have quite large pages and need to do sanity checks on each
memory access.
Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
---
tests/tcg/multiarch/test-mmap.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/tcg/multiarch/test-mmap.c b/tests/tcg/multiarch/test-mmap.c
index 11d0e777b1..b77deee37e 100644
--- a/tests/tcg/multiarch/test-mmap.c
+++ b/tests/tcg/multiarch/test-mmap.c
@@ -58,12 +58,12 @@ void check_aligned_anonymous_unfixed_mmaps(void)
int i;
fprintf(stdout, "%s", __func__);
- for (i = 0; i < 0x1fff; i++)
+ for (i = 0; i < 0x1ff; i++)
{
While you're at it, you could also fix the coding style here and put the
curly bracket on the right hand side of the for-statement.
size_t len;
len = pagesize + (pagesize * i & 7);
- p1 = mmap(NULL, len, PROT_READ,
+ p1 = mmap(NULL, len, PROT_READ,
MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
p2 = mmap(NULL, len, PROT_READ,
MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
@@ -142,7 +142,7 @@ void check_aligned_anonymous_unfixed_colliding_mmaps(void)
int i;
fprintf(stdout, "%s", __func__);
- for (i = 0; i < 0x2fff; i++)
+ for (i = 0; i < 0x2ff; i++)
dito
{
int nlen;
p1 = mmap(NULL, pagesize, PROT_READ,
Thomas
