diff --git a/compiler-rt/lib/scudo/standalone/tests/common_test.cpp b/compiler-rt/lib/scudo/standalone/tests/common_test.cpp --- a/compiler-rt/lib/scudo/standalone/tests/common_test.cpp +++ b/compiler-rt/lib/scudo/standalone/tests/common_test.cpp @@ -69,9 +69,11 @@ unmap(P, Size, 0, &Data); } -#if SCUDO_LINUX && !defined(__powerpc__) -// This test fails intermediately on PPC, which is why this test is disabled -// for now on this platform. +#if 0 +// This test is temorarily disabled because it may not work as expected. E.g., +// it doesn't dirty the pages so the pages may not be commited and it may only +// work on the single thread environment. As a result, this test is flaky and is +// impacting many test scenarios. TEST(ScudoCommonTest, GetRssFromBuffer) { constexpr int64_t AllocSize = 10000000; constexpr int64_t Error = 3000000; @@ -88,6 +90,6 @@ EXPECT_LE(std::abs(Rss - AllocSize - Prev), Error); } } -#endif // SCUDO_LINUX +#endif } // namespace scudo