Merge branch 'SaschaWillems:master' into master

Author: LegNeato

README.md

@@ -83,7 +83,7 @@ Note that some examples require specific device features, and if you are on a mu
 
 ## Shaders
 
-Vulkan consumes shaders in an intermediate representation called SPIR-V. This makes it possible to use different shader languages by compiling them to that bytecode format. The primary shader language used here is [GLSL](shaders/glsl), most samples also come with [slang](shaders/slang/) and [HLSL](shaders/hlsl) shader sources, making it easy to compare the differences between those shading languages.
+Vulkan consumes shaders in an intermediate representation called SPIR-V. This makes it possible to use different shader languages by compiling them to that bytecode format. The primary shader language used here is [GLSL](shaders/glsl), most samples also come with [slang](shaders/slang/) and [HLSL](shaders/hlsl) shader sources, making it easy to compare the differences between those shading languages. The [Rust GPU](https://rust-gpu.github.io/) project maintains [Rust](https://www.rust-lang.org/) shader sources in a [separate repo](https://github.com/Rust-GPU/VulkanShaderExamples/tree/master/shaders/rust).
 
 ## A note on synchronization
 

examples/triangle/triangle.cpp

@@ -110,9 +110,8 @@ class VulkanExample : public VulkanExampleBase
 	std::array<VkCommandBuffer, MAX_CONCURRENT_FRAMES> commandBuffers{};
 	std::array<VkFence, MAX_CONCURRENT_FRAMES> waitFences{};
 
-	// To select the correct sync and command objects, we need to keep track of the current frame and (swapchain) image index
+	// To select the correct sync and command objects, we need to keep track of the current frame
 	uint32_t currentFrame{ 0 };
-	uint32_t currentSemaphore{ 0 };
 
 	VulkanExample() : VulkanExampleBase()
 	{
@@ -189,16 +188,19 @@ class VulkanExample : public VulkanExampleBase
 			// Fence used to ensure that command buffer has completed exection before using it again
 			VK_CHECK_RESULT(vkCreateFence(device, &fenceCI, nullptr, &waitFences[i]));
 		}
-		// Semaphores are per swapchain image
-		presentCompleteSemaphores.resize(swapChain.images.size());
+		// Semaphores are used for correct command ordering within a queue
+		// Used to ensure that image presentation is complete before starting to submit again
+		presentCompleteSemaphores.resize(MAX_CONCURRENT_FRAMES);
+		for (auto& semaphore : presentCompleteSemaphores) {
+			VkSemaphoreCreateInfo semaphoreCI{ VK_STRUCTURE_TYPE_SEMAPHORE_CREATE_INFO };
+			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &semaphore));
+		}
+		// Render completion
+		// Semaphore used to ensure that all commands submitted have been finished before submitting the image to the queue
 		renderCompleteSemaphores.resize(swapChain.images.size());
-		for (size_t i = 0; i < swapChain.images.size(); i++) {
-			// Semaphores are used for correct command ordering within a queue
+		for (auto& semaphore : renderCompleteSemaphores) {
 			VkSemaphoreCreateInfo semaphoreCI{ VK_STRUCTURE_TYPE_SEMAPHORE_CREATE_INFO };
-			// Semaphore used to ensure that image presentation is complete before starting to submit again
-			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &presentCompleteSemaphores[i]));
-			// Semaphore used to ensure that all commands submitted have been finished before submitting the image to the queue
-			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &renderCompleteSemaphores[i]));
+			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &semaphore));
 		}
 	}
 
@@ -915,7 +917,7 @@ class VulkanExample : public VulkanExampleBase
 		// Get the next swap chain image from the implementation
 		// Note that the implementation is free to return the images in any order, so we must use the acquire function and can't just cycle through the images/imageIndex on our own
 		uint32_t imageIndex;
