Reduce usage of waitIdle in 15_hello_triangle

Author: asuessenbach

attachments/15_hello_triangle.cpp

@@ -63,7 +63,7 @@ class HelloTriangleApplication {
     vk::raii::CommandBuffer commandBuffer = nullptr;
 
     vk::raii::Semaphore presentCompleteSemaphore = nullptr;
-    vk::raii::Semaphore renderFinishedSemaphore = nullptr;
+    std::vector<vk::raii::Semaphore> renderFinishedSemaphores;
     vk::raii::Fence drawFence = nullptr;
 
     std::vector<const char*> requiredDeviceExtension = {
@@ -101,13 +101,11 @@ class HelloTriangleApplication {
             glfwPollEvents();
             drawFrame();
         }
-
-        device.waitIdle();
+        device.waitIdle();    // wait for device to finish operations before destroying resources
     }
 
     void cleanup() {
         glfwDestroyWindow(window);
-
         glfwTerminate();
     }
 
@@ -451,27 +449,36 @@ class HelloTriangleApplication {
 
     void createSyncObjects() {
         presentCompleteSemaphore =vk::raii::Semaphore(device, vk::SemaphoreCreateInfo());
-        renderFinishedSemaphore = vk::raii::Semaphore(device, vk::SemaphoreCreateInfo());
+        for (size_t i = 0; i < swapChainImages.size(); i++)
+        {
+          renderFinishedSemaphores.emplace_back(device, vk::SemaphoreCreateInfo());
+        }
         drawFence = vk::raii::Fence(device, {.flags = vk::FenceCreateFlagBits::eSignaled});
     }
 
     void drawFrame() {
-        queue.waitIdle();
+        while (vk::Result::eTimeout == device.waitForFences(*drawFence, vk::True, UINT64_MAX))
+          ;
+        device.resetFences(*drawFence);
 
         auto [result, imageIndex] = swapChain.acquireNextImage( UINT64_MAX, *presentCompleteSemaphore, nullptr );
         recordCommandBuffer(imageIndex);
 
-        device.resetFences(  *drawFence );
         vk::PipelineStageFlags waitDestinationStageMask( vk::PipelineStageFlagBits::eColorAttachmentOutput );
-        const vk::SubmitInfo submitInfo{ .waitSemaphoreCount = 1, .pWaitSemaphores = &*presentCompleteSemaphore,
-                            .pWaitDstStageMask = &waitDestinationStageMask, .commandBufferCount = 1, .pCommandBuffers = &*commandBuffer,
-                            .signalSemaphoreCount = 1, .pSignalSemaphores = &*renderFinishedSemaphore };
+        const vk::SubmitInfo submitInfo{ .waitSemaphoreCount = 1,
+                                         .pWaitSemaphores = &*presentCompleteSemaphore,
+                                         .pWaitDstStageMask = &waitDestinationStageMask,
+                                         .commandBufferCount = 1,
+                                         .pCommandBuffers = &*commandBuffer,
+                                         .signalSemaphoreCount = 1,
+                                         .pSignalSemaphores = &*renderFinishedSemaphores[imageIndex]};
         queue.submit(submitInfo, *drawFence);
-        while ( vk::Result::eTimeout == device.waitForFences( *drawFence, vk::True, UINT64_MAX ) )
-            ;
 
-        const vk::PresentInfoKHR presentInfoKHR{ .waitSemaphoreCount = 1, .pWaitSemaphores = &*renderFinishedSemaphore,
-                                                .swapchainCount = 1, .pSwapchains = &*swapChain, .pImageIndices = &imageIndex };
+        const vk::PresentInfoKHR presentInfoKHR{ .waitSemaphoreCount = 1,
+                                                 .pWaitSemaphores = &*renderFinishedSemaphores[imageIndex],
+                                                 .swapchainCount = 1,
+                                                 .pSwapchains = &*swapChain,
+                                                 .pImageIndices = &imageIndex };
         result = queue.presentKHR( presentInfoKHR );
         switch ( result )
         {