From 5a1e09a37d3a71eb1a5b17ee6dad8a634e8bb8bc Mon Sep 17 00:00:00 2001 From: kjayapra-amd Date: Tue, 9 Jun 2020 20:10:18 -0400 Subject: [PATCH] SWDEV-229840 - Add Debug prints when the element is already present in MemObjMap_ Change-Id: I21129d087e73cc2a9e35f03e6a1a2dc527626f48 [ROCm/clr commit: 71c05075ba4654b207f822dcc981359f815254e4] --- projects/clr/rocclr/device/device.cpp | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) mode change 100644 => 100755 projects/clr/rocclr/device/device.cpp diff --git a/projects/clr/rocclr/device/device.cpp b/projects/clr/rocclr/device/device.cpp old mode 100644 new mode 100755 index 64bc69d4fb..979fcb0d47 --- a/projects/clr/rocclr/device/device.cpp +++ b/projects/clr/rocclr/device/device.cpp @@ -95,12 +95,22 @@ size_t MemObjMap::size() { void MemObjMap::AddMemObj(const void* k, amd::Memory* v) { amd::ScopedLock lock(AllocatedLock_); - MemObjMap_.insert({reinterpret_cast(k), v}); + auto rval = MemObjMap_.insert({ reinterpret_cast(k), v }); + if (!rval.second) { + DevLogPrintfError("Memobj map already has an entry for ptr: 0x%x", + reinterpret_cast(k)); + guarantee(false); + } } void MemObjMap::RemoveMemObj(const void* k) { amd::ScopedLock lock(AllocatedLock_); - MemObjMap_.erase(reinterpret_cast(k)); + auto rval = MemObjMap_.erase(reinterpret_cast(k)); + if (rval != 1) { + DevLogPrintfError("Memobj map does not have ptr: 0x%x", + reinterpret_cast(k)); + guarantee(false); + } } amd::Memory* MemObjMap::FindMemObj(const void* k) {