-		VkResult result = vkAcquireNextImageKHR(device, swapChain.swapChain, UINT64_MAX, presentCompleteSemaphores[currentSemaphore], VK_NULL_HANDLE, &imageIndex);
+		VkResult result = vkAcquireNextImageKHR(device, swapChain.swapChain, UINT64_MAX, presentCompleteSemaphores[currentFrame], VK_NULL_HANDLE, &imageIndex);
 		if (result == VK_ERROR_OUT_OF_DATE_KHR) {
 			windowResize();
 			return;
@@ -1011,10 +1013,10 @@ class VulkanExample : public VulkanExampleBase
 		submitInfo.commandBufferCount = 1;                  // We submit a single command buffer
 
 		// Semaphore to wait upon before the submitted command buffer starts executing
-		submitInfo.pWaitSemaphores = &presentCompleteSemaphores[currentSemaphore];
+		submitInfo.pWaitSemaphores = &presentCompleteSemaphores[currentFrame];
 		submitInfo.waitSemaphoreCount = 1;
 		// Semaphore to be signaled when command buffers have completed
-		submitInfo.pSignalSemaphores = &renderCompleteSemaphores[currentSemaphore];
+		submitInfo.pSignalSemaphores = &renderCompleteSemaphores[imageIndex];
 		submitInfo.signalSemaphoreCount = 1;
 
 		// Submit to the graphics queue passing a wait fence
@@ -1027,7 +1029,7 @@ class VulkanExample : public VulkanExampleBase
 		VkPresentInfoKHR presentInfo{};
 		presentInfo.sType = VK_STRUCTURE_TYPE_PRESENT_INFO_KHR;
 		presentInfo.waitSemaphoreCount = 1;
-		presentInfo.pWaitSemaphores = &renderCompleteSemaphores[currentSemaphore];
+		presentInfo.pWaitSemaphores = &renderCompleteSemaphores[imageIndex];
 		presentInfo.swapchainCount = 1;
 		presentInfo.pSwapchains = &swapChain.swapChain;
 		presentInfo.pImageIndices = &imageIndex;
@@ -1042,8 +1044,6 @@ class VulkanExample : public VulkanExampleBase
 
 		// Select the next frame to render to, based on the max. no. of concurrent frames
 		currentFrame = (currentFrame + 1) % MAX_CONCURRENT_FRAMES;
-		// Similar for the semaphores, which need to be unique to the swapchain images
-		currentSemaphore = (currentSemaphore + 1) % swapChain.imageCount;
 	}
 };
 

examples/trianglevulkan13/trianglevulkan13.cpp

@@ -95,9 +95,8 @@ class VulkanExample : public VulkanExampleBase
 	VkCommandPool commandPool{ VK_NULL_HANDLE };
 	std::array<VkCommandBuffer, MAX_CONCURRENT_FRAMES> commandBuffers{};
 
-	// To select the correct sync and command objects, we need to keep track of the current frame and (swapchain) image index
+	// To select the correct sync and command objects, we need to keep track of the current frame
 	uint32_t currentFrame{ 0 };
-	uint32_t currentSemaphore{ 0 };
 
 	VkPhysicalDeviceVulkan13Features enabledFeatures{ VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_VULKAN_1_3_FEATURES };
 
@@ -182,16 +181,19 @@ class VulkanExample : public VulkanExampleBase
 			fenceCI.flags = VK_FENCE_CREATE_SIGNALED_BIT;
 			VK_CHECK_RESULT(vkCreateFence(device, &fenceCI, nullptr, &waitFences[i]));
 		}
-		// Semaphores are per swapchain image
-		presentCompleteSemaphores.resize(swapChain.images.size());
+		// Semaphores are used for correct command ordering within a queue
+		// Used to ensure that image presentation is complete before starting to submit again
+		presentCompleteSemaphores.resize(MAX_CONCURRENT_FRAMES);
+		for (auto& semaphore : presentCompleteSemaphores) {
+			VkSemaphoreCreateInfo semaphoreCI{ VK_STRUCTURE_TYPE_SEMAPHORE_CREATE_INFO };
+			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &semaphore));
+		}
+		// Render completion
+		// Semaphore used to ensure that all commands submitted have been finished before submitting the image to the queue
 		renderCompleteSemaphores.resize(swapChain.images.size());
