From 0a7eb1fb2e9919034d8a0d64ddd9266ee898ed29 Mon Sep 17 00:00:00 2001
From: cosmonaut <evan@moonside.games>
Date: Mon, 28 Dec 2020 14:07:13 -0800
Subject: [PATCH] dummy uniform buffers

---
 src/Refresh_Driver_Vulkan.c | 85 ++++++++++++++++++++++++++++---------
 1 file changed, 65 insertions(+), 20 deletions(-)

diff --git a/src/Refresh_Driver_Vulkan.c b/src/Refresh_Driver_Vulkan.c
index f68d7a3..8a88938 100644
--- a/src/Refresh_Driver_Vulkan.c
+++ b/src/Refresh_Driver_Vulkan.c
@@ -1031,6 +1031,8 @@ typedef struct VulkanRenderer
 
 	VkDescriptorSetLayout vertexParamLayout;
 	VkDescriptorSetLayout fragmentParamLayout;
+	VulkanBuffer *dummyVertexUniformBuffer;
+	VulkanBuffer *dummyFragmentUniformBuffer;
 
 	VulkanBuffer *textureStagingBuffer;
 
@@ -3528,13 +3530,31 @@ static REFRESH_GraphicsPipeline* VULKAN_CreateGraphicsPipeline(
 		&graphicsPipeline->fragmentUBODescriptorSet
 	);
 
-	vertexUniformBufferInfo.buffer = renderer->vertexUBO->subBuffers[0]->buffer;
-	vertexUniformBufferInfo.offset = 0;
-	vertexUniformBufferInfo.range = graphicsPipeline->vertexUBOBlockSize;
+	if (graphicsPipeline->vertexUBOBlockSize == 0)
+	{
+		vertexUniformBufferInfo.buffer = renderer->dummyVertexUniformBuffer->subBuffers[0]->buffer;
+		vertexUniformBufferInfo.offset = 0;
+		vertexUniformBufferInfo.range = renderer->dummyVertexUniformBuffer->subBuffers[0]->size;
+	}
+	else
+	{
+		vertexUniformBufferInfo.buffer = renderer->vertexUBO->subBuffers[0]->buffer;
+		vertexUniformBufferInfo.offset = 0;
+		vertexUniformBufferInfo.range = graphicsPipeline->vertexUBOBlockSize;
+	}
 
-	fragmentUniformBufferInfo.buffer = renderer->fragmentUBO->subBuffers[0]->buffer;
-	fragmentUniformBufferInfo.offset = 0;
-	fragmentUniformBufferInfo.range = graphicsPipeline->fragmentUBOBlockSize;
+	if (graphicsPipeline->fragmentUBOBlockSize == 0)
+	{
+		fragmentUniformBufferInfo.buffer = renderer->dummyFragmentUniformBuffer->subBuffers[0]->buffer;
+		fragmentUniformBufferInfo.offset = 0;
+		fragmentUniformBufferInfo.range = renderer->dummyFragmentUniformBuffer->subBuffers[0]->size;
+	}
+	else
+	{
+		fragmentUniformBufferInfo.buffer = renderer->fragmentUBO->subBuffers[0]->buffer;
+		fragmentUniformBufferInfo.offset = 0;
+		fragmentUniformBufferInfo.range = graphicsPipeline->fragmentUBOBlockSize;
+	}
 
 	uboWriteDescriptorSets[0].sType = VK_STRUCTURE_TYPE_WRITE_DESCRIPTOR_SET;
 	uboWriteDescriptorSets[0].pNext = NULL;
