From 6be10a232d46512e58304cf75497abbbbe68eb75 Mon Sep 17 00:00:00 2001 From: "fu.lin" Date: Sun, 16 Jan 2022 18:37:09 +0800 Subject: [PATCH] zdtm: fix zdtm/static/maps00 case in arm64 This case sometimes will cause SIGILL signal in arm64 platform. <> notes: The ARM architecture does not require the hardware to ensure coherency between instruction caches and memory, even for locations of shared memory. Therefore, we need flush dcache and icache for self-modifying code. - https://developer.arm.com/documentation/den0024/a/Caches/Point-of-coherency-and-unification Signed-off-by: fu.lin --- test/zdtm/static/maps00.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/test/zdtm/static/maps00.c b/test/zdtm/static/maps00.c index 10a4cac79..b1e55e861 100644 --- a/test/zdtm/static/maps00.c +++ b/test/zdtm/static/maps00.c @@ -158,7 +158,13 @@ static int check_map(struct map *map) if (!sigsetjmp(segv_ret, 1)) { if (map->prot & PROT_WRITE) { - memcpy(map->ptr, test_func, getpagesize()); + memcpy(map->ptr, test_func, ONE_MAP_SIZE); + /* The ARM ARM architecture does not require the + * hardware to ensure coherency between instruction + * caches and memory, flushing dcache and icache is + * necessory to prevent SIGILL signal. + */ + __builtin___clear_cache(map->ptr, map->ptr + ONE_MAP_SIZE); } else { if (!(map->flag & MAP_ANONYMOUS)) { uint8_t funlen = (uint8_t *)check_map - (uint8_t *)test_func;