From 7ec6819ff5340f4f43f9ee7273d084ce02168cbd Mon Sep 17 00:00:00 2001 From: Shane Xiao Date: Thu, 22 Aug 2024 11:10:06 +0800 Subject: [PATCH] Revert "gfx11 is able to perform atomic ops even PCI reports no atomic support." This reverts commit 7daedd5eef6d7062caf4cc64f28318132b879d90. For APU, the PCIe atomic is supported by default. However, the PCIe atomic feature needs to checked for dGPU. The kfd driver has already set PCIe atomic support for APUs, so this patch can be reverted. Change-Id: I131d5b8e095c1104e1695e7cf8b1ed178bccddde Signed-off-by: Shane Xiao [ROCm/ROCR-Runtime commit: 821f6e58f9e87040dc77b289c5a2fcbba47c7644] --- .../rocr-runtime/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/projects/rocr-runtime/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp b/projects/rocr-runtime/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp index a31630c510..866e8d8e86 100644 --- a/projects/rocr-runtime/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp +++ b/projects/rocr-runtime/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp @@ -148,10 +148,6 @@ bool hasPciAtomicsSupport(int node) { if (pNodeProperties->NumCPUCores && pNodeProperties->NumFComputeCores) return true; - /* gfx11 is able to perform aotmic ops even PCI reports no atomic support. */ - if (pNodeProperties->EngineId.ui32.Major >= 11) - return true; - HsaIoLinkProperties *IolinkProperties = new HsaIoLinkProperties[pNodeProperties->NumIOLinks]; if (hsaKmtGetNodeIoLinkProperties(node, pNodeProperties->NumIOLinks, IolinkProperties)) { LOG() << "Unable to get Node IO Link Information for node " << node << std::endl;