Allow layers to expose new extensions

solidpixel · solidpixel · commit 5f2c9a09ce9d · 2025-12-09T09:33:48.000Z
diff --git a/generator/vk_layer/source/instance.cpp b/generator/vk_layer/source/instance.cpp
@@ -38,10 +38,13 @@ static std::unordered_map<void*, std::unique_ptr<Instance>> g_instances;
 const APIVersion Instance::minAPIVersion { 1, 1 };
 
 /* See header for documentation. */
-const std::vector<std::string> Instance::extraExtensions {
+const std::vector<std::string> Instance::extraDriverExtensions {
     VK_EXT_DEBUG_UTILS_EXTENSION_NAME
 };
 
+/* See header for documentation. */
+const std::vector<std::string> Instance::injectedLayerExtensions {};
+
 /* See header for documentation. */
 void Instance::store(
     VkInstance handle,
diff --git a/generator/vk_layer/source/instance.hpp b/generator/vk_layer/source/instance.hpp
@@ -143,7 +143,12 @@ class Instance
     static const APIVersion minAPIVersion;
 
     /**
-     * @brief The minimum set of instance extensions needed by this layer.
+     * @brief The minimum set of instance extensions needed from the underlying driver.
      */
-    static const std::vector<std::string> extraExtensions;
+    static const std::vector<std::string> extraDriverExtensions;
+
+    /**
+     * @brief The minimum set of instance extensions injected by the layer.
+     */
+    static const std::vector<std::string> injectedLayerExtensions;
 };
diff --git a/layer_example/source/instance.cpp b/layer_example/source/instance.cpp
@@ -35,10 +35,15 @@
 static std::unordered_map<void*, std::unique_ptr<Instance>> g_instances;
 
 /* See header for documentation. */
-const APIVersion Instance::minAPIVersion {1, 1};
+const APIVersion Instance::minAPIVersion { 1, 1 };
 
 /* See header for documentation. */
-const std::vector<std::string> Instance::extraExtensions {VK_EXT_DEBUG_UTILS_EXTENSION_NAME};
+const std::vector<std::string> Instance::extraDriverExtensions {
+    VK_EXT_DEBUG_UTILS_EXTENSION_NAME
+};
+
+/* See header for documentation. */
+const std::vector<std::string> Instance::injectedLayerExtensions {};
 
 /* See header for documentation. */
 void Instance::store(VkInstance handle, std::unique_ptr<Instance>& instance)
diff --git a/layer_example/source/instance.hpp b/layer_example/source/instance.hpp
@@ -137,7 +137,12 @@ class Instance
     static const APIVersion minAPIVersion;
 
     /**
-     * @brief The minimum set of instance extensions needed by this layer.
+     * @brief The minimum set of instance extensions needed from the underlying driver.
      */
-    static const std::vector<std::string> extraExtensions;
+    static const std::vector<std::string> extraDriverExtensions;
+
+    /**
+     * @brief The minimum set of instance extensions injected by the layer.
+     */
+    static const std::vector<std::string> injectedLayerExtensions;
 };
diff --git a/layer_gpu_profile/source/instance.cpp b/layer_gpu_profile/source/instance.cpp
@@ -35,13 +35,16 @@
 static std::unordered_map<void*, std::unique_ptr<Instance>> g_instances;
 
 /* See header for documentation. */
-const APIVersion Instance::minAPIVersion {1, 1};
+const APIVersion Instance::minAPIVersion { 1, 1 };
 
 /* See header for documentation. */
