From e4ae79ca4a8dba86275f0cbe0b6a73fd14aac3ba Mon Sep 17 00:00:00 2001 From: German Andryeyev Date: Mon, 13 Sep 2021 15:05:20 -0400 Subject: [PATCH] SWDEV-282419 - Use HSA_AMD_SVM_ATTRIB_AGENT_ACCESSIBLE for unset When unsetting runtime should use HSA_AMD_SVM_ATTRIB_AGENT_ACCESSIBLE for the agent and not HSA_AMD_SVM_ATTRIB_AGENT_ACCESSIBLE_IN_PLACE Change-Id: I3814802d1fb3b72c54e7566defafafed6b0d5cee [ROCm/clr commit: d8a86e487026d6c64878df520f662c78c1b60fca] --- projects/clr/rocclr/device/rocm/rocdevice.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/projects/clr/rocclr/device/rocm/rocdevice.cpp b/projects/clr/rocclr/device/rocm/rocdevice.cpp index b4be397737..bfde477a4c 100644 --- a/projects/clr/rocclr/device/rocm/rocdevice.cpp +++ b/projects/clr/rocclr/device/rocm/rocdevice.cpp @@ -2268,8 +2268,8 @@ bool Device::SetSvmAttributesInt(const void* dev_ptr, size_t count, break; } case amd::MemoryAdvice::UnsetAccessedBy: - // @note: 0 may cause a failure on old runtimes - attr.push_back({HSA_AMD_SVM_ATTRIB_AGENT_ACCESSIBLE_IN_PLACE, 0}); + // When unsetting we should use HSA_AMD_SVM_ATTRIB_AGENT_ACCESSIBLE for the agent + attr.push_back({HSA_AMD_SVM_ATTRIB_AGENT_ACCESSIBLE, getBackendDevice().handle}); break; case amd::MemoryAdvice::SetCoarseGrain: attr.push_back({HSA_AMD_SVM_ATTRIB_GLOBAL_FLAG, HSA_AMD_SVM_GLOBAL_FLAG_COARSE_GRAINED});