From 97515872396c55f5c27e3ab2b85a417a9b2bbaa6 Mon Sep 17 00:00:00 2001 From: Sean Keely Date: Wed, 11 Jul 2018 18:31:14 -0500 Subject: [PATCH] Do not initialize runtime internal queues based on mapping memory to a GPU. Conserves VMIDs when multiple processes are in use and memory operations are not GPU specific. For instance HIP API hipHostMalloc does not accept a target GPU so when used with one process per GPU (ie GPU == MPI rank) we can quickly exceed the available VMID slots if every process consumes a VMID on every GPU. Change-Id: Ib6fa051290089f71581029c09f9a44b9992237d1 [ROCm/ROCR-Runtime commit: 35a270ef7ec10df15cf5f14f9b1511f68f15d76c] --- .../hsa-runtime/core/runtime/amd_memory_region.cpp | 8 -------- 1 file changed, 8 deletions(-) diff --git a/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_memory_region.cpp b/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_memory_region.cpp index 39814839e5..4ed0f627b0 100644 --- a/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_memory_region.cpp +++ b/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_memory_region.cpp @@ -524,10 +524,6 @@ hsa_status_t MemoryRegion::AllowAccess(uint32_t num_agents, lock.Release(); - for (GpuAgentInt* gpu : whitelist_gpus) { - gpu->PreloadBlits(); - } - return HSA_STATUS_SUCCESS; } @@ -598,10 +594,6 @@ hsa_status_t MemoryRegion::Lock(uint32_t num_agents, const hsa_agent_t* agents, *agent_ptr = host_ptr; } - for (auto gpu : whitelist_gpus) { - static_cast(gpu)->PreloadBlits(); - } - return HSA_STATUS_SUCCESS; } amd::MemoryRegion::DeregisterMemory(host_ptr);