commit 0f1257e42c84f64dc135335f13b11b154a0181ac
author Brian Sheedy <bsheedy@google.com> Tue Aug 20 20:06:56 2024 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Aug 20 20:06:56 2024 +0000
tree 0b069deb359ef34da305498f7efcb24e108212d6
parent 4c92f4b85680a5f32f635c3abb7091b2f1ef2595

Skip BufferHostMappedPointerTest on Mac

Skips BufferHostMappedPointerTests.InitialDataAndCopySrc on Mac/AMD
since it is suspected of causing flaky crashes in subsequent tests on
the 16" AMD Macbook configuration.

Bug: 358296955
Change-Id: Ic5f4d2d9d33e87d1ed1fff39d375974176d2c8c2
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/203066
Commit-Queue: Brian Sheedy <bsheedy@google.com>
Auto-Submit: Brian Sheedy <bsheedy@google.com>
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Austin Eng <enga@chromium.org>

src/dawn/tests/end2end/BufferHostMappedPointerTests.cpp

diff --git a/src/dawn/tests/end2end/BufferHostMappedPointerTests.cpp b/src/dawn/tests/end2end/BufferHostMappedPointerTests.cpp
index 9aa041b..6221d5a 100644
--- a/src/dawn/tests/end2end/BufferHostMappedPointerTests.cpp
+++ b/src/dawn/tests/end2end/BufferHostMappedPointerTests.cpp
@@ -128,6 +128,10 @@
 // It should be GPU-visible immediately after creation.
 // Then, change the host pointer, and see changes reflected on the GPU.
 TEST_P(BufferHostMappedPointerTests, InitialDataAndCopySrc) {
+    // TODO(crbug.com/358296955): Re-enable this once it no longer causes
+    // subsequent tests to flakily crash.
+    DAWN_SUPPRESS_TEST_IF(IsMacOS() && IsAMD() && IsMetal());
+
     // Set up expected data.
     uint32_t bufferSize = mRequiredAlignment;
     std::vector<uint32_t> expected(bufferSize / sizeof(uint32_t));