From dc784ba482aef13fbce998d076443ced1c91f3c9 Mon Sep 17 00:00:00 2001 From: Kent Russell Date: Wed, 6 May 2020 14:01:59 -0400 Subject: [PATCH] Fix HostAccess flags for XGMI configs in P2PTest We need HostAccess for large-bar XGMI configs for this test, but we can't use it on small-bar XGMI configs, so deal with it appropriately Change-Id: Ic16b810de03adfc63de21a00c55e5f3ea1ce66d1 --- tests/kfdtest/src/KFDQMTest.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tests/kfdtest/src/KFDQMTest.cpp b/tests/kfdtest/src/KFDQMTest.cpp index 10f74ae808..f2b68ce233 100644 --- a/tests/kfdtest/src/KFDQMTest.cpp +++ b/tests/kfdtest/src/KFDQMTest.cpp @@ -1671,11 +1671,16 @@ TEST_F(KFDQMTest, P2PTest) { HsaMemFlags memFlags = {0}; HsaMemMapFlags mapFlags = {0}; memFlags.ui32.PageSize = HSA_PAGE_SIZE_4KB; - memFlags.ui32.HostAccess = 0; + memFlags.ui32.HostAccess = 1; memFlags.ui32.NonPaged = 1; memFlags.ui32.NoNUMABind = 1; unsigned int end = size / sizeof(HSAuint32) - 1; + if (!m_NodeInfo.IsGPUNodeLargeBar(g_TestDstNodeId) && + m_NodeInfo.AreGPUNodesXGMI(g_TestNodeId, g_TestDstNodeId)) { + memFlags.ui32.HostAccess = 0; + } + /* 1. Allocate a system buffer and allow the access to GPUs */ EXPECT_SUCCESS(hsaKmtAllocMemory(0, size, memFlags, reinterpret_cast(&sysBuf)));