Changeset c074f81 in opengl-game for vulkan-utils.hpp

Timestamp:

Jun 8, 2021, 2:29:12 PM (3 years ago)

Author:

Dmitry Portnoy <dportnoy@…>

Branches:

feature/imgui-sdl

Children:

8aa4888

Parents:

567fa88

Message:

Change VulkanUtils::copyDataToMemory() to always require a size and to optionally flush the memory. Also add a VulkanUtils::copyDataToMappedMemory function that does the same thing, but assumes the data is already mapped.

File:

• vulkan-utils.hpp (modified) (2 diffs)

vulkan-utils.hpp

@@ -103 +103 @@
 
       template<class DataType>
-      static void copyDataToMemory(VkDevice device, const DataType& srcData, VkDeviceMemory bufferMemory,
-                                   VkDeviceSize offset);
+      static void copyDataToMemory(VkDevice device, DataType* srcData, VkDeviceMemory bufferMemory,
+                                   VkDeviceSize offset, VkDeviceSize size, bool flush);
 
       template<class DataType>
-      static void copyDataToMemory(VkDevice device, const DataType& srcData, VkDeviceMemory bufferMemory,
-                                   VkDeviceSize offset, VkDeviceSize size);
+      static void copyDataToMappedMemory(VkDevice device, DataType* srcData, void* mappedData,
+                                         VkDeviceMemory bufferMemory, VkDeviceSize size, bool flush);
 
       static bool hasStencilComponent(VkFormat format);
@@ -140 +140 @@
 
 template<class DataType>
-void VulkanUtils::copyDataToMemory(VkDevice device, const DataType& srcData, VkDeviceMemory bufferMemory,
-                                   VkDeviceSize offset) {
-   copyDataToMemory(device, srcData, bufferMemory, offset, sizeof(DataType));
-}
-
-// TODO: This would be used when the GPU memory is host-coherent. If it it not, I also need to use vkFlushMappedMemoryRanges
-// I should create a variant that supports non-coherent memory
-template<class DataType>
-void VulkanUtils::copyDataToMemory(VkDevice device, const DataType& srcData, VkDeviceMemory bufferMemory,
-                                   VkDeviceSize offset, VkDeviceSize size) {
+void VulkanUtils::copyDataToMemory(VkDevice device, DataType* srcData, VkDeviceMemory bufferMemory,
+                                   VkDeviceSize offset, VkDeviceSize size, bool flush) {
    void* data;
 
    vkMapMemory(device, bufferMemory, offset, size, 0, &data);
-   memcpy(data, &srcData, size);
+
+   memcpy(data, srcData, size);
+
+   if (flush) {
+      VkMappedMemoryRange memoryRange{};
+      memoryRange.sType = VK_STRUCTURE_TYPE_MAPPED_MEMORY_RANGE;
+      memoryRange.memory = bufferMemory;
+      memoryRange.size = size;
+
+      vkFlushMappedMemoryRanges(device, 1, &memoryRange);
+   }
+
    vkUnmapMemory(device, bufferMemory);
+}
+
+template<class DataType>
+void VulkanUtils::copyDataToMappedMemory(VkDevice device, DataType* srcData, void* mappedData,
+                                         VkDeviceMemory bufferMemory, VkDeviceSize size, bool flush) {
+   memcpy(mappedData, srcData, size);
+
+   if (flush) {
+      VkMappedMemoryRange memoryRange{};
+      memoryRange.sType = VK_STRUCTURE_TYPE_MAPPED_MEMORY_RANGE;
+      memoryRange.memory = bufferMemory;
+      memoryRange.size = size;
+
+      vkFlushMappedMemoryRanges(device, 1, &memoryRange);
+   }
 }