-		for (size_t i = 0; i < swapChain.images.size(); i++) {
-			// Semaphores are used for correct command ordering within a queue
+		for (auto& semaphore : renderCompleteSemaphores) {
 			VkSemaphoreCreateInfo semaphoreCI{ VK_STRUCTURE_TYPE_SEMAPHORE_CREATE_INFO };
-			// Semaphore used to ensure that image presentation is complete before starting to submit again
-			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &presentCompleteSemaphores[i]));
-			// Semaphore used to ensure that all commands submitted have been finished before submitting the image to the queue
-			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &renderCompleteSemaphores[i]));
+			VK_CHECK_RESULT(vkCreateSemaphore(device, &semaphoreCI, nullptr, &semaphore));
 		}
 	}
 
@@ -701,7 +703,7 @@ class VulkanExample : public VulkanExampleBase
 		// Get the next swap chain image from the implementation
 		// Note that the implementation is free to return the images in any order, so we must use the acquire function and can't just cycle through the images/imageIndex on our own
 		uint32_t imageIndex{ 0 };
-		VkResult result = vkAcquireNextImageKHR(device, swapChain.swapChain, UINT64_MAX, presentCompleteSemaphores[currentSemaphore], VK_NULL_HANDLE, &imageIndex);
+		VkResult result = vkAcquireNextImageKHR(device, swapChain.swapChain, UINT64_MAX, presentCompleteSemaphores[currentFrame], VK_NULL_HANDLE, &imageIndex);
 		if (result == VK_ERROR_OUT_OF_DATE_KHR) {
 			windowResize();
 			return;
@@ -788,10 +790,10 @@ class VulkanExample : public VulkanExampleBase
 		submitInfo.commandBufferCount = 1;                  // We submit a single command buffer
 
 		// Semaphore to wait upon before the submitted command buffer starts executing
-		submitInfo.pWaitSemaphores = &presentCompleteSemaphores[currentSemaphore];
+		submitInfo.pWaitSemaphores = &presentCompleteSemaphores[currentFrame];
 		submitInfo.waitSemaphoreCount = 1;
 		// Semaphore to be signaled when command buffers have completed
-		submitInfo.pSignalSemaphores = &renderCompleteSemaphores[currentSemaphore];
+		submitInfo.pSignalSemaphores = &renderCompleteSemaphores[imageIndex];
 		submitInfo.signalSemaphoreCount = 1;
 
 		// Submit to the graphics queue passing a wait fence
@@ -802,7 +804,7 @@ class VulkanExample : public VulkanExampleBase
 		// This ensures that the image is not presented to the windowing system until all commands have been submitted
 		VkPresentInfoKHR presentInfo{ VK_STRUCTURE_TYPE_PRESENT_INFO_KHR };
 		presentInfo.waitSemaphoreCount = 1;
-		presentInfo.pWaitSemaphores = &renderCompleteSemaphores[currentSemaphore];
+		presentInfo.pWaitSemaphores = &renderCompleteSemaphores[imageIndex];
 		presentInfo.swapchainCount = 1;
 		presentInfo.pSwapchains = &swapChain.swapChain;
 		presentInfo.pImageIndices = &imageIndex;
@@ -815,8 +817,6 @@ class VulkanExample : public VulkanExampleBase
 
 		// Select the next frame to render to, based on the max. no. of concurrent frames
 		currentFrame = (currentFrame + 1) % MAX_CONCURRENT_FRAMES;
-		// Similar for the semaphores, which need to be unique to the swapchain images
-		currentSemaphore = (currentSemaphore + 1) % swapChain.imageCount;
 	}
 
 	// Override these as otherwise the base class would generate frame buffers and render passes

examples/validate_all.py

@@ -3,16 +3,21 @@
 
 # Runs all samples in benchmark mode and stores all validation messages into a single text file
 
-# Note: Need to be copied to where the binary files have been compiled
+# Note: Needs to be copied to where the binary files have been compiled (e.g. build/windows/bin/debug)
 
 import glob
 import subprocess
 import os
+import platform
 
 if os.path.exists("validation_output.txt"):
   os.remove("validation_output.txt")
-for sample in glob.glob("*.exe"):
-    # Skip headless samples, as they require manual keypress
+if platform.system() == 'Linux' or platform.system() == 'Darwin':
+    binaries = "./*"
+else:
+    binaries = "*.exe"  
+for sample in glob.glob(binaries):
+    # Skip headless samples, as they require a manual keypress
     if "headless" in sample:
        continue
     subprocess.call("%s -v -vl -b -bfs %s" % (sample, 50), shell=True)