From a665581684b612afb225485d2e93f07b2274eb16 Mon Sep 17 00:00:00 2001 From: ReinUsesLisp Date: Mon, 20 Jan 2020 18:29:54 -0300 Subject: [PATCH] vk_blit_screen: Address feedback --- .../renderer_vulkan/vk_blit_screen.cpp | 28 ++++++++++--------- .../renderer_vulkan/vk_blit_screen.h | 7 +++-- .../renderer_vulkan/vk_swapchain.cpp | 2 +- src/video_core/renderer_vulkan/vk_swapchain.h | 10 +++---- 4 files changed, 25 insertions(+), 22 deletions(-) diff --git a/src/video_core/renderer_vulkan/vk_blit_screen.cpp b/src/video_core/renderer_vulkan/vk_blit_screen.cpp index 7c8bff5d23..855cfc8835 100644 --- a/src/video_core/renderer_vulkan/vk_blit_screen.cpp +++ b/src/video_core/renderer_vulkan/vk_blit_screen.cpp @@ -226,7 +226,7 @@ std::tuple VKBlitScreen::Draw(const Tegra::FramebufferC // Finish any pending renderpass scheduler.RequestOutsideRenderPassOperationContext(); - const u32 image_index = swapchain.GetImageIndex(); + const std::size_t image_index = swapchain.GetImageIndex(); watches[image_index]->Watch(scheduler.GetFence()); VKImage* blit_image = use_accelerated ? screen_info.image : raw_images[image_index].get(); @@ -345,10 +345,11 @@ void VKBlitScreen::CreateSemaphores() { void VKBlitScreen::CreateDescriptorPool() { const std::array pool_sizes{ - vk::DescriptorPoolSize{vk::DescriptorType::eUniformBuffer, image_count}, - vk::DescriptorPoolSize{vk::DescriptorType::eCombinedImageSampler, image_count}}; - const vk::DescriptorPoolCreateInfo pool_ci({}, image_count, static_cast(pool_sizes.size()), - pool_sizes.data()); + vk::DescriptorPoolSize{vk::DescriptorType::eUniformBuffer, static_cast(image_count)}, + vk::DescriptorPoolSize{vk::DescriptorType::eCombinedImageSampler, + static_cast(image_count)}}; + const vk::DescriptorPoolCreateInfo pool_ci( + {}, static_cast(image_count), static_cast(pool_sizes.size()), pool_sizes.data()); const auto dev = device.GetLogical(); descriptor_pool = dev.createDescriptorPoolUnique(pool_ci, nullptr, device.GetDispatchLoader()); } @@ -397,7 +398,7 @@ void VKBlitScreen::CreateDescriptorSets() { const auto& dld = device.GetDispatchLoader(); descriptor_sets.resize(image_count); - for (u32 i = 0; i < image_count; ++i) { + for (std::size_t i = 0; i < image_count; ++i) { const vk::DescriptorSetLayout layout = *descriptor_set_layout; const vk::DescriptorSetAllocateInfo descriptor_set_ai(*descriptor_pool, 1, &layout); const vk::Result result = @@ -487,7 +488,7 @@ void VKBlitScreen::CreateFramebuffers() { const auto dev = device.GetLogical(); const auto& dld = device.GetDispatchLoader(); - for (u32 i = 0; i < image_count; ++i) { + for (std::size_t i = 0; i < image_count; ++i) { const vk::ImageView image_view{swapchain.GetImageViewIndex(i)}; const vk::FramebufferCreateInfo framebuffer_ci({}, *renderpass, 1, &image_view, size.width, size.height, 1); @@ -496,7 +497,7 @@ void VKBlitScreen::CreateFramebuffers() { } void VKBlitScreen::ReleaseRawImages() { - for (u32 i = 0; i < static_cast(raw_images.size()); ++i) { + for (std::size_t i = 0; i < raw_images.size(); ++i) { watches[i]->Wait(); } raw_images.clear(); @@ -523,7 +524,7 @@ void VKBlitScreen::CreateRawImages(const Tegra::FramebufferConfig& framebuffer) raw_buffer_commits.resize(image_count); const auto format = GetFormat(framebuffer); - for (u32 i = 0; i < image_count; ++i) { + for (std::size_t i = 0; i < image_count; ++i) { const vk::ImageCreateInfo image_ci( {}, vk::ImageType::e2D, format, {framebuffer.width, framebuffer.height, 1}, 1, 1, vk::SampleCountFlagBits::e1, vk::ImageTiling::eOptimal, @@ -536,7 +537,7 @@ void VKBlitScreen::CreateRawImages(const Tegra::FramebufferConfig& framebuffer) } } -void VKBlitScreen::UpdateDescriptorSet(u32 image_index, vk::ImageView image_view) const { +void VKBlitScreen::UpdateDescriptorSet(std::size_t image_index, vk::ImageView image_view) const { const vk::DescriptorSet descriptor_set = descriptor_sets[image_index]; const vk::DescriptorBufferInfo buffer_info(*buffer, offsetof(BufferData, uniform), @@ -568,7 +569,7 @@ void VKBlitScreen::SetVertexData(BufferData& data, const auto& framebuffer_transform_flags = framebuffer.transform_flags; const auto& framebuffer_crop_rect = framebuffer.crop_rect; - const Common::Rectangle texcoords{0.f, 0.f, 1.f, 1.f}; + static constexpr Common::Rectangle texcoords{0.f, 0.f, 1.f, 1.f}; auto left = texcoords.left; auto right = texcoords.right; @@ -591,7 +592,8 @@ void VKBlitScreen::SetVertexData(BufferData& data, // Scale the output by the crop width/height. This is commonly used with 1280x720 rendering // (e.g. handheld mode) on a 1920x1080 framebuffer. - f32 scale_u = 1.0f, scale_v = 1.0f; + f32 scale_u = 1.0f; + f32 scale_v = 1.0f; if (framebuffer_crop_rect.GetWidth() > 0) { scale_u = static_cast(framebuffer_crop_rect.GetWidth()) / static_cast(screen_info.width); @@ -617,7 +619,7 @@ u64 VKBlitScreen::CalculateBufferSize(const Tegra::FramebufferConfig& framebuffe } u64 VKBlitScreen::GetRawImageOffset(const Tegra::FramebufferConfig& framebuffer, - u32 image_index) const { + std::size_t image_index) const { constexpr auto first_image_offset = static_cast(sizeof(BufferData)); return first_image_offset + GetSizeInBytes(framebuffer) * image_index; } diff --git a/src/video_core/renderer_vulkan/vk_blit_screen.h b/src/video_core/renderer_vulkan/vk_blit_screen.h index 743bb0317f..ea680b3f58 100644 --- a/src/video_core/renderer_vulkan/vk_blit_screen.h +++ b/src/video_core/renderer_vulkan/vk_blit_screen.h @@ -74,12 +74,13 @@ private: void CreateStagingBuffer(const Tegra::FramebufferConfig& framebuffer); void CreateRawImages(const Tegra::FramebufferConfig& framebuffer); - void UpdateDescriptorSet(u32 image_index, vk::ImageView image_view) const; + void UpdateDescriptorSet(std::size_t image_index, vk::ImageView image_view) const; void SetUniformData(BufferData& data, const Tegra::FramebufferConfig& framebuffer) const; void SetVertexData(BufferData& data, const Tegra::FramebufferConfig& framebuffer) const; u64 CalculateBufferSize(const Tegra::FramebufferConfig& framebuffer) const; - u64 GetRawImageOffset(const Tegra::FramebufferConfig& framebuffer, u32 image_index) const; + u64 GetRawImageOffset(const Tegra::FramebufferConfig& framebuffer, + std::size_t image_index) const; Core::System& system; Core::Frontend::EmuWindow& render_window; @@ -89,7 +90,7 @@ private: VKMemoryManager& memory_manager; VKSwapchain& swapchain; VKScheduler& scheduler; - const u32 image_count; + const std::size_t image_count; const VKScreenInfo& screen_info; UniqueShaderModule vertex_shader; diff --git a/src/video_core/renderer_vulkan/vk_swapchain.cpp b/src/video_core/renderer_vulkan/vk_swapchain.cpp index ebc68f0305..f47b691a8d 100644 --- a/src/video_core/renderer_vulkan/vk_swapchain.cpp +++ b/src/video_core/renderer_vulkan/vk_swapchain.cpp @@ -123,7 +123,7 @@ bool VKSwapchain::Present(vk::Semaphore render_semaphore, VKFence& fence) { ASSERT(fences[image_index] == nullptr); fences[image_index] = &fence; - frame_index = (frame_index + 1) % image_count; + frame_index = (frame_index + 1) % static_cast(image_count); return recreated; } diff --git a/src/video_core/renderer_vulkan/vk_swapchain.h b/src/video_core/renderer_vulkan/vk_swapchain.h index a1e7938d2f..2f3b2ccd55 100644 --- a/src/video_core/renderer_vulkan/vk_swapchain.h +++ b/src/video_core/renderer_vulkan/vk_swapchain.h @@ -40,19 +40,19 @@ public: return extent; } - u32 GetImageCount() const { + std::size_t GetImageCount() const { return image_count; } - u32 GetImageIndex() const { + std::size_t GetImageIndex() const { return image_index; } - vk::Image GetImageIndex(u32 index) const { + vk::Image GetImageIndex(std::size_t index) const { return images[index]; } - vk::ImageView GetImageViewIndex(u32 index) const { + vk::ImageView GetImageViewIndex(std::size_t index) const { return *image_views[index]; } @@ -77,7 +77,7 @@ private: UniqueSwapchainKHR swapchain; - u32 image_count{}; + std::size_t image_count{}; std::vector images; std::vector image_views; std::vector framebuffers;