From ad026925a6e8223b171c6eb7f2726bd31ff653b3 Mon Sep 17 00:00:00 2001
From: baldurk <baldurk@baldurk.org>
Date: Fri, 3 Nov 2017 10:13:08 +0000
Subject: [PATCH] Fix remapping multiple capture vulkan phys. devs to 1 replay
 phys. dev

---
 .../driver/vulkan/wrappers/vk_device_funcs.cpp | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp b/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp
index 2386054d0..c4bc0d674 100644
--- a/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp
+++ b/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp
@@ -668,7 +668,11 @@ bool WrappedVulkan::Serialise_vkEnumeratePhysicalDevices(Serialiser *localSerial
 
     pd = m_ReplayPhysicalDevices[bestIdx];
 
-    GetResourceManager()->AddLiveResource(physId, pd);
+    if(!m_ReplayPhysicalDevicesUsed[bestIdx])
+      GetResourceManager()->AddLiveResource(physId, pd);
+    else
+      GetResourceManager()->ReplaceResource(physId,
+                                            GetResourceManager()->GetOriginalID(GetResID(pd)));
 
     if(physIndex >= m_PhysicalDevices.size())
       m_PhysicalDevices.resize(physIndex + 1);
@@ -678,16 +682,22 @@ bool WrappedVulkan::Serialise_vkEnumeratePhysicalDevices(Serialiser *localSerial
     {
       // error if we're remapping multiple physical devices to the same best match
       RDCERR(
-          "Mappnig multiple capture-time physical devices to a single replay-time physical device."
+          "Mapping multiple capture-time physical devices to a single replay-time physical device."
           "This means the HW has changed between capture and replay and may cause bugs.");
     }
-    else
+    else if(m_MemIdxMaps[bestIdx] == NULL)
     {
       // the first physical device 'wins' for the memory index map
       uint32_t *storedMap = new uint32_t[32];
       memcpy(storedMap, memIdxMap, sizeof(memIdxMap));
-      m_MemIdxMaps[physIndex] = storedMap;
+
+      for(uint32_t i = 0; i < 32; i++)
+        storedMap[i] = i;
+
+      m_MemIdxMaps[bestIdx] = storedMap;
     }
+
+    m_ReplayPhysicalDevicesUsed[bestIdx] = true;
   }
 
   return true;