Currently proc-self-map-files-002.c sets va_max (max test address
of user virtual address) to 4GB, but it is too big for 32bit
arch and 1UL << 32 is overflow on 32bit long.
Also since this value should be enough bigger than vm.mmap_min_addr
(64KB or 32KB by default), 1MB should be enough.

Make va_max 1MB unconditionally.

Signed-off-by: Masami Hiramatsu <mhira...@kernel.org>
Cc: Alexey Dobriyan <adobri...@gmail.com>
---
 Changes in v3:
  - Make the va_max 1MB unconditionally, according to Alexey's comment.
 Changes in v2:
  - Make the va_max 1GB according to Alexey's comment.
---
 .../selftests/proc/proc-self-map-files-002.c       |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/tools/testing/selftests/proc/proc-self-map-files-002.c 
b/tools/testing/selftests/proc/proc-self-map-files-002.c
index 47b7473dedef..e6aa00a183bc 100644
--- a/tools/testing/selftests/proc/proc-self-map-files-002.c
+++ b/tools/testing/selftests/proc/proc-self-map-files-002.c
@@ -47,7 +47,11 @@ static void fail(const char *fmt, unsigned long a, unsigned 
long b)
 int main(void)
 {
        const int PAGE_SIZE = sysconf(_SC_PAGESIZE);
-       const unsigned long va_max = 1UL << 32;
+       /*
+        * va_max must be enough bigger than vm.mmap_min_addr, which is
+        * 64KB/32KB by default. (depends on CONFIG_LSM_MMAP_MIN_ADDR)
+        */
+       const unsigned long va_max = 1UL << 20;
        unsigned long va;
        void *p;
        int fd;

Reply via email to