From 64f98f286eaa485acabe40b25bf03b4866022798 Mon Sep 17 00:00:00 2001 From: baldurk Date: Thu, 20 Aug 2020 13:48:17 +0100 Subject: [PATCH] Add test of heavy descriptor reallocation and reuse --- renderdoc/api/replay/renderdoc_replay.h | 7 + renderdoc/replay/entry_points.cpp | 9 + util/test/demos/CMakeLists.txt | 1 + util/test/demos/demos.vcxproj | 1 + util/test/demos/demos.vcxproj.filters | 3 + util/test/demos/vk/vk_descriptor_reuse.cpp | 530 ++++++++++++++++++ util/test/demos/vk/vk_shader_debug_zoo.cpp | 9 +- util/test/tests/Vulkan/VK_Descriptor_Reuse.py | 48 ++ 8 files changed, 604 insertions(+), 4 deletions(-) create mode 100644 util/test/demos/vk/vk_descriptor_reuse.cpp create mode 100644 util/test/tests/Vulkan/VK_Descriptor_Reuse.py diff --git a/renderdoc/api/replay/renderdoc_replay.h b/renderdoc/api/replay/renderdoc_replay.h index 260622269..7f0a59ada 100644 --- a/renderdoc/api/replay/renderdoc_replay.h +++ b/renderdoc/api/replay/renderdoc_replay.h @@ -1976,6 +1976,13 @@ This will be in the form "MAJOR.MINOR" )"); extern "C" RENDERDOC_API const char *RENDERDOC_CC RENDERDOC_GetVersionString(); +DOCUMENT(R"(Determines if this is a release build of RenderDoc or not. + +:return: ``True`` if the replay is running on a release build. +:rtype: ``bool`` +)"); +extern "C" RENDERDOC_API bool RENDERDOC_CC RENDERDOC_IsReleaseBuild(); + DOCUMENT(R"(Retrieves the commit hash used to build. This will be in the form "0123456789abcdef0123456789abcdef01234567" diff --git a/renderdoc/replay/entry_points.cpp b/renderdoc/replay/entry_points.cpp index c885427b3..95708ed64 100644 --- a/renderdoc/replay/entry_points.cpp +++ b/renderdoc/replay/entry_points.cpp @@ -177,6 +177,15 @@ extern "C" RENDERDOC_API const char *RENDERDOC_CC RENDERDOC_GetVersionString() return MAJOR_MINOR_VERSION_STRING; } +extern "C" RENDERDOC_API bool RENDERDOC_CC RENDERDOC_IsReleaseBuild() +{ +#if ENABLED(RDOC_RELEASE) + return true; +#else + return false; +#endif +} + extern "C" RENDERDOC_API const char *RENDERDOC_CC RENDERDOC_GetCommitHash() { return GitVersionHash; diff --git a/util/test/demos/CMakeLists.txt b/util/test/demos/CMakeLists.txt index 90389cd6f..4cf6059c4 100644 --- a/util/test/demos/CMakeLists.txt +++ b/util/test/demos/CMakeLists.txt @@ -12,6 +12,7 @@ set(VULKAN_SRC vk/vk_cbuffer_zoo.cpp vk/vk_custom_border_color.cpp vk/vk_descriptor_index.cpp + vk/vk_descriptor_reuse.cpp vk/vk_discard_rects.cpp vk/vk_discard_zoo.cpp vk/vk_draw_zoo.cpp diff --git a/util/test/demos/demos.vcxproj b/util/test/demos/demos.vcxproj index 14bfba4a0..dadc3c08a 100644 --- a/util/test/demos/demos.vcxproj +++ b/util/test/demos/demos.vcxproj @@ -263,6 +263,7 @@ + diff --git a/util/test/demos/demos.vcxproj.filters b/util/test/demos/demos.vcxproj.filters index 9ae2cae90..3932bb7a5 100644 --- a/util/test/demos/demos.vcxproj.filters +++ b/util/test/demos/demos.vcxproj.filters @@ -538,6 +538,9 @@ D3D12\demos + + Vulkan\demos + diff --git a/util/test/demos/vk/vk_descriptor_reuse.cpp b/util/test/demos/vk/vk_descriptor_reuse.cpp new file mode 100644 index 000000000..d6d7d0908 --- /dev/null +++ b/util/test/demos/vk/vk_descriptor_reuse.cpp @@ -0,0 +1,530 @@ +/****************************************************************************** + * The MIT License (MIT) + * + * Copyright (c) 2019-2020 Baldur Karlsson + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to deal + * in the Software without restriction, including without limitation the rights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + * THE SOFTWARE. + ******************************************************************************/ + +#include +#include +#include +#include +#include "3rdparty/fmt/core.h" +#include "vk_test.h" + +RD_TEST(VK_Descriptor_Reuse, VulkanGraphicsTest) +{ + static constexpr const char *Description = + "Allocates and reuses a large number of descriptors to stress re-allocation."; + + std::string pixel = R"EOSHADER( + +#version 460 core + +#define v2f v2f_block \ +{ \ + vec4 pos; \ + vec4 col; \ + vec4 uv; \ +} + +layout(location = 0) in v2f vertIn; + +layout(location = 0, index = 0) out vec4 Color; + +layout(set = 0, binding = 10, std140) uniform constsbuf1 +{ + vec4 val1; +} cbuf1; + +layout(set = 0, binding = 11, std140) uniform constsbuf2 +{ + vec4 val2; +} cbuf2; + +layout(set = 0, binding = 3) uniform sampler2D samp1; +layout(set = 0, binding = 4) uniform sampler2D samp2; +layout(set = 0, binding = 5) uniform sampler2D samp3; + +void main() +{ + Color = (vertIn.col * 0.4f) + + cbuf1.val1 + cbuf2.val2 + + texture(samp1, vec2(0)) + texture(samp2, vec2(0)) + texture(samp3, vec2(0)); +} + +)EOSHADER"; + + int main() + { + // initialise, create window, create context, etc + if(!Init()) + return 3; + + const size_t numBufs = 1024; + const size_t numImages = 1024; + const size_t descriptorCount = 512; + const size_t setLayoutCount = 64; + const size_t ringSize = 3; + const size_t threadCount = 8; + + std::vector setlayout; + + for(size_t i = 0; i < setLayoutCount; i++) + setlayout.push_back(createDescriptorSetLayout(vkh::DescriptorSetLayoutCreateInfo({ + {10, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, 1, VK_SHADER_STAGE_FRAGMENT_BIT}, + {11, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, 1, VK_SHADER_STAGE_FRAGMENT_BIT}, + {3, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, 1, VK_SHADER_STAGE_FRAGMENT_BIT}, + {4, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, 1, VK_SHADER_STAGE_FRAGMENT_BIT}, + {5, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, 1, VK_SHADER_STAGE_FRAGMENT_BIT}, + }))); + + VkPipelineLayout layout = createPipelineLayout(vkh::PipelineLayoutCreateInfo({setlayout[0]})); + + AllocatedImage img[threadCount]; + + for(size_t i = 0; i < threadCount; i++) + { + img[i] = AllocatedImage( + this, + vkh::ImageCreateInfo(mainWindow->scissor.extent.width, mainWindow->scissor.extent.height, + 0, VK_FORMAT_R32G32B32A32_SFLOAT, + VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_GPU_ONLY})); + setName(img[i].image, fmt::format("Offscreen{}", i)); + } + + VkImageView imgview[threadCount]; + for(size_t i = 0; i < threadCount; i++) + imgview[i] = createImageView(vkh::ImageViewCreateInfo(img[i].image, VK_IMAGE_VIEW_TYPE_2D, + VK_FORMAT_R32G32B32A32_SFLOAT)); + + vkh::RenderPassCreator renderPassCreateInfo; + + renderPassCreateInfo.attachments.push_back( + vkh::AttachmentDescription(VK_FORMAT_R32G32B32A32_SFLOAT, VK_IMAGE_LAYOUT_UNDEFINED, + VK_IMAGE_LAYOUT_GENERAL, VK_ATTACHMENT_LOAD_OP_CLEAR)); + + renderPassCreateInfo.addSubpass({VkAttachmentReference({0, VK_IMAGE_LAYOUT_GENERAL})}); + + VkRenderPass renderPass = createRenderPass(renderPassCreateInfo); + + VkFramebuffer framebuffer[threadCount]; + for(size_t i = 0; i < threadCount; i++) + framebuffer[i] = createFramebuffer( + vkh::FramebufferCreateInfo(renderPass, {imgview[i]}, mainWindow->scissor.extent)); + + vkh::GraphicsPipelineCreateInfo pipeCreateInfo; + + pipeCreateInfo.layout = layout; + pipeCreateInfo.renderPass = renderPass; + + pipeCreateInfo.vertexInputState.vertexBindingDescriptions = {vkh::vertexBind(0, DefaultA2V)}; + pipeCreateInfo.vertexInputState.vertexAttributeDescriptions = { + vkh::vertexAttr(0, 0, DefaultA2V, pos), vkh::vertexAttr(1, 0, DefaultA2V, col), + vkh::vertexAttr(2, 0, DefaultA2V, uv), + }; + + pipeCreateInfo.stages = { + CompileShaderModule(VKDefaultVertex, ShaderLang::glsl, ShaderStage::vert, "main"), + CompileShaderModule(pixel, ShaderLang::glsl, ShaderStage::frag, "main"), + }; + + VkPipeline pipe = createGraphicsPipeline(pipeCreateInfo); + + AllocatedBuffer vb( + this, vkh::BufferCreateInfo(sizeof(DefaultTri), VK_BUFFER_USAGE_VERTEX_BUFFER_BIT | + VK_BUFFER_USAGE_TRANSFER_DST_BIT), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_CPU_TO_GPU})); + + vb.upload(DefaultTri); + + Vec4f val1(0.4f, 0.0f, 0.0f, 0.0f); + Vec4f val2(0.0f, 0.0f, 0.4f, 0.0f); + + std::vector val1bufs; + std::vector val2bufs; + + for(size_t i = 0; i < numBufs; i++) + { + val1bufs.push_back(AllocatedBuffer( + this, vkh::BufferCreateInfo(sizeof(Vec4f), VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT | + VK_BUFFER_USAGE_TRANSFER_DST_BIT), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_CPU_TO_GPU}))); + val2bufs.push_back(AllocatedBuffer( + this, vkh::BufferCreateInfo(sizeof(Vec4f), VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT | + VK_BUFFER_USAGE_TRANSFER_DST_BIT), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_CPU_TO_GPU}))); + + val1bufs.back().upload(&val1, sizeof(Vec4f)); + val2bufs.back().upload(&val2, sizeof(Vec4f)); + } + + std::vector samps1; + std::vector samps2; + std::vector samps3; + + std::vector views1; + std::vector views2; + std::vector views3; + + { + VkCommandBuffer cmd = GetCommandBuffer(); + + vkBeginCommandBuffer(cmd, vkh::CommandBufferBeginInfo()); + + for(size_t i = 0; i < numImages; i++) + { + samps1.push_back(AllocatedImage( + this, + vkh::ImageCreateInfo(16, 16, 0, VK_FORMAT_R32G32B32A32_SFLOAT, + VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_SAMPLED_BIT, 3), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_GPU_ONLY}))); + samps2.push_back(AllocatedImage( + this, + vkh::ImageCreateInfo(16, 16, 0, VK_FORMAT_R32G32B32A32_SFLOAT, + VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_SAMPLED_BIT, 3), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_GPU_ONLY}))); + samps3.push_back(AllocatedImage( + this, + vkh::ImageCreateInfo(16, 16, 0, VK_FORMAT_R32G32B32A32_SFLOAT, + VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_SAMPLED_BIT, 3), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_GPU_ONLY}))); + + views1.push_back(createImageView(vkh::ImageViewCreateInfo( + samps1.back().image, VK_IMAGE_VIEW_TYPE_2D, VK_FORMAT_R32G32B32A32_SFLOAT, {}, + vkh::ImageSubresourceRange(VK_IMAGE_ASPECT_COLOR_BIT, rand() % 2)))); + views2.push_back(createImageView(vkh::ImageViewCreateInfo( + samps2.back().image, VK_IMAGE_VIEW_TYPE_2D, VK_FORMAT_R32G32B32A32_SFLOAT, {}, + vkh::ImageSubresourceRange(VK_IMAGE_ASPECT_COLOR_BIT, rand() % 2)))); + views3.push_back(createImageView(vkh::ImageViewCreateInfo( + samps3.back().image, VK_IMAGE_VIEW_TYPE_2D, VK_FORMAT_R32G32B32A32_SFLOAT, {}, + vkh::ImageSubresourceRange(VK_IMAGE_ASPECT_COLOR_BIT, rand() % 2)))); + + vkh::cmdPipelineBarrier( + cmd, + { + vkh::ImageMemoryBarrier(0, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_UNDEFINED, + VK_IMAGE_LAYOUT_GENERAL, samps1.back().image), + vkh::ImageMemoryBarrier(0, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_UNDEFINED, + VK_IMAGE_LAYOUT_GENERAL, samps2.back().image), + vkh::ImageMemoryBarrier(0, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_UNDEFINED, + VK_IMAGE_LAYOUT_GENERAL, samps3.back().image), + }); + + vkCmdClearColorImage(cmd, samps1.back().image, VK_IMAGE_LAYOUT_GENERAL, + vkh::ClearColorValue(0.4f, 0.0f, 0.0f, 0.0f), 1, + vkh::ImageSubresourceRange()); + vkCmdClearColorImage(cmd, samps2.back().image, VK_IMAGE_LAYOUT_GENERAL, + vkh::ClearColorValue(0.0f, 0.4f, 0.0f, 0.0f), 1, + vkh::ImageSubresourceRange()); + vkCmdClearColorImage(cmd, samps3.back().image, VK_IMAGE_LAYOUT_GENERAL, + vkh::ClearColorValue(0.0f, 0.0f, 0.4f, 0.0f), 1, + vkh::ImageSubresourceRange()); + } + + vkEndCommandBuffer(cmd); + + Submit(99, 99, {cmd}); + } + + VkSampler sampler = createSampler(vkh::SamplerCreateInfo(VK_FILTER_LINEAR)); + + VkFence ringComplete[ringSize]; + for(size_t r = 0; r < ringSize; r++) + CHECK_VKR(vkCreateFence(device, vkh::FenceCreateInfo(VK_FENCE_CREATE_SIGNALED_BIT), NULL, + &ringComplete[r])); + + struct ThreadData + { + VkCommandPool cmdPool; + VkDescriptorPool descPools[ringSize]; + VkCommandBuffer cmdBufs[ringSize]; + + std::mutex lock; + std::condition_variable cv; + std::atomic_bool kill = false, run = false; + }; + + ThreadData threadData[threadCount]; + std::atomic_int threadsDone = 0; + std::mutex doneLock; + std::condition_variable doneCV; + + for(size_t t = 0; t < threadCount; t++) + { + CHECK_VKR(vkCreateCommandPool( + device, vkh::CommandPoolCreateInfo(VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT), NULL, + &threadData[t].cmdPool)); + + for(size_t r = 0; r < ringSize; r++) + { + CHECK_VKR(vkCreateDescriptorPool( + device, vkh::DescriptorPoolCreateInfo( + descriptorCount, + { + {VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, descriptorCount * 3}, + {VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, descriptorCount * 2}, + }), + NULL, &threadData[t].descPools[r])); + } + + CHECK_VKR(vkAllocateCommandBuffers( + device, vkh::CommandBufferAllocateInfo(threadData[t].cmdPool, ringSize), + threadData[t].cmdBufs)); + } + + size_t ringIndex = 0; + + std::vector threads(threadCount); + + auto threadFunc = [&](size_t threadIndex) { + // pre-bake descriptor allocate/update infos, that we just patch and use. Saves on overhead of + // temporary std::vector work that is usually worth it for convenience + + VkDescriptorBufferInfo bufs[2] = { + vkh::DescriptorBufferInfo(VK_NULL_HANDLE), vkh::DescriptorBufferInfo(VK_NULL_HANDLE), + }; + + VkDescriptorImageInfo imInfo[3] = { + vkh::DescriptorImageInfo(VK_NULL_HANDLE, VK_IMAGE_LAYOUT_GENERAL, sampler), + vkh::DescriptorImageInfo(VK_NULL_HANDLE, VK_IMAGE_LAYOUT_GENERAL, sampler), + vkh::DescriptorImageInfo(VK_NULL_HANDLE, VK_IMAGE_LAYOUT_GENERAL, sampler), + }; + + VkWriteDescriptorSet writes[5] = { + vkh::WriteDescriptorSet(VK_NULL_HANDLE, 10, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, {bufs[0]}), + vkh::WriteDescriptorSet(VK_NULL_HANDLE, 11, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, {bufs[1]}), + vkh::WriteDescriptorSet(VK_NULL_HANDLE, 3, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, + {imInfo[0]}), + vkh::WriteDescriptorSet(VK_NULL_HANDLE, 4, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, + {imInfo[1]}), + vkh::WriteDescriptorSet(VK_NULL_HANDLE, 5, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, + {imInfo[2]}), + }; + + writes[0].pBufferInfo = &bufs[0]; + writes[1].pBufferInfo = &bufs[1]; + writes[2].pImageInfo = &imInfo[0]; + writes[3].pImageInfo = &imInfo[1]; + writes[4].pImageInfo = &imInfo[2]; + + VkDescriptorSetAllocateInfo info = + vkh::DescriptorSetAllocateInfo(VK_NULL_HANDLE, {setlayout[0]}); + + while(!threadData[threadIndex].kill) + { + { + std::unique_lock scoped(threadData[threadIndex].lock); + while(!threadData[threadIndex].kill && !threadData[threadIndex].run) + threadData[threadIndex].cv.wait(scoped); + threadData[threadIndex].run = false; + if(threadData[threadIndex].kill) + break; + } + + VkCommandBuffer cmd = threadData[threadIndex].cmdBufs[ringIndex]; + VkDescriptorPool descPool = threadData[threadIndex].descPools[ringIndex]; + + info.descriptorPool = descPool; + + vkResetDescriptorPool(device, descPool, 0); + vkResetCommandBuffer(cmd, 0); + + vkBeginCommandBuffer(cmd, vkh::CommandBufferBeginInfo()); + + vkCmdBeginRenderPass( + cmd, vkh::RenderPassBeginInfo(renderPass, framebuffer[threadIndex], mainWindow->scissor, + {vkh::ClearValue(0.0f, 0.0f, 0.0f, 1.0f)}), + VK_SUBPASS_CONTENTS_INLINE); + + vkCmdBindPipeline(cmd, VK_PIPELINE_BIND_POINT_GRAPHICS, pipe); + vkh::cmdBindVertexBuffers(cmd, 0, {vb.buffer}, {0}); + + VkRect2D s = {{0, 0}, + {uint32_t(screenWidth / (int)sqrt(descriptorCount)), + uint32_t(screenHeight / (int)sqrt(descriptorCount))}}; + VkViewport v = {0, 0, (float)s.extent.width, (float)s.extent.height, 0, 1}; + + size_t randSeed = curFrame * threadIndex + threadIndex; + + for(size_t i = 0; i < descriptorCount; i++) + { + randSeed = (1103515245 * randSeed + 12345) % 0x7fffffff; + info.pSetLayouts = &setlayout[randSeed % setlayout.size()]; + + VkDescriptorSet set; + CHECK_VKR(vkAllocateDescriptorSets(device, &info, &set)); + + for(VkWriteDescriptorSet &write : writes) + write.dstSet = set; + + randSeed = (1103515245 * randSeed + 12345) % 0x7fffffff; + bufs[0].buffer = val1bufs[randSeed % val1bufs.size()].buffer; + randSeed = (1103515245 * randSeed + 12345) % 0x7fffffff; + bufs[1].buffer = val2bufs[randSeed % val2bufs.size()].buffer; + + randSeed = (1103515245 * randSeed + 12345) % 0x7fffffff; + imInfo[0].imageView = views1[randSeed % views1.size()]; + randSeed = (1103515245 * randSeed + 12345) % 0x7fffffff; + imInfo[1].imageView = views2[randSeed % views2.size()]; + randSeed = (1103515245 * randSeed + 12345) % 0x7fffffff; + imInfo[2].imageView = views3[randSeed % views3.size()]; + + vkUpdateDescriptorSets(device, (uint32_t)ARRAY_COUNT(writes), writes, 0, NULL); + + vkCmdBindDescriptorSets(cmd, VK_PIPELINE_BIND_POINT_GRAPHICS, layout, 0, 1, &set, 0, NULL); + vkCmdSetViewport(cmd, 0, 1, &v); + vkCmdSetScissor(cmd, 0, 1, &s); + vkCmdDraw(cmd, 3, 1, 0, 0); + + v.x += v.width; + s.offset.x += s.extent.width; + if(v.x >= screenWidth) + { + v.x = 0; + s.offset.x = 0; + v.y += v.height; + s.offset.y += s.extent.height; + } + } + + vkCmdEndRenderPass(cmd); + + vkEndCommandBuffer(cmd); + + { + std::unique_lock scoped(doneLock); + threadsDone++; + doneCV.notify_one(); + } + } + }; + + for(size_t i = 0; i < threads.size(); i++) + threads[i] = std::thread(threadFunc, i); + + typedef std::chrono::high_resolution_clock Clock; + typedef std::chrono::time_point Time; + + Time prev = Clock::now(); + + double totalMS = 0; + uint32_t frames = 0; + + double minframetimeMS = 100000.0; + + while(Running()) + { + // wait for the previous version of this ring to complete. We expect this to be done. + vkWaitForFences(device, 1, &ringComplete[ringIndex], VK_TRUE, 1000000); + + // reset it so we can use it in the next submit + vkResetFences(device, 1, &ringComplete[ringIndex]); + + for(size_t i = 0; i < threads.size(); i++) + { + std::unique_lock scoped(threadData[i].lock); + threadData[i].run = true; + threadData[i].cv.notify_one(); + } + + { + std::unique_lock scoped(doneLock); + while(threadsDone < threadCount) + doneCV.wait(scoped); + threadsDone = 0; + } + + VkCommandBuffer cmd = GetCommandBuffer(); + + vkBeginCommandBuffer(cmd, vkh::CommandBufferBeginInfo()); + + VkImage swapimg = + StartUsingBackbuffer(cmd, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_GENERAL); + + vkCmdClearColorImage(cmd, swapimg, VK_IMAGE_LAYOUT_GENERAL, + vkh::ClearColorValue(0.2f, 0.2f, 0.2f, 1.0f), 1, + vkh::ImageSubresourceRange()); + + FinishUsingBackbuffer(cmd, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_GENERAL); + + vkEndCommandBuffer(cmd); + + std::vector cmds; + for(size_t i = 0; i < threadCount; i++) + cmds.push_back(threadData[i].cmdBufs[ringIndex]); + + VkSubmitInfo submit = vkh::SubmitInfo(cmds); + CHECK_VKR(vkQueueSubmit(queue, 1, &submit, ringComplete[ringIndex])); + Submit(0, 1, {cmd}); + + ringIndex = (ringIndex + 1) % ringSize; + + Time cur = Clock::now(); + double frametimeMS = + double(std::chrono::duration_cast(cur - prev).count()) / 1000.0; + prev = cur; + + if(curFrame > 1) + minframetimeMS = std::min(minframetimeMS, frametimeMS); + + setMarker(queue, fmt::format("Min Duration = {}", minframetimeMS)); + + totalMS += frametimeMS; + frames++; + + if(totalMS > 1000.0) + { + TEST_LOG("%u frames in %f ms = %f average frametime", frames, totalMS, totalMS / frames); + frames = 0; + totalMS = 0.0; + } + + Present(); + } + + for(size_t i = 0; i < threads.size(); i++) + { + std::unique_lock scoped(threadData[i].lock); + + threadData[i].kill = true; + threadData[i].cv.notify_one(); + } + + for(size_t i = 0; i < threads.size(); i++) + threads[i].join(); + + for(size_t r = 0; r < ringSize; r++) + vkDestroyFence(device, ringComplete[r], NULL); + + for(size_t t = 0; t < threadCount; t++) + { + vkDestroyCommandPool(device, threadData[t].cmdPool, NULL); + + for(size_t r = 0; r < ringSize; r++) + vkDestroyDescriptorPool(device, threadData[t].descPools[r], NULL); + } + + return 0; + } +}; + +REGISTER_TEST(); diff --git a/util/test/demos/vk/vk_shader_debug_zoo.cpp b/util/test/demos/vk/vk_shader_debug_zoo.cpp index 3336ff174..f54e77760 100644 --- a/util/test/demos/vk/vk_shader_debug_zoo.cpp +++ b/util/test/demos/vk/vk_shader_debug_zoo.cpp @@ -3415,8 +3415,8 @@ OpMemberDecorate %cbuffer_struct 17 Offset 216 ; double doublePackSource // this set layout has arrays of each type. We'll uniformly, dynamic-uniformly, and // non-uniformly access each of these - VkDescriptorSetLayout setlayout1; - VkDescriptorSetLayout setlayout2; + VkDescriptorSetLayout setlayout1 = VK_NULL_HANDLE; + VkDescriptorSetLayout setlayout2 = VK_NULL_HANDLE; if(descIndexing) { @@ -3650,6 +3650,7 @@ OpMemberDecorate %cbuffer_struct 17 Offset 216 ; double doublePackSource float f[4]; int i[4]; } rnd; + memset(&rnd, 0, sizeof(rnd)); for(size_t x = 0; x < 16; x++) { @@ -3758,8 +3759,8 @@ OpMemberDecorate %cbuffer_struct 17 Offset 216 ; double doublePackSource VK_BORDER_COLOR_FLOAT_TRANSPARENT_BLACK, 0.0f, 0.0f, 0.0f, VK_COMPARE_OP_LESS_OR_EQUAL)); VkDescriptorSet descset0 = allocateDescriptorSet(setlayout0); - VkDescriptorSet descset1; - VkDescriptorSet descset2; + VkDescriptorSet descset1 = VK_NULL_HANDLE; + VkDescriptorSet descset2 = VK_NULL_HANDLE; if(descIndexing) { diff --git a/util/test/tests/Vulkan/VK_Descriptor_Reuse.py b/util/test/tests/Vulkan/VK_Descriptor_Reuse.py new file mode 100644 index 000000000..61d773111 --- /dev/null +++ b/util/test/tests/Vulkan/VK_Descriptor_Reuse.py @@ -0,0 +1,48 @@ +import renderdoc as rd +import struct +import rdtest + + +class VK_Descriptor_Reuse(rdtest.TestCase): + demos_test_name = 'VK_Descriptor_Reuse' + demos_frame_cap = 100 + + def check_capture(self): + last_draw: rd.DrawcallDescription = self.get_last_draw() + + self.controller.SetFrameEvent(last_draw.eventId, True) + + draw: rd.DrawcallDescription = self.find_draw('Duration') + + min_duration = float(draw.name.split(' = ')[1]) + + if rd.IsReleaseBuild(): + if min_duration >= 15.0: + raise rdtest.TestFailureException("Minimum duration noted {} ms is too high".format(min_duration)) + rdtest.log.success("Minimum duration ({}) is OK".format(min_duration)) + else: + rdtest.log.print("Not checking duration ({}) in non-release build".format(min_duration)) + + resources = self.controller.GetResources() + for i in range(8): + res: rd.ResourceDescription = [r for r in resources if r.name == 'Offscreen{}'.format(i)][0] + tex: rd.TextureDescription = self.get_texture(res.resourceId) + + data = self.controller.GetTextureData(res.resourceId, rd.Subresource(0, 0, 0)) + + pixels = [struct.unpack_from("4f", data, 16 * p) for p in range(tex.width * tex.height)] + + unique_pixels = list(set(pixels)) + + if len(unique_pixels) > 2: + raise rdtest.TestFailureException("Too many pixel values found ({})".format(len(unique_pixels))) + + if (0.0, 0.0, 0.0, 1.0) not in unique_pixels: + raise rdtest.TestFailureException("Didn't find background colour in unique pixels list") + + unique_pixels.remove((0.0, 0.0, 0.0, 1.0)) + + if not rdtest.value_compare((0.8, 0.8, 0.8, 0.4), unique_pixels[0]): + raise rdtest.TestFailureException("Didn't find foreground colour in unique pixels list") + + rdtest.log.success("{} has correct contents".format(res.name))