archive/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2024-09-21 11:51:48 +02:00
Code Issues Projects Releases Packages Wiki Activity

Vulkan: Fix incorrect fence being assigned to staging texture

Browse source
This commit is contained in:
Stenzek 2018-11-12 01:02:32 +10:00
parent f1c41c9a62
commit a0653a1a34
1 changed files with 15 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
Source/Core/VideoBackends/Vulkan/VKTexture.cpp
View file

@ -407,7 +407,7 @@ void VKStagingTexture::CopyFromTexture(const AbstractTexture* src,
const MathUtil::Rectangle<int>& src_rect, u32 src_layer, const MathUtil::Rectangle<int>& src_rect, u32 src_layer,
u32 src_level, const MathUtil::Rectangle<int>& dst_rect) u32 src_level, const MathUtil::Rectangle<int>& dst_rect)
{ {
ASSERT(m_type == StagingTextureType::Readback); ASSERT(m_type == StagingTextureType::Readback || m_type == StagingTextureType::Mutable);
ASSERT(src_rect.GetWidth() == dst_rect.GetWidth() && ASSERT(src_rect.GetWidth() == dst_rect.GetWidth() &&
src_rect.GetHeight() == dst_rect.GetHeight()); src_rect.GetHeight() == dst_rect.GetHeight());
ASSERT(src_rect.left >= 0 && static_cast<u32>(src_rect.right) <= src->GetConfig().width && ASSERT(src_rect.left >= 0 && static_cast<u32>(src_rect.right) <= src->GetConfig().width &&
@ -459,9 +459,15 @@ void VKStagingTexture::CopyFromTexture(Texture2D* src, const MathUtil::Rectangle
g_command_buffer_mgr->AddFencePointCallback(this, g_command_buffer_mgr->AddFencePointCallback(this,
[this](VkCommandBuffer buf, VkFence fence) { [this](VkCommandBuffer buf, VkFence fence) {
ASSERT(m_needs_flush); ASSERT(m_needs_flush);
if (m_flush_fence != VK_NULL_HANDLE)
return;
m_flush_fence = fence; m_flush_fence = fence;
}, },
[this](VkFence fence) { [this](VkFence fence) {
if (m_flush_fence != fence)
return;
m_flush_fence = VK_NULL_HANDLE; m_flush_fence = VK_NULL_HANDLE;
m_needs_flush = false; m_needs_flush = false;
g_command_buffer_mgr->RemoveFencePointCallback( g_command_buffer_mgr->RemoveFencePointCallback(
@ -474,7 +480,7 @@ void VKStagingTexture::CopyToTexture(const MathUtil::Rectangle<int>& src_rect, A
const MathUtil::Rectangle<int>& dst_rect, u32 dst_layer, const MathUtil::Rectangle<int>& dst_rect, u32 dst_layer,
u32 dst_level) u32 dst_level)
{ {
ASSERT(m_type == StagingTextureType::Upload); ASSERT(m_type == StagingTextureType::Upload || m_type == StagingTextureType::Mutable);
ASSERT(src_rect.GetWidth() == dst_rect.GetWidth() && ASSERT(src_rect.GetWidth() == dst_rect.GetWidth() &&
src_rect.GetHeight() == dst_rect.GetHeight()); src_rect.GetHeight() == dst_rect.GetHeight());
ASSERT(src_rect.left >= 0 && static_cast<u32>(src_rect.right) <= m_config.width && ASSERT(src_rect.left >= 0 && static_cast<u32>(src_rect.right) <= m_config.width &&
@ -520,9 +526,15 @@ void VKStagingTexture::CopyToTexture(const MathUtil::Rectangle<int>& src_rect, A
g_command_buffer_mgr->AddFencePointCallback(this, g_command_buffer_mgr->AddFencePointCallback(this,
[this](VkCommandBuffer buf, VkFence fence) { [this](VkCommandBuffer buf, VkFence fence) {
ASSERT(m_needs_flush); ASSERT(m_needs_flush);
if (m_flush_fence != VK_NULL_HANDLE)
return;
m_flush_fence = fence; m_flush_fence = fence;
}, },
[this](VkFence fence) { [this](VkFence fence) {
if (m_flush_fence != fence)
return;
m_flush_fence = VK_NULL_HANDLE; m_flush_fence = VK_NULL_HANDLE;
m_needs_flush = false; m_needs_flush = false;
g_command_buffer_mgr->RemoveFencePointCallback( g_command_buffer_mgr->RemoveFencePointCallback(
@ -563,7 +575,7 @@ void VKStagingTexture::Flush()
m_needs_flush = false; m_needs_flush = false;
// For readback textures, invalidate the CPU cache as there is new data there. // For readback textures, invalidate the CPU cache as there is new data there.
if (m_type == StagingTextureType::Readback) if (m_type == StagingTextureType::Readback || m_type == StagingTextureType::Mutable)
m_staging_buffer->InvalidateCPUCache(); m_staging_buffer->InvalidateCPUCache();
} }