From 32fb476f6d725c78dd4a18ddff42a8309a9527d3 Mon Sep 17 00:00:00 2001 From: Mateusz Hoppe Date: Wed, 4 Sep 2024 09:30:40 +0000 Subject: [PATCH] refactor: debugbreak in simulation on memcopy to multi bank allocation - if allocation has multiple banks, memcpy cannot write different data for multiple banks Signed-off-by: Mateusz Hoppe --- shared/source/memory_manager/memory_manager.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/shared/source/memory_manager/memory_manager.cpp b/shared/source/memory_manager/memory_manager.cpp index 32dd60f58c..fe2ddb51b2 100644 --- a/shared/source/memory_manager/memory_manager.cpp +++ b/shared/source/memory_manager/memory_manager.cpp @@ -891,6 +891,8 @@ bool MemoryManager::copyMemoryToAllocation(GraphicsAllocation *graphicsAllocatio } bool MemoryManager::copyMemoryToAllocationBanks(GraphicsAllocation *graphicsAllocation, size_t destinationOffset, const void *memoryToCopy, size_t sizeToCopy, DeviceBitfield handleMask) { + DEBUG_BREAK_IF(graphicsAllocation->storageInfo.getNumBanks() > 1 && handleMask.count() > 0); + memcpy_s(ptrOffset(static_cast(graphicsAllocation->getUnderlyingBuffer()), destinationOffset), (graphicsAllocation->getUnderlyingBufferSize() - destinationOffset), memoryToCopy, sizeToCopy); return true;