[dxvk] Move adapter list to the DXVK instance

src/dxgi/dxgi_factory.cpp

@@ -4,10 +4,9 @@
 namespace dxvk {
   
   DxgiFactory::DxgiFactory()
-  : m_instance(new DxvkInstance()),
+  : m_instance(new DxvkInstance()) {
-    m_adapters(m_instance->enumAdapters()) {
+    for (uint32_t i = 0; m_instance->enumAdapters(i) != nullptr; i++)
-    for (const auto& adapter : m_adapters)
+      m_instance->enumAdapters(i)->logAdapterInfo();
-      adapter->logAdapterInfo();
   }
   
   
@@ -182,11 +181,13 @@ namespace dxvk {
     if (ppAdapter == nullptr)
       return DXGI_ERROR_INVALID_CALL;
     
-    if (Adapter >= m_adapters.size())
+    Rc<DxvkAdapter> dxvkAdapter
+      = m_instance->enumAdapters(Adapter);
+    
+    if (dxvkAdapter == nullptr)
       return DXGI_ERROR_NOT_FOUND;
     
-    *ppAdapter = ref(new DxgiAdapter(
+    *ppAdapter = ref(new DxgiAdapter(this, dxvkAdapter));
-      this, m_adapters.at(Adapter)));
     return S_OK;
   }
   

src/dxgi/dxgi_factory.h

@@ -102,8 +102,7 @@ namespace dxvk {
     
   private:
     
-    Rc<DxvkInstance>              m_instance;
+    Rc<DxvkInstance> m_instance;
-    std::vector<Rc<DxvkAdapter>>  m_adapters;
     
     HWND m_associatedWindow = nullptr;
     

src/dxvk/dxvk_instance.cpp

@@ -8,7 +8,7 @@ namespace dxvk {
   DxvkInstance::DxvkInstance()
   : m_vkl(new vk::LibraryFn()),
     m_vki(new vk::InstanceFn(this->createInstance())) {
-    
+    this->createAdapters();
   }
   
   
@@ -17,26 +17,10 @@ namespace dxvk {
   }
   
   
-  std::vector<Rc<DxvkAdapter>> DxvkInstance::enumAdapters() {
+  Rc<DxvkAdapter> DxvkInstance::enumAdapters(uint32_t index) const {
-    uint32_t numAdapters = 0;
+    return index < m_adapters.size()
-    if (m_vki->vkEnumeratePhysicalDevices(m_vki->instance(), &numAdapters, nullptr) != VK_SUCCESS)
+      ? m_adapters[index]
-      throw DxvkError("DxvkInstance::enumAdapters: Failed to enumerate adapters");
+      : nullptr;
-    
-    std::vector<VkPhysicalDevice> adapters(numAdapters);
-    if (m_vki->vkEnumeratePhysicalDevices(m_vki->instance(), &numAdapters, adapters.data()) != VK_SUCCESS)
-      throw DxvkError("DxvkInstance::enumAdapters: Failed to enumerate adapters");
-    
-    std::vector<Rc<DxvkAdapter>> result;
-    for (uint32_t i = 0; i < numAdapters; i++)
-      result.push_back(new DxvkAdapter(this, adapters[i]));
-    
-    std::sort(result.begin(), result.end(),
-      [this] (const Rc<DxvkAdapter>& a, const Rc<DxvkAdapter>& b) -> bool {
-        return a->deviceProperties().deviceType == VK_PHYSICAL_DEVICE_TYPE_DISCRETE_GPU
-            && b->deviceProperties().deviceType != VK_PHYSICAL_DEVICE_TYPE_DISCRETE_GPU;
-      });
-    
-    return result;
   }
   
   
@@ -90,6 +74,26 @@ namespace dxvk {
   }
   
   
+  void DxvkInstance::createAdapters() {
+    uint32_t numAdapters = 0;
+    if (m_vki->vkEnumeratePhysicalDevices(m_vki->instance(), &numAdapters, nullptr) != VK_SUCCESS)
+      throw DxvkError("DxvkInstance::enumAdapters: Failed to enumerate adapters");
+    
+    std::vector<VkPhysicalDevice> adapters(numAdapters);
+    if (m_vki->vkEnumeratePhysicalDevices(m_vki->instance(), &numAdapters, adapters.data()) != VK_SUCCESS)
+      throw DxvkError("DxvkInstance::enumAdapters: Failed to enumerate adapters");
+    
+    for (uint32_t i = 0; i < numAdapters; i++)
+      m_adapters.push_back(new DxvkAdapter(this, adapters[i]));
+    
+    std::sort(m_adapters.begin(), m_adapters.end(),
+      [this] (const Rc<DxvkAdapter>& a, const Rc<DxvkAdapter>& b) -> bool {
+        return a->deviceProperties().deviceType == VK_PHYSICAL_DEVICE_TYPE_DISCRETE_GPU
+            && b->deviceProperties().deviceType != VK_PHYSICAL_DEVICE_TYPE_DISCRETE_GPU;
+      });
+  }
+  
+  
   void DxvkInstance::logNameList(const vk::NameList& names) {
     for (uint32_t i = 0; i < names.count(); i++)
       Logger::info(str::format("  ", names.name(i)));

src/dxvk/dxvk_instance.h

@@ -37,10 +37,13 @@ namespace dxvk {
     }
     
     /**
-     * \brief Retrieves a list of adapters
+     * \brief Retrieves an adapter
-     * \returns List of adapter objects
+     * 
+     * \param [in] index Adapter index
+     * \returns The adapter, or \c nullptr.
      */
-    std::vector<Rc<DxvkAdapter>> enumAdapters();
+    Rc<DxvkAdapter> enumAdapters(
+            uint32_t      index) const;
     
     /**
      * \brief Queries extra device extensions
@@ -58,8 +61,12 @@ namespace dxvk {
     Rc<vk::LibraryFn>   m_vkl;
     Rc<vk::InstanceFn>  m_vki;
 
+    std::vector<Rc<DxvkAdapter>> m_adapters;
+    
     VkInstance createInstance();
 
+    void createAdapters();
+    
     void logNameList(const vk::NameList& names);
     
   };