From c53e11ec202e2d825b01caa37e0501bbbfa050a2 Mon Sep 17 00:00:00 2001 From: David Yat Sin Date: Tue, 30 Apr 2024 03:16:00 +0000 Subject: [PATCH] Temporary: Do not early release mutex when not ganging It seesm the Release() function is not reliable and can cause segfaults. This is a temporary work-around until the Release() function is fixed. Change-Id: I95470a800c6153673e4b8f4fe46a646903325074 [ROCm/ROCR-Runtime commit: ac5fb8be9e1bbe8de0ef7e59b7e8fc6f033a4efc] --- .../runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp b/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp index 8273ca2cb5..40d6ea06ad 100644 --- a/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp +++ b/projects/rocr-runtime/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp @@ -945,7 +945,6 @@ hsa_status_t GpuAgent::DmaCopy(void* dst, core::Agent& dst_agent, } ScopedAcquire lock(&sdma_gang_lock_); - if (gang_factor == 1) sdma_gang_lock_.Release(); // Manage internal gang signals std::vector gang_signals; if (gang_factor > 1) {