[Rendering] Reuse command buffer;

Author: LittleCodingFox

Engine/Staple.Core/Rendering/RenderSystem/Backend/IRenderCommand.cs

@@ -1,4 +1,5 @@
 using System;
+using System.Numerics;
 
 namespace Staple.Internal;
 
@@ -22,7 +23,8 @@ internal interface IRendererBackend
 
     void UpdateViewport(int width, int height);
 
-    IRenderCommand BeginCommand();
+    IRenderPass BeginRenderPass(RenderTarget target, CameraClearMode clear, Color clearColor, Vector4 viewport,
+        Matrix4x4 view, Matrix4x4 projection);
 
     VertexBuffer CreateVertexBuffer(Span<byte> data, VertexLayout layout, RenderBufferFlags flags);
 

Engine/Staple.Core/Rendering/RenderSystem/Backend/IRendererBackend.cs

@@ -10,17 +10,17 @@ internal class SDLGPUIndexBuffer : IndexBuffer
 
     private readonly nint device;
     private readonly RenderBufferFlags flags;
-    private readonly Func<SDLGPURenderCommand> commandSupplier;
+    private readonly SDLGPURendererBackend backend;
     private nint transferBuffer;
     private int length;
 
     public bool Valid => buffer != nint.Zero && transferBuffer != nint.Zero;
 
-    public SDLGPUIndexBuffer(nint device, RenderBufferFlags flags, Func<SDLGPURenderCommand> commandSupplier)
+    public SDLGPUIndexBuffer(nint device, RenderBufferFlags flags, SDLGPURendererBackend backend)
     {
         this.device = device;
         this.flags = flags;
-        this.commandSupplier = commandSupplier;
+        this.backend = backend;
 
         ResourceManager.instance.userCreatedIndexBuffers.Add(new(this));
     }
@@ -130,14 +130,14 @@ public override void Update(Span<ushort> data)
 
         ResizeIfNeeded(data.Length * byteSize);
 
-        if (Valid == false)
+        if (Valid == false || backend.TryGetCommandBuffer(out var command) == false)
         {
             return;
         }
 
-        var command = commandSupplier();
+        var copyPass = SDL.SDL_BeginGPUCopyPass(command);
 
-        if (command == null)
+        if (copyPass == nint.Zero)
         {
             return;
         }
@@ -153,15 +153,6 @@ public override void Update(Span<ushort> data)
 
         SDL.SDL_UnmapGPUTransferBuffer(device, transferBuffer);
 
-        var copyPass = SDL.SDL_BeginGPUCopyPass(command.commandBuffer);
-
-        if (copyPass == nint.Zero)
-        {
-            command.Discard();
-
-            return;
-        }
-
         var location = new SDL.SDL_GPUTransferBufferLocation()
         {
             transfer_buffer = transferBuffer,
@@ -178,8 +169,6 @@ public override void Update(Span<ushort> data)
 
         SDL.SDL_EndGPUCopyPass(copyPass);
 
-        command.Submit();
-
         Is32Bit = false;
     }
 
@@ -197,14 +186,7 @@ public override void Update(Span<uint> data)
 
         ResizeIfNeeded(byteSize);
 
-        if (Valid == false)
-        {
-            return;
-        }
-
-        var command = commandSupplier();
-
-        if (command == null)
+        if (Valid == false || backend.TryGetCommandBuffer(out var command) == false)
         {
             return;
         }
@@ -220,12 +202,10 @@ public override void Update(Span<uint> data)
 
         SDL.SDL_UnmapGPUTransferBuffer(device, transferBuffer);
 
-        var copyPass = SDL.SDL_BeginGPUCopyPass(command.commandBuffer);
+        var copyPass = SDL.SDL_BeginGPUCopyPass(command);
 
         if (copyPass == nint.Zero)
         {
-            command.Discard();
-
             return;
         }
 
@@ -245,8 +225,6 @@ public override void Update(Span<uint> data)
 
         SDL.SDL_EndGPUCopyPass(copyPass);
 
-        command.Submit();
-
         Is32Bit = true;
     }
 }