From 1c517346429228e6139060a47351358d4719cea9 Mon Sep 17 00:00:00 2001 From: Mateusz Hoppe Date: Fri, 30 Aug 2024 13:10:58 +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;