-const std::vector<std::string> Instance::extraExtensions {
+const std::vector<std::string> Instance::extraDriverExtensions {
     VK_EXT_DEBUG_UTILS_EXTENSION_NAME,
 };
 
+/* See header for documentation. */
+const std::vector<std::string> Instance::injectedLayerExtensions {};
+
 /* See header for documentation. */
 void Instance::store(VkInstance handle, std::unique_ptr<Instance>& instance)
 {
diff --git a/layer_gpu_profile/source/instance.hpp b/layer_gpu_profile/source/instance.hpp
@@ -143,7 +143,12 @@ class Instance
     static const APIVersion minAPIVersion;
 
     /**
-     * @brief The minimum set of instance extensions needed by this layer.
+     * @brief The minimum set of instance extensions needed from the underlying driver.
      */
-    static const std::vector<std::string> extraExtensions;
+    static const std::vector<std::string> extraDriverExtensions;
+
+    /**
+     * @brief The minimum set of instance extensions injected by the layer.
+     */
+    static const std::vector<std::string> injectedLayerExtensions;
 };
diff --git a/layer_gpu_support/source/instance.cpp b/layer_gpu_support/source/instance.cpp
@@ -35,13 +35,16 @@
 static std::unordered_map<void*, std::unique_ptr<Instance>> g_instances;
 
 /* See header for documentation. */
-const APIVersion Instance::minAPIVersion {1, 1};
+const APIVersion Instance::minAPIVersion { 1, 1 };
 
 /* See header for documentation. */
-const std::vector<std::string> Instance::extraExtensions {
+const std::vector<std::string> Instance::extraDriverExtensions {
     VK_EXT_DEBUG_UTILS_EXTENSION_NAME,
 };
 
+/* See header for documentation. */
+const std::vector<std::string> Instance::injectedLayerExtensions {};
+
 /* See header for documentation. */
 void Instance::store(VkInstance handle, std::unique_ptr<Instance>& instance)
 {
diff --git a/layer_gpu_support/source/instance.hpp b/layer_gpu_support/source/instance.hpp
@@ -142,7 +142,12 @@ class Instance
     static const APIVersion minAPIVersion;
 
     /**
-     * @brief The minimum set of instance extensions needed by this layer.
+     * @brief The minimum set of instance extensions needed from the underlying driver.
      */
-    static const std::vector<std::string> extraExtensions;
+    static const std::vector<std::string> extraDriverExtensions;
+
+    /**
+     * @brief The minimum set of instance extensions injected by the layer.
+     */
+    static const std::vector<std::string> injectedLayerExtensions;
 };
diff --git a/layer_gpu_timeline/source/instance.cpp b/layer_gpu_timeline/source/instance.cpp
@@ -35,13 +35,16 @@
 static std::unordered_map<void*, std::unique_ptr<Instance>> g_instances;
 
 /* See header for documentation. */
-const APIVersion Instance::minAPIVersion {1, 1};
+const APIVersion Instance::minAPIVersion { 1, 1 };
 
 /* See header for documentation. */
-const std::vector<std::string> Instance::extraExtensions {
+const std::vector<std::string> Instance::extraDriverExtensions {
     VK_EXT_DEBUG_UTILS_EXTENSION_NAME,
 };
 
+/* See header for documentation. */
+const std::vector<std::string> Instance::injectedLayerExtensions {};
+
 /* See header for documentation. */
 void Instance::store(VkInstance handle, std::unique_ptr<Instance>& instance)
 {
diff --git a/layer_gpu_timeline/source/instance.hpp b/layer_gpu_timeline/source/instance.hpp
@@ -137,7 +137,12 @@ class Instance
     static const APIVersion minAPIVersion;
 
     /**
-     * @brief The minimum set of instance extensions needed by this layer.
+     * @brief The minimum set of instance extensions needed from the underlying driver.
      */
-    static const std::vector<std::string> extraExtensions;
+    static const std::vector<std::string> extraDriverExtensions;
+
+    /**
+     * @brief The minimum set of instance extensions injected by the layer.
+     */
+    static const std::vector<std::string> injectedLayerExtensions;
 };
diff --git a/source_common/framework/manual_functions.cpp b/source_common/framework/manual_functions.cpp
@@ -381,7 +381,9 @@ std::vector<const char*> cloneExtensionList(uint32_t extensionCount, const char*
 static void enableInstanceVkExtDebugUtils(vku::safe_VkInstanceCreateInfo& createInfo,
                                           const std::vector<std::string>& supported)
 {
-    static const std::string target {VK_EXT_DEBUG_UTILS_EXTENSION_NAME};
+    static const std::string target {
+        VK_EXT_DEBUG_UTILS_EXTENSION_NAME
+    };
 
     // Test if the desired extension is supported. If supported list is
     // empty then we didn't query and assume extension is supported.
@@ -411,7 +413,9 @@ void enableDeviceVkKhrTimelineSemaphore(Instance& instance,
     UNUSED(instance);
     UNUSED(physicalDevice);
 
-    static const std::string target {VK_KHR_TIMELINE_SEMAPHORE_EXTENSION_NAME};
+    static const std::string target {
+        VK_KHR_TIMELINE_SEMAPHORE_EXTENSION_NAME
+    };
 
     // We know we can const-cast here because createInfo is a safe-struct clone
     void* pNextBase = const_cast<void*>(createInfo.pNext);
@@ -427,8 +431,7 @@ void enableDeviceVkKhrTimelineSemaphore(Instance& instance,
 
     // Enable the extension - this will skip adding if already enabled
     if (vku::AddExtension(createInfo, target.c_str()))
-    {
-        LAYER_LOG("Device extension added: %s", target.c_str());
+    {        LAYER_LOG("Device extension added: %s", target.c_str());
     }
 
     // Check if user provided a VkPhysicalDeviceTimelineSemaphoreFeatures
@@ -766,7 +769,7 @@ VKAPI_ATTR VkResult VKAPI_CALL layer_vkCreateInstance<default_tag>(const VkInsta
     newCreateInfoSafe.pApplicationInfo->apiVersion = VK_MAKE_API_VERSION(0, newVersion.first, newVersion.second, 0);
 
     // Enable extra extensions
-    for (const auto& newExt : Instance::extraExtensions)
+    for (const auto& newExt : Instance::extraDriverExtensions)
     {
         if (newExt == VK_EXT_DEBUG_UTILS_EXTENSION_NAME)
         {
