lee/renderdoc
1
0
Fork 0
mirror of https://github.com/baldurk/renderdoc.git synced 2026-05-12 13:00:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

Handle duplicate semaphores (mesa returns all handles as 0x1 atm)

Browse Source
This commit is contained in:
baldurk
2016-07-13 17:07:16 +02:00
parent edf553137c
commit 13a6c98143
1 changed files with 22 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
renderdoc/driver/vulkan/wrappers/vk_sync_funcs.cpp
+22 -2
View File
@@ -481,8 +481,28 @@ bool WrappedVulkan::Serialise_vkCreateSemaphore(Serialiser *localSerialiser, VkD
}
else
{
ResourceId live = GetResourceManager()->WrapResource(Unwrap(device), sem);
GetResourceManager()->AddLiveResource(id, sem);
ResourceId live;
if(GetResourceManager()->HasWrapper(ToTypedHandle(sem)))
{
live = GetResourceManager()->GetNonDispWrapper(sem)->id;
RDCWARN(
"On replay, semaphore got a duplicate handle - maybe a bug, or it could be an "
"indication of an implementation that doesn't use semaphores");
// destroy this instance of the duplicate, as we must have matching create/destroy
// calls and there won't be a wrapped resource hanging around to destroy this one.
ObjDisp(device)->DestroySemaphore(Unwrap(device), sem, NULL);
// whenever the new ID is requested, return the old ID, via replacements.
GetResourceManager()->ReplaceResource(id, GetResourceManager()->GetOriginalID(live));
}
else
{
live = GetResourceManager()->WrapResource(Unwrap(device), sem);
GetResourceManager()->AddLiveResource(id, sem);
}
}
}