@@ -6667,8 +6687,7 @@ static REFRESH_Device* VULKAN_CreateDevice(
 	/* Variables: UBO Creation */
 	VkDescriptorPoolCreateInfo defaultDescriptorPoolInfo;
 	VkDescriptorPoolSize poolSizes[2];
-	VkDescriptorSetAllocateInfo emptyVertexSamplerDescriptorAllocateInfo;
-	VkDescriptorSetAllocateInfo emptyFragmentSamplerDescriptorAllocateInfo;
+	VkDescriptorSetAllocateInfo descriptorAllocateInfo;
 
     result = (REFRESH_Device*) SDL_malloc(sizeof(REFRESH_Device));
     ASSIGN_DRIVER(VULKAN)
@@ -7076,27 +7095,23 @@ static REFRESH_Device* VULKAN_CreateDevice(
 		&renderer->defaultDescriptorPool
 	);
 
-	emptyVertexSamplerDescriptorAllocateInfo.sType = VK_STRUCTURE_TYPE_DESCRIPTOR_SET_ALLOCATE_INFO;
-	emptyVertexSamplerDescriptorAllocateInfo.pNext = NULL;
-	emptyVertexSamplerDescriptorAllocateInfo.descriptorPool = renderer->defaultDescriptorPool;
-	emptyVertexSamplerDescriptorAllocateInfo.descriptorSetCount = 1;
-	emptyVertexSamplerDescriptorAllocateInfo.pSetLayouts = &renderer->emptyVertexSamplerLayout;
+	descriptorAllocateInfo.sType = VK_STRUCTURE_TYPE_DESCRIPTOR_SET_ALLOCATE_INFO;
+	descriptorAllocateInfo.pNext = NULL;
+	descriptorAllocateInfo.descriptorPool = renderer->defaultDescriptorPool;
+	descriptorAllocateInfo.descriptorSetCount = 1;
+	descriptorAllocateInfo.pSetLayouts = &renderer->emptyVertexSamplerLayout;
 
 	renderer->vkAllocateDescriptorSets(
 		renderer->logicalDevice,
-		&emptyVertexSamplerDescriptorAllocateInfo,
+		&descriptorAllocateInfo,
 		&renderer->emptyVertexSamplerDescriptorSet
 	);
 
-	emptyFragmentSamplerDescriptorAllocateInfo.sType = VK_STRUCTURE_TYPE_DESCRIPTOR_SET_ALLOCATE_INFO;
-	emptyFragmentSamplerDescriptorAllocateInfo.pNext = NULL;
-	emptyFragmentSamplerDescriptorAllocateInfo.descriptorPool = renderer->defaultDescriptorPool;
-	emptyFragmentSamplerDescriptorAllocateInfo.descriptorSetCount = 1;
-	emptyFragmentSamplerDescriptorAllocateInfo.pSetLayouts = &renderer->emptyFragmentSamplerLayout;
+	descriptorAllocateInfo.pSetLayouts = &renderer->emptyFragmentSamplerLayout;
 
 	renderer->vkAllocateDescriptorSets(
 		renderer->logicalDevice,
-		&emptyFragmentSamplerDescriptorAllocateInfo,
+		&descriptorAllocateInfo,
 		&renderer->emptyFragmentSamplerDescriptorSet
 	);
 
@@ -7130,6 +7145,36 @@ static REFRESH_Device* VULKAN_CreateDevice(
 		return NULL;
 	}
 
+	/* Dummy Uniform Buffers */
+
+	renderer->dummyVertexUniformBuffer = (VulkanBuffer*) SDL_malloc(sizeof(VulkanBuffer));
+
+	if (!VULKAN_INTERNAL_CreateBuffer(
+		renderer,
+		16,
+		RESOURCE_ACCESS_VERTEX_SHADER_READ_UNIFORM_BUFFER,
+		VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT,
+		1,
+		renderer->dummyVertexUniformBuffer
+	)) {
+		REFRESH_LogError("Failed to create dummy vertex uniform buffer!");
+		return NULL;
+	}
+
+	renderer->dummyFragmentUniformBuffer = (VulkanBuffer*) SDL_malloc(sizeof(VulkanBuffer));
+
+	if (!VULKAN_INTERNAL_CreateBuffer(
+		renderer,
+		16,
+		RESOURCE_ACCESS_FRAGMENT_SHADER_READ_UNIFORM_BUFFER,
+		VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT,
+		1,
+		renderer->dummyFragmentUniformBuffer
+	)) {
+		REFRESH_LogError("Failed to create dummy fragment uniform buffer!");
+		return NULL;
+	}
+
 	/* Initialize caches */
 
 	for (i = 0; i < NUM_PIPELINE_LAYOUT_BUCKETS; i += 1)