SWDEV-511055 - fix HIP PAL memory allocation workaround for APU (#40)

lttamd · web-flow · commit 95cdc83eaf58 · 2025-04-24T15:07:16.000-07:00
diff --git a/rocclr/device/pal/palmemory.cpp b/rocclr/device/pal/palmemory.cpp
@@ -118,8 +118,7 @@ bool Memory::create(Resource::MemoryType memType, Resource::CreateParams* params
     if (amd::IS_HIP && dev().settings().apuSystem_) {
       const Pal::GpuMemoryHeapProperties& invisibleHeap = dev().GetGpuHeapInvisible();
       Pal::gpusize totalAlloc = dev().TotalAlloc();
-      if (invisibleHeap.logicalSize > 0 && memType == Local &&
-          (totalAlloc > invisibleHeap.logicalSize)) {
+      if (memType == Local && (totalAlloc > invisibleHeap.logicalSize)) {
         memType = RemoteUSWC;
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -118,8 +118,7 @@ bool Memory::create(Resource::MemoryType memType, Resource::CreateParams* params`
`118`	`118`	`if (amd::IS_HIP && dev().settings().apuSystem_) {`
`119`	`119`	`const Pal::GpuMemoryHeapProperties& invisibleHeap = dev().GetGpuHeapInvisible();`
`120`	`120`	`Pal::gpusize totalAlloc = dev().TotalAlloc();`
`121`		`- if (invisibleHeap.logicalSize > 0 && memType == Local &&`
`122`		`- (totalAlloc > invisibleHeap.logicalSize)) {`
	`121`	`+ if (memType == Local && (totalAlloc > invisibleHeap.logicalSize)) {`
`123`	`122`	`memType = RemoteUSWC;`
`124`	`123`	`}`
`125`	`124`	`}`