commit a49a4fba5f4c2cb9d5791db33e36035b0fe84cea
author dan sinclair <dsinclair@chromium.org> Mon Mar 31 08:52:40 2025 -0700
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Mar 31 08:52:40 2025 -0700
tree ac97aba39cd7d5b05dd74cd410f1b68e8d576a31
parent 10b7f5ab4bbe084ec38c11196e6580fc060fe937

[tsan] Suppress BufferTests.CreateBufferOOMWithValidationError on TSAN.

When running under TSAN the
`BufferTests.CreateBufferOOMWithValidationError` test attempts
to allocate a buffer which is larger then the max allowed. This causes
the TSAN test to fail. Add a suppression for the test instead of
attempting to run and failing.

Bug: 407519731
Change-Id: I9339de7e5e27a7f1978a4b7b347a44e53f8b5c8c
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/233794
Auto-Submit: dan sinclair <dsinclair@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: dan sinclair <dsinclair@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>

src/dawn/tests/end2end/BufferTests.cpp

diff --git a/src/dawn/tests/end2end/BufferTests.cpp b/src/dawn/tests/end2end/BufferTests.cpp
index ce6c2dd..7db701b 100644
--- a/src/dawn/tests/end2end/BufferTests.cpp
+++ b/src/dawn/tests/end2end/BufferTests.cpp
@@ -1084,6 +1084,10 @@
 
 // Test that in the creation of buffers validation errors should always be prior to OOM.
 TEST_P(BufferTests, CreateBufferOOMWithValidationError) {
+    // Fails on TSAN due to allowing too much memory. TSAN max is `0x10000000000` and the test
+    // allocates `0x4000000000000000`
+    DAWN_SUPPRESS_TEST_IF(IsTsan());
+
     DAWN_TEST_UNSUPPORTED_IF(HasToggleEnabled("skip_validation"));
 
     wgpu::BufferDescriptor descriptor;