Buffer and Submission Rework (#1)

We used to have monolithic uniform buffers on the VulkanRenderer object, but this was inefficient in the case of threaded usage.

Now, we have a pool of uniform buffers. A uniform buffer is selected from the pool when a pipeline is bound. The uniform buffers are rotated upon presentation.

Now pushing uniforms is now a concern of the command buffer instead of the pipeline. The pipeline should just always have been a static object anyway.

Additionally, we now do extra buffer record-keeping so that buffer data can be updated after a bind/draw.

Fence submission has also been restructured so that submissions don't cause unnecessary blocks. Now we assign one fence per submission, and we don't wait for fences until it's time to present.

Reviewed-on: MoonsideGames/Refresh#1
Co-authored-by: cosmonaut <evan@moonside.games>
Co-committed-by: cosmonaut <evan@moonside.games>

include/Refresh.h

@@ -983,7 +983,7 @@ REFRESHAPI void Refresh_SetBufferData(
  */
 REFRESHAPI uint32_t Refresh_PushVertexShaderUniforms(
 	Refresh_Device *device,
-	Refresh_GraphicsPipeline *pipeline,
+	Refresh_CommandBuffer * commandBuffer,
 	void *data,
 	uint32_t dataLengthInBytes
 );
@@ -1000,7 +1000,7 @@ REFRESHAPI uint32_t Refresh_PushVertexShaderUniforms(
  */
 REFRESHAPI uint32_t Refresh_PushFragmentShaderUniforms(
 	Refresh_Device *device,
-	Refresh_GraphicsPipeline *pipeline,
+	Refresh_CommandBuffer *commandBuffer,
 	void *data,
 	uint32_t dataLengthInBytes
 );
@@ -1017,7 +1017,7 @@ REFRESHAPI uint32_t Refresh_PushFragmentShaderUniforms(
  */
 REFRESHAPI uint32_t Refresh_PushComputeShaderUniforms(
 	Refresh_Device *device,
-	Refresh_ComputePipeline *pipeline,
+	Refresh_CommandBuffer * commandBuffer,
 	void *data,
 	uint32_t dataLengthInBytes
 );

src/Refresh.c

@@ -470,14 +470,14 @@ void Refresh_SetBufferData(
 
 uint32_t Refresh_PushVertexShaderUniforms(
 	Refresh_Device *device,
-    Refresh_GraphicsPipeline *pipeline,
+    Refresh_CommandBuffer *commandBuffer,
 	void *data,
 	uint32_t dataLengthInBytes
 ) {
     if (device == NULL) { return 0; }
     return device->PushVertexShaderUniforms(
         device->driverData,
-        pipeline,
+        commandBuffer,
         data,
         dataLengthInBytes
     );
@@ -485,14 +485,14 @@ uint32_t Refresh_PushVertexShaderUniforms(
 
 uint32_t Refresh_PushFragmentShaderUniforms(
 	Refresh_Device *device,
-    Refresh_GraphicsPipeline * pipeline,
+    Refresh_CommandBuffer *commandBuffer,
 	void *data,
 	uint32_t dataLengthInBytes
 ) {
     if (device == NULL) { return 0; }
     return device->PushFragmentShaderUniforms(
         device->driverData,
-        pipeline,
+        commandBuffer,
         data,
         dataLengthInBytes
     );
@@ -500,14 +500,14 @@ uint32_t Refresh_PushFragmentShaderUniforms(
 
 uint32_t Refresh_PushComputeShaderUniforms(
     Refresh_Device *device,
-    Refresh_ComputePipeline *pipeline,
+    Refresh_CommandBuffer *commandBuffer,
     void *data,
     uint32_t dataLengthInBytes
 ) {
     if (device == NULL) { return 0; }
     return device->PushComputeShaderUniforms(
         device->driverData,
-        pipeline,
+        commandBuffer,
         data,
         dataLengthInBytes
     );

src/Refresh_Driver.h

@@ -314,21 +314,21 @@ struct Refresh_Device
 
     uint32_t(*PushVertexShaderUniforms)(
         Refresh_Renderer *driverData,
-        Refresh_GraphicsPipeline* pipeline,
+        Refresh_CommandBuffer *commandBuffer,
         void *data,
         uint32_t dataLengthInBytes
     );
 
     uint32_t(*PushFragmentShaderUniforms)(
         Refresh_Renderer *driverData,
-        Refresh_GraphicsPipeline *pipeline,
+        Refresh_CommandBuffer *commandBuffer,
         void *data,
         uint32_t dataLengthInBytes
     );
 
     uint32_t (*PushComputeShaderUniforms)(
         Refresh_Renderer *driverData,
-        Refresh_ComputePipeline *pipeline,
+        Refresh_CommandBuffer *commandBuffer,
         void *data,
         uint32_t dataLengthInBytes
     );