forked from suyu/suyu
renderer_vulkan: do not recreate swapchain for srgb
This commit is contained in:
parent
5fb1a83e4c
commit
382cf087a0
11 changed files with 71 additions and 74 deletions
|
@ -714,7 +714,8 @@ bool RasterizerOpenGL::AccelerateDisplay(const Tegra::FramebufferConfig& config,
|
||||||
MICROPROFILE_SCOPE(OpenGL_CacheManagement);
|
MICROPROFILE_SCOPE(OpenGL_CacheManagement);
|
||||||
|
|
||||||
std::scoped_lock lock{texture_cache.mutex};
|
std::scoped_lock lock{texture_cache.mutex};
|
||||||
ImageView* const image_view{texture_cache.TryFindFramebufferImageView(framebuffer_addr)};
|
ImageView* const image_view{
|
||||||
|
texture_cache.TryFindFramebufferImageView(config, framebuffer_addr)};
|
||||||
if (!image_view) {
|
if (!image_view) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
|
@ -94,7 +94,7 @@ RendererVulkan::RendererVulkan(Core::TelemetrySession& telemetry_session_,
|
||||||
device(CreateDevice(instance, dld, *surface)), memory_allocator(device), state_tracker(),
|
device(CreateDevice(instance, dld, *surface)), memory_allocator(device), state_tracker(),
|
||||||
scheduler(device, state_tracker),
|
scheduler(device, state_tracker),
|
||||||
swapchain(*surface, device, scheduler, render_window.GetFramebufferLayout().width,
|
swapchain(*surface, device, scheduler, render_window.GetFramebufferLayout().width,
|
||||||
render_window.GetFramebufferLayout().height, false),
|
render_window.GetFramebufferLayout().height),
|
||||||
present_manager(instance, render_window, device, memory_allocator, scheduler, swapchain,
|
present_manager(instance, render_window, device, memory_allocator, scheduler, swapchain,
|
||||||
surface),
|
surface),
|
||||||
blit_screen(cpu_memory, render_window, device, memory_allocator, swapchain, present_manager,
|
blit_screen(cpu_memory, render_window, device, memory_allocator, swapchain, present_manager,
|
||||||
|
@ -131,11 +131,10 @@ void RendererVulkan::SwapBuffers(const Tegra::FramebufferConfig* framebuffer) {
|
||||||
const VAddr framebuffer_addr = framebuffer->address + framebuffer->offset;
|
const VAddr framebuffer_addr = framebuffer->address + framebuffer->offset;
|
||||||
const bool use_accelerated =
|
const bool use_accelerated =
|
||||||
rasterizer.AccelerateDisplay(*framebuffer, framebuffer_addr, framebuffer->stride);
|
rasterizer.AccelerateDisplay(*framebuffer, framebuffer_addr, framebuffer->stride);
|
||||||
const bool is_srgb = use_accelerated && screen_info.is_srgb;
|
|
||||||
RenderScreenshot(*framebuffer, use_accelerated);
|
RenderScreenshot(*framebuffer, use_accelerated);
|
||||||
|
|
||||||
Frame* frame = present_manager.GetRenderFrame();
|
Frame* frame = present_manager.GetRenderFrame();
|
||||||
blit_screen.DrawToSwapchain(frame, *framebuffer, use_accelerated, is_srgb);
|
blit_screen.DrawToSwapchain(frame, *framebuffer, use_accelerated);
|
||||||
scheduler.Flush(*frame->render_ready);
|
scheduler.Flush(*frame->render_ready);
|
||||||
present_manager.Present(frame);
|
present_manager.Present(frame);
|
||||||
|
|
||||||
|
@ -205,7 +204,7 @@ void Vulkan::RendererVulkan::RenderScreenshot(const Tegra::FramebufferConfig& fr
|
||||||
.flags = 0,
|
.flags = 0,
|
||||||
.image = *staging_image,
|
.image = *staging_image,
|
||||||
.viewType = VK_IMAGE_VIEW_TYPE_2D,
|
.viewType = VK_IMAGE_VIEW_TYPE_2D,
|
||||||
.format = screen_info.is_srgb ? VK_FORMAT_B8G8R8A8_SRGB : VK_FORMAT_B8G8R8A8_UNORM,
|
.format = VK_FORMAT_B8G8R8A8_UNORM,
|
||||||
.components{
|
.components{
|
||||||
.r = VK_COMPONENT_SWIZZLE_IDENTITY,
|
.r = VK_COMPONENT_SWIZZLE_IDENTITY,
|
||||||
.g = VK_COMPONENT_SWIZZLE_IDENTITY,
|
.g = VK_COMPONENT_SWIZZLE_IDENTITY,
|
||||||
|
|
|
@ -127,9 +127,9 @@ BlitScreen::BlitScreen(Core::Memory::Memory& cpu_memory_, Core::Frontend::EmuWin
|
||||||
Scheduler& scheduler_, const ScreenInfo& screen_info_)
|
Scheduler& scheduler_, const ScreenInfo& screen_info_)
|
||||||
: cpu_memory{cpu_memory_}, render_window{render_window_}, device{device_},
|
: cpu_memory{cpu_memory_}, render_window{render_window_}, device{device_},
|
||||||
memory_allocator{memory_allocator_}, swapchain{swapchain_}, present_manager{present_manager_},
|
memory_allocator{memory_allocator_}, swapchain{swapchain_}, present_manager{present_manager_},
|
||||||
scheduler{scheduler_}, image_count{swapchain.GetImageCount()}, screen_info{screen_info_},
|
scheduler{scheduler_}, image_count{swapchain.GetImageCount()}, screen_info{screen_info_} {
|
||||||
current_srgb{swapchain.IsSrgb()}, image_view_format{swapchain.GetImageViewFormat()} {
|
|
||||||
resource_ticks.resize(image_count);
|
resource_ticks.resize(image_count);
|
||||||
|
swapchain_view_format = swapchain.GetImageViewFormat();
|
||||||
|
|
||||||
CreateStaticResources();
|
CreateStaticResources();
|
||||||
CreateDynamicResources();
|
CreateDynamicResources();
|
||||||
|
@ -480,28 +480,22 @@ void BlitScreen::Draw(const Tegra::FramebufferConfig& framebuffer,
|
||||||
}
|
}
|
||||||
|
|
||||||
void BlitScreen::DrawToSwapchain(Frame* frame, const Tegra::FramebufferConfig& framebuffer,
|
void BlitScreen::DrawToSwapchain(Frame* frame, const Tegra::FramebufferConfig& framebuffer,
|
||||||
bool use_accelerated, bool is_srgb) {
|
bool use_accelerated) {
|
||||||
// Recreate dynamic resources if the the image count or colorspace changed
|
// Recreate dynamic resources if the the image count or input format changed
|
||||||
|
const VkFormat current_framebuffer_format =
|
||||||
|
std::exchange(framebuffer_view_format, GetFormat(framebuffer));
|
||||||
if (const std::size_t swapchain_images = swapchain.GetImageCount();
|
if (const std::size_t swapchain_images = swapchain.GetImageCount();
|
||||||
swapchain_images != image_count || current_srgb != is_srgb) {
|
swapchain_images != image_count || current_framebuffer_format != framebuffer_view_format) {
|
||||||
current_srgb = is_srgb;
|
|
||||||
#ifdef ANDROID
|
|
||||||
// Android is already ordered the same as Switch.
|
|
||||||
image_view_format = current_srgb ? VK_FORMAT_R8G8B8A8_SRGB : VK_FORMAT_R8G8B8A8_UNORM;
|
|
||||||
#else
|
|
||||||
image_view_format = current_srgb ? VK_FORMAT_B8G8R8A8_SRGB : VK_FORMAT_B8G8R8A8_UNORM;
|
|
||||||
#endif
|
|
||||||
image_count = swapchain_images;
|
image_count = swapchain_images;
|
||||||
Recreate();
|
Recreate();
|
||||||
}
|
}
|
||||||
|
|
||||||
// Recreate the presentation frame if the dimensions of the window changed
|
// Recreate the presentation frame if the dimensions of the window changed
|
||||||
const Layout::FramebufferLayout layout = render_window.GetFramebufferLayout();
|
const Layout::FramebufferLayout layout = render_window.GetFramebufferLayout();
|
||||||
if (layout.width != frame->width || layout.height != frame->height ||
|
if (layout.width != frame->width || layout.height != frame->height) {
|
||||||
is_srgb != frame->is_srgb) {
|
|
||||||
Recreate();
|
Recreate();
|
||||||
present_manager.RecreateFrame(frame, layout.width, layout.height, is_srgb,
|
present_manager.RecreateFrame(frame, layout.width, layout.height, swapchain_view_format,
|
||||||
image_view_format, *renderpass);
|
*renderpass);
|
||||||
}
|
}
|
||||||
|
|
||||||
const VkExtent2D render_area{frame->width, frame->height};
|
const VkExtent2D render_area{frame->width, frame->height};
|
||||||
|
@ -629,7 +623,7 @@ void BlitScreen::CreateDescriptorPool() {
|
||||||
}
|
}
|
||||||
|
|
||||||
void BlitScreen::CreateRenderPass() {
|
void BlitScreen::CreateRenderPass() {
|
||||||
renderpass = CreateRenderPassImpl(image_view_format);
|
renderpass = CreateRenderPassImpl(swapchain_view_format);
|
||||||
}
|
}
|
||||||
|
|
||||||
vk::RenderPass BlitScreen::CreateRenderPassImpl(VkFormat format) {
|
vk::RenderPass BlitScreen::CreateRenderPassImpl(VkFormat format) {
|
||||||
|
@ -1149,7 +1143,7 @@ void BlitScreen::CreateRawImages(const Tegra::FramebufferConfig& framebuffer) {
|
||||||
.pNext = nullptr,
|
.pNext = nullptr,
|
||||||
.flags = 0,
|
.flags = 0,
|
||||||
.imageType = VK_IMAGE_TYPE_2D,
|
.imageType = VK_IMAGE_TYPE_2D,
|
||||||
.format = used_on_framebuffer ? VK_FORMAT_R16G16B16A16_SFLOAT : GetFormat(framebuffer),
|
.format = used_on_framebuffer ? VK_FORMAT_R16G16B16A16_SFLOAT : framebuffer_view_format,
|
||||||
.extent =
|
.extent =
|
||||||
{
|
{
|
||||||
.width = (up_scale * framebuffer.width) >> down_shift,
|
.width = (up_scale * framebuffer.width) >> down_shift,
|
||||||
|
@ -1174,7 +1168,7 @@ void BlitScreen::CreateRawImages(const Tegra::FramebufferConfig& framebuffer) {
|
||||||
.flags = 0,
|
.flags = 0,
|
||||||
.image = *image,
|
.image = *image,
|
||||||
.viewType = VK_IMAGE_VIEW_TYPE_2D,
|
.viewType = VK_IMAGE_VIEW_TYPE_2D,
|
||||||
.format = used_on_framebuffer ? VK_FORMAT_R16G16B16A16_SFLOAT : GetFormat(framebuffer),
|
.format = used_on_framebuffer ? VK_FORMAT_R16G16B16A16_SFLOAT : framebuffer_view_format,
|
||||||
.components =
|
.components =
|
||||||
{
|
{
|
||||||
.r = VK_COMPONENT_SWIZZLE_IDENTITY,
|
.r = VK_COMPONENT_SWIZZLE_IDENTITY,
|
||||||
|
|
|
@ -52,7 +52,6 @@ struct ScreenInfo {
|
||||||
VkImageView image_view{};
|
VkImageView image_view{};
|
||||||
u32 width{};
|
u32 width{};
|
||||||
u32 height{};
|
u32 height{};
|
||||||
bool is_srgb{};
|
|
||||||
};
|
};
|
||||||
|
|
||||||
class BlitScreen {
|
class BlitScreen {
|
||||||
|
@ -69,7 +68,7 @@ public:
|
||||||
const Layout::FramebufferLayout layout, VkExtent2D render_area, bool use_accelerated);
|
const Layout::FramebufferLayout layout, VkExtent2D render_area, bool use_accelerated);
|
||||||
|
|
||||||
void DrawToSwapchain(Frame* frame, const Tegra::FramebufferConfig& framebuffer,
|
void DrawToSwapchain(Frame* frame, const Tegra::FramebufferConfig& framebuffer,
|
||||||
bool use_accelerated, bool is_srgb);
|
bool use_accelerated);
|
||||||
|
|
||||||
[[nodiscard]] vk::Framebuffer CreateFramebuffer(const VkImageView& image_view,
|
[[nodiscard]] vk::Framebuffer CreateFramebuffer(const VkImageView& image_view,
|
||||||
VkExtent2D extent);
|
VkExtent2D extent);
|
||||||
|
@ -161,8 +160,8 @@ private:
|
||||||
u32 raw_width = 0;
|
u32 raw_width = 0;
|
||||||
u32 raw_height = 0;
|
u32 raw_height = 0;
|
||||||
Service::android::PixelFormat pixel_format{};
|
Service::android::PixelFormat pixel_format{};
|
||||||
bool current_srgb;
|
VkFormat framebuffer_view_format;
|
||||||
VkFormat image_view_format;
|
VkFormat swapchain_view_format;
|
||||||
|
|
||||||
std::unique_ptr<FSR> fsr;
|
std::unique_ptr<FSR> fsr;
|
||||||
std::unique_ptr<SMAA> smaa;
|
std::unique_ptr<SMAA> smaa;
|
||||||
|
|
|
@ -172,13 +172,12 @@ void PresentManager::Present(Frame* frame) {
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
void PresentManager::RecreateFrame(Frame* frame, u32 width, u32 height, bool is_srgb,
|
void PresentManager::RecreateFrame(Frame* frame, u32 width, u32 height, VkFormat image_view_format,
|
||||||
VkFormat image_view_format, VkRenderPass rd) {
|
VkRenderPass rd) {
|
||||||
auto& dld = device.GetLogical();
|
auto& dld = device.GetLogical();
|
||||||
|
|
||||||
frame->width = width;
|
frame->width = width;
|
||||||
frame->height = height;
|
frame->height = height;
|
||||||
frame->is_srgb = is_srgb;
|
|
||||||
|
|
||||||
frame->image = memory_allocator.CreateImage({
|
frame->image = memory_allocator.CreateImage({
|
||||||
.sType = VK_STRUCTURE_TYPE_IMAGE_CREATE_INFO,
|
.sType = VK_STRUCTURE_TYPE_IMAGE_CREATE_INFO,
|
||||||
|
@ -289,7 +288,7 @@ void PresentManager::PresentThread(std::stop_token token) {
|
||||||
}
|
}
|
||||||
|
|
||||||
void PresentManager::RecreateSwapchain(Frame* frame) {
|
void PresentManager::RecreateSwapchain(Frame* frame) {
|
||||||
swapchain.Create(*surface, frame->width, frame->height, frame->is_srgb);
|
swapchain.Create(*surface, frame->width, frame->height);
|
||||||
image_count = swapchain.GetImageCount();
|
image_count = swapchain.GetImageCount();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -319,12 +318,12 @@ void PresentManager::CopyToSwapchain(Frame* frame) {
|
||||||
void PresentManager::CopyToSwapchainImpl(Frame* frame) {
|
void PresentManager::CopyToSwapchainImpl(Frame* frame) {
|
||||||
MICROPROFILE_SCOPE(Vulkan_CopyToSwapchain);
|
MICROPROFILE_SCOPE(Vulkan_CopyToSwapchain);
|
||||||
|
|
||||||
// If the size or colorspace of the incoming frames has changed, recreate the swapchain
|
// If the size of the incoming frames has changed, recreate the swapchain
|
||||||
// to account for that.
|
// to account for that.
|
||||||
const bool srgb_changed = swapchain.NeedsRecreation(frame->is_srgb);
|
const bool is_suboptimal = swapchain.NeedsRecreation();
|
||||||
const bool size_changed =
|
const bool size_changed =
|
||||||
swapchain.GetWidth() != frame->width || swapchain.GetHeight() != frame->height;
|
swapchain.GetWidth() != frame->width || swapchain.GetHeight() != frame->height;
|
||||||
if (srgb_changed || size_changed) {
|
if (is_suboptimal || size_changed) {
|
||||||
RecreateSwapchain(frame);
|
RecreateSwapchain(frame);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -25,7 +25,6 @@ class Swapchain;
|
||||||
struct Frame {
|
struct Frame {
|
||||||
u32 width;
|
u32 width;
|
||||||
u32 height;
|
u32 height;
|
||||||
bool is_srgb;
|
|
||||||
vk::Image image;
|
vk::Image image;
|
||||||
vk::ImageView image_view;
|
vk::ImageView image_view;
|
||||||
vk::Framebuffer framebuffer;
|
vk::Framebuffer framebuffer;
|
||||||
|
@ -48,8 +47,8 @@ public:
|
||||||
void Present(Frame* frame);
|
void Present(Frame* frame);
|
||||||
|
|
||||||
/// Recreates the present frame to match the provided parameters
|
/// Recreates the present frame to match the provided parameters
|
||||||
void RecreateFrame(Frame* frame, u32 width, u32 height, bool is_srgb,
|
void RecreateFrame(Frame* frame, u32 width, u32 height, VkFormat image_view_format,
|
||||||
VkFormat image_view_format, VkRenderPass rd);
|
VkRenderPass rd);
|
||||||
|
|
||||||
/// Waits for the present thread to finish presenting all queued frames.
|
/// Waits for the present thread to finish presenting all queued frames.
|
||||||
void WaitPresent();
|
void WaitPresent();
|
||||||
|
|
|
@ -777,7 +777,8 @@ bool RasterizerVulkan::AccelerateDisplay(const Tegra::FramebufferConfig& config,
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
std::scoped_lock lock{texture_cache.mutex};
|
std::scoped_lock lock{texture_cache.mutex};
|
||||||
ImageView* const image_view = texture_cache.TryFindFramebufferImageView(framebuffer_addr);
|
ImageView* const image_view =
|
||||||
|
texture_cache.TryFindFramebufferImageView(config, framebuffer_addr);
|
||||||
if (!image_view) {
|
if (!image_view) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
@ -786,7 +787,6 @@ bool RasterizerVulkan::AccelerateDisplay(const Tegra::FramebufferConfig& config,
|
||||||
screen_info.image_view = image_view->Handle(Shader::TextureType::Color2D);
|
screen_info.image_view = image_view->Handle(Shader::TextureType::Color2D);
|
||||||
screen_info.width = image_view->size.width;
|
screen_info.width = image_view->size.width;
|
||||||
screen_info.height = image_view->size.height;
|
screen_info.height = image_view->size.height;
|
||||||
screen_info.is_srgb = VideoCore::Surface::IsPixelFormatSRGB(image_view->format);
|
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -105,14 +105,14 @@ VkCompositeAlphaFlagBitsKHR ChooseAlphaFlags(const VkSurfaceCapabilitiesKHR& cap
|
||||||
} // Anonymous namespace
|
} // Anonymous namespace
|
||||||
|
|
||||||
Swapchain::Swapchain(VkSurfaceKHR surface_, const Device& device_, Scheduler& scheduler_,
|
Swapchain::Swapchain(VkSurfaceKHR surface_, const Device& device_, Scheduler& scheduler_,
|
||||||
u32 width_, u32 height_, bool srgb)
|
u32 width_, u32 height_)
|
||||||
: surface{surface_}, device{device_}, scheduler{scheduler_} {
|
: surface{surface_}, device{device_}, scheduler{scheduler_} {
|
||||||
Create(surface_, width_, height_, srgb);
|
Create(surface_, width_, height_);
|
||||||
}
|
}
|
||||||
|
|
||||||
Swapchain::~Swapchain() = default;
|
Swapchain::~Swapchain() = default;
|
||||||
|
|
||||||
void Swapchain::Create(VkSurfaceKHR surface_, u32 width_, u32 height_, bool srgb) {
|
void Swapchain::Create(VkSurfaceKHR surface_, u32 width_, u32 height_) {
|
||||||
is_outdated = false;
|
is_outdated = false;
|
||||||
is_suboptimal = false;
|
is_suboptimal = false;
|
||||||
width = width_;
|
width = width_;
|
||||||
|
@ -127,7 +127,7 @@ void Swapchain::Create(VkSurfaceKHR surface_, u32 width_, u32 height_, bool srgb
|
||||||
|
|
||||||
Destroy();
|
Destroy();
|
||||||
|
|
||||||
CreateSwapchain(capabilities, srgb);
|
CreateSwapchain(capabilities);
|
||||||
CreateSemaphores();
|
CreateSemaphores();
|
||||||
|
|
||||||
resource_ticks.clear();
|
resource_ticks.clear();
|
||||||
|
@ -196,7 +196,7 @@ void Swapchain::Present(VkSemaphore render_semaphore) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
void Swapchain::CreateSwapchain(const VkSurfaceCapabilitiesKHR& capabilities, bool srgb) {
|
void Swapchain::CreateSwapchain(const VkSurfaceCapabilitiesKHR& capabilities) {
|
||||||
const auto physical_device{device.GetPhysical()};
|
const auto physical_device{device.GetPhysical()};
|
||||||
const auto formats{physical_device.GetSurfaceFormatsKHR(surface)};
|
const auto formats{physical_device.GetSurfaceFormatsKHR(surface)};
|
||||||
const auto present_modes = physical_device.GetSurfacePresentModesKHR(surface);
|
const auto present_modes = physical_device.GetSurfacePresentModesKHR(surface);
|
||||||
|
@ -274,15 +274,14 @@ void Swapchain::CreateSwapchain(const VkSurfaceCapabilitiesKHR& capabilities, bo
|
||||||
swapchain = device.GetLogical().CreateSwapchainKHR(swapchain_ci);
|
swapchain = device.GetLogical().CreateSwapchainKHR(swapchain_ci);
|
||||||
|
|
||||||
extent = swapchain_ci.imageExtent;
|
extent = swapchain_ci.imageExtent;
|
||||||
current_srgb = srgb;
|
|
||||||
|
|
||||||
images = swapchain.GetImages();
|
images = swapchain.GetImages();
|
||||||
image_count = static_cast<u32>(images.size());
|
image_count = static_cast<u32>(images.size());
|
||||||
#ifdef ANDROID
|
#ifdef ANDROID
|
||||||
// Android is already ordered the same as Switch.
|
// Android is already ordered the same as Switch.
|
||||||
image_view_format = srgb ? VK_FORMAT_R8G8B8A8_SRGB : VK_FORMAT_R8G8B8A8_UNORM;
|
image_view_format = VK_FORMAT_R8G8B8A8_UNORM;
|
||||||
#else
|
#else
|
||||||
image_view_format = srgb ? VK_FORMAT_B8G8R8A8_SRGB : VK_FORMAT_B8G8R8A8_UNORM;
|
image_view_format = VK_FORMAT_B8G8R8A8_UNORM;
|
||||||
#endif
|
#endif
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -20,11 +20,11 @@ class Scheduler;
|
||||||
class Swapchain {
|
class Swapchain {
|
||||||
public:
|
public:
|
||||||
explicit Swapchain(VkSurfaceKHR surface, const Device& device, Scheduler& scheduler, u32 width,
|
explicit Swapchain(VkSurfaceKHR surface, const Device& device, Scheduler& scheduler, u32 width,
|
||||||
u32 height, bool srgb);
|
u32 height);
|
||||||
~Swapchain();
|
~Swapchain();
|
||||||
|
|
||||||
/// Creates (or recreates) the swapchain with a given size.
|
/// Creates (or recreates) the swapchain with a given size.
|
||||||
void Create(VkSurfaceKHR surface, u32 width, u32 height, bool srgb);
|
void Create(VkSurfaceKHR surface, u32 width, u32 height);
|
||||||
|
|
||||||
/// Acquires the next image in the swapchain, waits as needed.
|
/// Acquires the next image in the swapchain, waits as needed.
|
||||||
bool AcquireNextImage();
|
bool AcquireNextImage();
|
||||||
|
@ -33,13 +33,8 @@ public:
|
||||||
void Present(VkSemaphore render_semaphore);
|
void Present(VkSemaphore render_semaphore);
|
||||||
|
|
||||||
/// Returns true when the swapchain needs to be recreated.
|
/// Returns true when the swapchain needs to be recreated.
|
||||||
bool NeedsRecreation(bool is_srgb) const {
|
bool NeedsRecreation() const {
|
||||||
return HasColorSpaceChanged(is_srgb) || IsSubOptimal() || NeedsPresentModeUpdate();
|
return IsSubOptimal() || NeedsPresentModeUpdate();
|
||||||
}
|
|
||||||
|
|
||||||
/// Returns true when the color space has changed.
|
|
||||||
bool HasColorSpaceChanged(bool is_srgb) const {
|
|
||||||
return current_srgb != is_srgb;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Returns true when the swapchain is outdated.
|
/// Returns true when the swapchain is outdated.
|
||||||
|
@ -52,11 +47,6 @@ public:
|
||||||
return is_suboptimal;
|
return is_suboptimal;
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Returns true when the swapchain format is in the srgb color space
|
|
||||||
bool IsSrgb() const {
|
|
||||||
return current_srgb;
|
|
||||||
}
|
|
||||||
|
|
||||||
VkExtent2D GetSize() const {
|
VkExtent2D GetSize() const {
|
||||||
return extent;
|
return extent;
|
||||||
}
|
}
|
||||||
|
@ -110,7 +100,7 @@ public:
|
||||||
}
|
}
|
||||||
|
|
||||||
private:
|
private:
|
||||||
void CreateSwapchain(const VkSurfaceCapabilitiesKHR& capabilities, bool srgb);
|
void CreateSwapchain(const VkSurfaceCapabilitiesKHR& capabilities);
|
||||||
void CreateSemaphores();
|
void CreateSemaphores();
|
||||||
void CreateImageViews();
|
void CreateImageViews();
|
||||||
|
|
||||||
|
@ -144,7 +134,6 @@ private:
|
||||||
bool has_mailbox{false};
|
bool has_mailbox{false};
|
||||||
bool has_fifo_relaxed{false};
|
bool has_fifo_relaxed{false};
|
||||||
|
|
||||||
bool current_srgb{};
|
|
||||||
bool is_outdated{};
|
bool is_outdated{};
|
||||||
bool is_suboptimal{};
|
bool is_suboptimal{};
|
||||||
};
|
};
|
||||||
|
|
|
@ -712,14 +712,15 @@ bool TextureCache<P>::BlitImage(const Tegra::Engines::Fermi2D::Surface& dst,
|
||||||
}
|
}
|
||||||
|
|
||||||
template <class P>
|
template <class P>
|
||||||
typename P::ImageView* TextureCache<P>::TryFindFramebufferImageView(VAddr cpu_addr) {
|
typename P::ImageView* TextureCache<P>::TryFindFramebufferImageView(
|
||||||
|
const Tegra::FramebufferConfig& config, VAddr cpu_addr) {
|
||||||
// TODO: Properly implement this
|
// TODO: Properly implement this
|
||||||
const auto it = page_table.find(cpu_addr >> YUZU_PAGEBITS);
|
const auto it = page_table.find(cpu_addr >> YUZU_PAGEBITS);
|
||||||
if (it == page_table.end()) {
|
if (it == page_table.end()) {
|
||||||
return nullptr;
|
return nullptr;
|
||||||
}
|
}
|
||||||
const auto& image_map_ids = it->second;
|
const auto& image_map_ids = it->second;
|
||||||
boost::container::small_vector<const ImageBase*, 4> valid_images;
|
boost::container::small_vector<ImageId, 4> valid_image_ids;
|
||||||
for (const ImageMapId map_id : image_map_ids) {
|
for (const ImageMapId map_id : image_map_ids) {
|
||||||
const ImageMapView& map = slot_map_views[map_id];
|
const ImageMapView& map = slot_map_views[map_id];
|
||||||
const ImageBase& image = slot_images[map.image_id];
|
const ImageBase& image = slot_images[map.image_id];
|
||||||
|
@ -729,18 +730,34 @@ typename P::ImageView* TextureCache<P>::TryFindFramebufferImageView(VAddr cpu_ad
|
||||||
if (image.image_view_ids.empty()) {
|
if (image.image_view_ids.empty()) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
valid_images.push_back(&image);
|
valid_image_ids.push_back(map.image_id);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (valid_images.size() == 1) [[likely]] {
|
const auto view_format = [&]() {
|
||||||
return &slot_image_views[valid_images[0]->image_view_ids.at(0)];
|
switch (config.pixel_format) {
|
||||||
|
case Service::android::PixelFormat::Rgb565:
|
||||||
|
return PixelFormat::R5G6B5_UNORM;
|
||||||
|
case Service::android::PixelFormat::Bgra8888:
|
||||||
|
return PixelFormat::B8G8R8A8_UNORM;
|
||||||
|
default:
|
||||||
|
return PixelFormat::A8B8G8R8_UNORM;
|
||||||
|
}
|
||||||
|
}();
|
||||||
|
|
||||||
|
const auto GetImageViewForFramebuffer = [&](ImageId image_id) {
|
||||||
|
const ImageViewInfo info{ImageViewType::e2D, view_format};
|
||||||
|
return &slot_image_views[FindOrEmplaceImageView(image_id, info)];
|
||||||
|
};
|
||||||
|
|
||||||
|
if (valid_image_ids.size() == 1) [[likely]] {
|
||||||
|
return GetImageViewForFramebuffer(valid_image_ids.front());
|
||||||
}
|
}
|
||||||
|
|
||||||
if (valid_images.size() > 0) [[unlikely]] {
|
if (valid_image_ids.size() > 0) [[unlikely]] {
|
||||||
std::ranges::sort(valid_images, [](const auto* a, const auto* b) {
|
auto most_recent = std::ranges::max_element(valid_image_ids, [&](auto a, auto b) {
|
||||||
return a->modification_tick > b->modification_tick;
|
return slot_images[a].modification_tick > slot_images[b].modification_tick;
|
||||||
});
|
});
|
||||||
return &slot_image_views[valid_images[0]->image_view_ids.at(0)];
|
return GetImageViewForFramebuffer(*most_recent);
|
||||||
}
|
}
|
||||||
|
|
||||||
return nullptr;
|
return nullptr;
|
||||||
|
|
|
@ -209,7 +209,8 @@ public:
|
||||||
const Tegra::Engines::Fermi2D::Config& copy);
|
const Tegra::Engines::Fermi2D::Config& copy);
|
||||||
|
|
||||||
/// Try to find a cached image view in the given CPU address
|
/// Try to find a cached image view in the given CPU address
|
||||||
[[nodiscard]] ImageView* TryFindFramebufferImageView(VAddr cpu_addr);
|
[[nodiscard]] ImageView* TryFindFramebufferImageView(const Tegra::FramebufferConfig& config,
|
||||||
|
VAddr cpu_addr);
|
||||||
|
|
||||||
/// Return true when there are uncommitted images to be downloaded
|
/// Return true when there are uncommitted images to be downloaded
|
||||||
[[nodiscard]] bool HasUncommittedFlushes() const noexcept;
|
[[nodiscard]] bool HasUncommittedFlushes() const noexcept;
|
||||||
|
|
Loading…
Reference in a new issue