From f853dda9ba4baba7d2f3db4eeb99c85713255733 Mon Sep 17 00:00:00 2001 From: Apurv Mishra Date: Thu, 8 May 2025 14:45:17 -0400 Subject: [PATCH] kfdtest: Disable tests that cause unwanted behavior disable KFDLocalMemoryTest.Fragmentation and KFDEventTest.MeasureInterruptConsumption as part of the KFD test suite improvement feature Signed-off-by: Apurv Mishra --- libhsakmt/tests/kfdtest/scripts/kfdtest.exclude | 8 +------- libhsakmt/tests/kfdtest/src/KFDEventTest.cpp | 2 +- libhsakmt/tests/kfdtest/src/KFDLocalMemoryTest.cpp | 2 +- 3 files changed, 3 insertions(+), 9 deletions(-) diff --git a/libhsakmt/tests/kfdtest/scripts/kfdtest.exclude b/libhsakmt/tests/kfdtest/scripts/kfdtest.exclude index 49230c766d..7416a80f48 100644 --- a/libhsakmt/tests/kfdtest/scripts/kfdtest.exclude +++ b/libhsakmt/tests/kfdtest/scripts/kfdtest.exclude @@ -34,10 +34,6 @@ FILTER[core]=\ # FILTER[pm] need human intervention, so put it here. Developers can run them # manually through "-p pm" option. # -# Fragmentation test fails intermittently on different systems (see KFD-344). It -# stresses memory management in unusual ways by having lots of memory allocated but -# not mapped, which is not relevant to compute applications over ROCr. -# # CU Masking Linear are not working correctly due to how the HW distributes work over CUs. # They are available for testing but are not currently expected to pass on CI/VI/AI. # @@ -48,9 +44,7 @@ FILTER[core]=\ # The CheckZeroInitializationVram test is no longer expected to pass as KFD no longer # clears memory at allocation time. PERMANENT_BLACKLIST_ALL_ASICS=\ -"-KFDEventTest.MeasureInterruptConsumption:"\ -"${FILTER[pm]}:"\ -"KFDLocalMemoryTest.Fragmentation:"\ +"-${FILTER[pm]}:"\ "KFDQMTest.BasicCuMaskingLinear:"\ "KFDQMTest.BasicCuMaskingEven:"\ "RDMATest.GPUDirect:"\ diff --git a/libhsakmt/tests/kfdtest/src/KFDEventTest.cpp b/libhsakmt/tests/kfdtest/src/KFDEventTest.cpp index 2c008afcd2..9a997e4be2 100644 --- a/libhsakmt/tests/kfdtest/src/KFDEventTest.cpp +++ b/libhsakmt/tests/kfdtest/src/KFDEventTest.cpp @@ -334,7 +334,7 @@ exit: } }; -TEST_F(KFDEventTest, MeasureInterruptConsumption) { +TEST_F(KFDEventTest, DISABLED_MeasureInterruptConsumption) { TEST_START(TESTPROFILE_RUNALL); QueueAndSignalBenchmark latencyBench(128); QueueAndSignalBenchmark sustainedBench(4095); diff --git a/libhsakmt/tests/kfdtest/src/KFDLocalMemoryTest.cpp b/libhsakmt/tests/kfdtest/src/KFDLocalMemoryTest.cpp index ccc68abd8a..fc8a8e1f68 100644 --- a/libhsakmt/tests/kfdtest/src/KFDLocalMemoryTest.cpp +++ b/libhsakmt/tests/kfdtest/src/KFDLocalMemoryTest.cpp @@ -435,7 +435,7 @@ static void Fragmentation(KFDTEST_PARAMETERS* pTestParamters){ EXPECT_SUCCESS_GPU(queue.Destroy(), gpuNode); } -TEST_F(KFDLocalMemoryTest, Fragmentation) { +TEST_F(KFDLocalMemoryTest, DISABLED_Fragmentation) { TEST_REQUIRE_ENV_CAPABILITIES(ENVCAPS_64BITLINUX); TEST_START(TESTPROFILE_RUNALL);