mirror of
https://github.com/yuzu-mirror/yuzu.git
synced 2024-09-08 06:09:08 +01:00
vk_blit_screen: Address feedback
This commit is contained in:
parent
f5dfe68a94
commit
a665581684
4 changed files with 25 additions and 22 deletions
|
@ -226,7 +226,7 @@ std::tuple<VKFence&, vk::Semaphore> VKBlitScreen::Draw(const Tegra::FramebufferC
|
||||||
// Finish any pending renderpass
|
// Finish any pending renderpass
|
||||||
scheduler.RequestOutsideRenderPassOperationContext();
|
scheduler.RequestOutsideRenderPassOperationContext();
|
||||||
|
|
||||||
const u32 image_index = swapchain.GetImageIndex();
|
const std::size_t image_index = swapchain.GetImageIndex();
|
||||||
watches[image_index]->Watch(scheduler.GetFence());
|
watches[image_index]->Watch(scheduler.GetFence());
|
||||||
|
|
||||||
VKImage* blit_image = use_accelerated ? screen_info.image : raw_images[image_index].get();
|
VKImage* blit_image = use_accelerated ? screen_info.image : raw_images[image_index].get();
|
||||||
|
@ -345,10 +345,11 @@ void VKBlitScreen::CreateSemaphores() {
|
||||||
|
|
||||||
void VKBlitScreen::CreateDescriptorPool() {
|
void VKBlitScreen::CreateDescriptorPool() {
|
||||||
const std::array<vk::DescriptorPoolSize, 2> pool_sizes{
|
const std::array<vk::DescriptorPoolSize, 2> pool_sizes{
|
||||||
vk::DescriptorPoolSize{vk::DescriptorType::eUniformBuffer, image_count},
|
vk::DescriptorPoolSize{vk::DescriptorType::eUniformBuffer, static_cast<u32>(image_count)},
|
||||||
vk::DescriptorPoolSize{vk::DescriptorType::eCombinedImageSampler, image_count}};
|
vk::DescriptorPoolSize{vk::DescriptorType::eCombinedImageSampler,
|
||||||
const vk::DescriptorPoolCreateInfo pool_ci({}, image_count, static_cast<u32>(pool_sizes.size()),
|
static_cast<u32>(image_count)}};
|
||||||
pool_sizes.data());
|
const vk::DescriptorPoolCreateInfo pool_ci(
|
||||||
|
{}, static_cast<u32>(image_count), static_cast<u32>(pool_sizes.size()), pool_sizes.data());
|
||||||
const auto dev = device.GetLogical();
|
const auto dev = device.GetLogical();
|
||||||
descriptor_pool = dev.createDescriptorPoolUnique(pool_ci, nullptr, device.GetDispatchLoader());
|
descriptor_pool = dev.createDescriptorPoolUnique(pool_ci, nullptr, device.GetDispatchLoader());
|
||||||
}
|
}
|
||||||
|
@ -397,7 +398,7 @@ void VKBlitScreen::CreateDescriptorSets() {
|
||||||
const auto& dld = device.GetDispatchLoader();
|
const auto& dld = device.GetDispatchLoader();
|
||||||
|
|
||||||
descriptor_sets.resize(image_count);
|
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::DescriptorSetLayout layout = *descriptor_set_layout;
|
||||||
const vk::DescriptorSetAllocateInfo descriptor_set_ai(*descriptor_pool, 1, &layout);
|
const vk::DescriptorSetAllocateInfo descriptor_set_ai(*descriptor_pool, 1, &layout);
|
||||||
const vk::Result result =
|
const vk::Result result =
|
||||||
|
@ -487,7 +488,7 @@ void VKBlitScreen::CreateFramebuffers() {
|
||||||
const auto dev = device.GetLogical();
|
const auto dev = device.GetLogical();
|
||||||
const auto& dld = device.GetDispatchLoader();
|
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::ImageView image_view{swapchain.GetImageViewIndex(i)};
|
||||||
const vk::FramebufferCreateInfo framebuffer_ci({}, *renderpass, 1, &image_view, size.width,
|
const vk::FramebufferCreateInfo framebuffer_ci({}, *renderpass, 1, &image_view, size.width,
|
||||||
size.height, 1);
|
size.height, 1);
|
||||||
|
@ -496,7 +497,7 @@ void VKBlitScreen::CreateFramebuffers() {
|
||||||
}
|
}
|
||||||
|
|
||||||
void VKBlitScreen::ReleaseRawImages() {
|
void VKBlitScreen::ReleaseRawImages() {
|
||||||
for (u32 i = 0; i < static_cast<u32>(raw_images.size()); ++i) {
|
for (std::size_t i = 0; i < raw_images.size(); ++i) {
|
||||||
watches[i]->Wait();
|
watches[i]->Wait();
|
||||||
}
|
}
|
||||||
raw_images.clear();
|
raw_images.clear();
|
||||||
|
@ -523,7 +524,7 @@ void VKBlitScreen::CreateRawImages(const Tegra::FramebufferConfig& framebuffer)
|
||||||
raw_buffer_commits.resize(image_count);
|
raw_buffer_commits.resize(image_count);
|
||||||
|
|
||||||
const auto format = GetFormat(framebuffer);
|
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(
|
const vk::ImageCreateInfo image_ci(
|
||||||
{}, vk::ImageType::e2D, format, {framebuffer.width, framebuffer.height, 1}, 1, 1,
|
{}, vk::ImageType::e2D, format, {framebuffer.width, framebuffer.height, 1}, 1, 1,
|
||||||
vk::SampleCountFlagBits::e1, vk::ImageTiling::eOptimal,
|
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::DescriptorSet descriptor_set = descriptor_sets[image_index];
|
||||||
|
|
||||||
const vk::DescriptorBufferInfo buffer_info(*buffer, offsetof(BufferData, uniform),
|
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_transform_flags = framebuffer.transform_flags;
|
||||||
const auto& framebuffer_crop_rect = framebuffer.crop_rect;
|
const auto& framebuffer_crop_rect = framebuffer.crop_rect;
|
||||||
|
|
||||||
const Common::Rectangle<f32> texcoords{0.f, 0.f, 1.f, 1.f};
|
static constexpr Common::Rectangle<f32> texcoords{0.f, 0.f, 1.f, 1.f};
|
||||||
auto left = texcoords.left;
|
auto left = texcoords.left;
|
||||||
auto right = texcoords.right;
|
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
|
// Scale the output by the crop width/height. This is commonly used with 1280x720 rendering
|
||||||
// (e.g. handheld mode) on a 1920x1080 framebuffer.
|
// (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) {
|
if (framebuffer_crop_rect.GetWidth() > 0) {
|
||||||
scale_u = static_cast<f32>(framebuffer_crop_rect.GetWidth()) /
|
scale_u = static_cast<f32>(framebuffer_crop_rect.GetWidth()) /
|
||||||
static_cast<f32>(screen_info.width);
|
static_cast<f32>(screen_info.width);
|
||||||
|
@ -617,7 +619,7 @@ u64 VKBlitScreen::CalculateBufferSize(const Tegra::FramebufferConfig& framebuffe
|
||||||
}
|
}
|
||||||
|
|
||||||
u64 VKBlitScreen::GetRawImageOffset(const Tegra::FramebufferConfig& framebuffer,
|
u64 VKBlitScreen::GetRawImageOffset(const Tegra::FramebufferConfig& framebuffer,
|
||||||
u32 image_index) const {
|
std::size_t image_index) const {
|
||||||
constexpr auto first_image_offset = static_cast<u64>(sizeof(BufferData));
|
constexpr auto first_image_offset = static_cast<u64>(sizeof(BufferData));
|
||||||
return first_image_offset + GetSizeInBytes(framebuffer) * image_index;
|
return first_image_offset + GetSizeInBytes(framebuffer) * image_index;
|
||||||
}
|
}
|
||||||
|
|
|
@ -74,12 +74,13 @@ private:
|
||||||
void CreateStagingBuffer(const Tegra::FramebufferConfig& framebuffer);
|
void CreateStagingBuffer(const Tegra::FramebufferConfig& framebuffer);
|
||||||
void CreateRawImages(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 SetUniformData(BufferData& data, const Tegra::FramebufferConfig& framebuffer) const;
|
||||||
void SetVertexData(BufferData& data, const Tegra::FramebufferConfig& framebuffer) const;
|
void SetVertexData(BufferData& data, const Tegra::FramebufferConfig& framebuffer) const;
|
||||||
|
|
||||||
u64 CalculateBufferSize(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::System& system;
|
||||||
Core::Frontend::EmuWindow& render_window;
|
Core::Frontend::EmuWindow& render_window;
|
||||||
|
@ -89,7 +90,7 @@ private:
|
||||||
VKMemoryManager& memory_manager;
|
VKMemoryManager& memory_manager;
|
||||||
VKSwapchain& swapchain;
|
VKSwapchain& swapchain;
|
||||||
VKScheduler& scheduler;
|
VKScheduler& scheduler;
|
||||||
const u32 image_count;
|
const std::size_t image_count;
|
||||||
const VKScreenInfo& screen_info;
|
const VKScreenInfo& screen_info;
|
||||||
|
|
||||||
UniqueShaderModule vertex_shader;
|
UniqueShaderModule vertex_shader;
|
||||||
|
|
|
@ -123,7 +123,7 @@ bool VKSwapchain::Present(vk::Semaphore render_semaphore, VKFence& fence) {
|
||||||
|
|
||||||
ASSERT(fences[image_index] == nullptr);
|
ASSERT(fences[image_index] == nullptr);
|
||||||
fences[image_index] = &fence;
|
fences[image_index] = &fence;
|
||||||
frame_index = (frame_index + 1) % image_count;
|
frame_index = (frame_index + 1) % static_cast<u32>(image_count);
|
||||||
return recreated;
|
return recreated;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -40,19 +40,19 @@ public:
|
||||||
return extent;
|
return extent;
|
||||||
}
|
}
|
||||||
|
|
||||||
u32 GetImageCount() const {
|
std::size_t GetImageCount() const {
|
||||||
return image_count;
|
return image_count;
|
||||||
}
|
}
|
||||||
|
|
||||||
u32 GetImageIndex() const {
|
std::size_t GetImageIndex() const {
|
||||||
return image_index;
|
return image_index;
|
||||||
}
|
}
|
||||||
|
|
||||||
vk::Image GetImageIndex(u32 index) const {
|
vk::Image GetImageIndex(std::size_t index) const {
|
||||||
return images[index];
|
return images[index];
|
||||||
}
|
}
|
||||||
|
|
||||||
vk::ImageView GetImageViewIndex(u32 index) const {
|
vk::ImageView GetImageViewIndex(std::size_t index) const {
|
||||||
return *image_views[index];
|
return *image_views[index];
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -77,7 +77,7 @@ private:
|
||||||
|
|
||||||
UniqueSwapchainKHR swapchain;
|
UniqueSwapchainKHR swapchain;
|
||||||
|
|
||||||
u32 image_count{};
|
std::size_t image_count{};
|
||||||
std::vector<vk::Image> images;
|
std::vector<vk::Image> images;
|
||||||
std::vector<UniqueImageView> image_views;
|
std::vector<UniqueImageView> image_views;
|
||||||
std::vector<UniqueFramebuffer> framebuffers;
|
std::vector<UniqueFramebuffer> framebuffers;
|
||||||
|
|
Loading…
Reference in a new issue