Skip to content

Conversation

@thetruestblue
Copy link
Contributor

@thetruestblue thetruestblue commented Apr 14, 2025

Because of the nature of this test (mmap stress-test) it tests too large of memory allocations to be stable on watchos. WatchOS has memory limits that can lead to the termination of this process before it reaches the limit set by the flag. Typical only on devices that are under heavy load. disable on watchOS.

rdar://147222346

@llvmbot
Copy link
Member

llvmbot commented Apr 14, 2025

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: None (thetruestblue)

Changes

Because of the nature of this test (mmap stress-test) it tests too large of memory allocations to be stable on watchos. WatchOS has memory limits that can lead to the termination of this process before it reaches the limit set by the flag. It is guaranteed to occur if a test device is under heavy load. disable on watchOS.

rdar://147222346


Full diff: https://github.com/llvm/llvm-project/pull/135671.diff

1 Files Affected:

  • (modified) compiler-rt/test/asan/TestCases/Posix/mmap_limit_mb.cpp (+4)
diff --git a/compiler-rt/test/asan/TestCases/Posix/mmap_limit_mb.cpp b/compiler-rt/test/asan/TestCases/Posix/mmap_limit_mb.cpp
index 87ac5b2925ed0..ed187e4481a4f 100644
--- a/compiler-rt/test/asan/TestCases/Posix/mmap_limit_mb.cpp
+++ b/compiler-rt/test/asan/TestCases/Posix/mmap_limit_mb.cpp
@@ -1,5 +1,9 @@
 // Test the mmap_limit_mb flag.
 //
+// stress-test isn't reliable on watchOS due to space limitations
+// UNSUPPORTED: watchos
+//
+//
 // RUN: %clangxx_asan -O2 %s -o %t
 // RUN: %run %t 20 16
 // RUN: %run %t 30 1000000

Because of the nature of this test (mmap stress-test) it tests too large of memory allocations to test on watchos. WatchOS has memory limits that can lead to the termination of this test before it reaches the limit set by the flag. We are going to disable on watchOS for now.

rdar://147222346
Copy link
Contributor

@wrotki wrotki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thetruestblue thetruestblue merged commit 8fe5ac8 into llvm:main Apr 14, 2025
9 of 10 checks passed
@thetruestblue thetruestblue deleted the mmap-limit branch April 14, 2025 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants