From 821f6e58f9e87040dc77b289c5a2fcbba47c7644 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 9f0f7741de0f41b9671da68ac2fadd5710aa64da. 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 --- libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp b/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp index a31630c510..866e8d8e86 100644 --- a/libhsakmt/tests/kfdtest/src/KFDTestUtil.cpp +++ b/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;