Remove Instance and Device extension logging
authorCharles Giessen <charles@lunarg.com>
Thu, 13 Jan 2022 22:48:16 +0000 (16:48 -0600)
committerCharles Giessen <46324611+charles-lunarg@users.noreply.github.com>
Mon, 7 Feb 2022 19:23:44 +0000 (12:23 -0700)
While the DEBUG level of logging is meant for verbose output, the actual extensions
enumerated rarely are useful for the purposes of debugging the loaders behavior.
With this in mind, the loader will now no longer print all the extensions found.

loader/loader.c

index 6dc6277f1c3a1a4308f7dd4587abad0dd22a6875..0bb5fc6ed1f083ca152cd78e6b4f14900b2a414a 100644 (file)
@@ -539,19 +539,10 @@ static VkResult loader_add_instance_extensions(const struct loader_instance *ins
     }
 
     for (i = 0; i < count; i++) {
-        char spec_version[64];
-
         bool ext_unsupported = wsi_unsupported_instance_extension(&ext_props[i]);
         if (!ext_unsupported) {
-            (void)snprintf(spec_version, sizeof(spec_version), "%d.%d.%d", VK_API_VERSION_MAJOR(ext_props[i].specVersion),
-                           VK_API_VERSION_MINOR(ext_props[i].specVersion), VK_API_VERSION_PATCH(ext_props[i].specVersion));
-            loader_log(inst, VULKAN_LOADER_DEBUG_BIT, 0, "Instance Extension: %s (%s) version %s", ext_props[i].extensionName,
-                       lib_name, spec_version);
-
             res = loader_add_to_ext_list(inst, ext_list, 1, &ext_props[i]);
             if (res != VK_SUCCESS) {
-                loader_log(inst, VULKAN_LOADER_ERROR_BIT, 0,
-                           "loader_add_instance_extensions: Failed to add %s to Instance extension list", lib_name);
                 goto out;
             }
         }
@@ -575,11 +566,6 @@ static VkResult loader_init_device_extensions(const struct loader_instance *inst
     }
 
     for (i = 0; i < count; i++) {
-        char spec_version[64];
-        (void)snprintf(spec_version, sizeof(spec_version), "%d.%d.%d", VK_API_VERSION_MAJOR(ext_props[i].specVersion),
-                       VK_API_VERSION_MINOR(ext_props[i].specVersion), VK_API_VERSION_PATCH(ext_props[i].specVersion));
-        loader_log(inst, VULKAN_LOADER_DEBUG_BIT, 0, "Device Extension: %s (%s) version %s", ext_props[i].extensionName,
-                   phys_dev_term->this_icd_term->scanned_icd->lib_name, spec_version);
         res = loader_add_to_ext_list(inst, ext_list, 1, &ext_props[i]);
         if (res != VK_SUCCESS) return res;
     }
@@ -608,11 +594,6 @@ VkResult loader_add_device_extensions(const struct loader_instance *inst,
             return res;
         }
         for (i = 0; i < count; i++) {
-            char spec_version[64];
-            (void)snprintf(spec_version, sizeof(spec_version), "%d.%d.%d", VK_API_VERSION_MAJOR(ext_props[i].specVersion),
-                           VK_API_VERSION_MINOR(ext_props[i].specVersion), VK_API_VERSION_PATCH(ext_props[i].specVersion));
-            loader_log(inst, VULKAN_LOADER_DEBUG_BIT, 0, "Device Extension: %s (%s) version %s", ext_props[i].extensionName,
-                       lib_name, spec_version);
             res = loader_add_to_ext_list(inst, ext_list, 1, &ext_props[i]);
             if (res != VK_SUCCESS) {
                 return res;
@@ -6860,7 +6841,6 @@ VkResult setup_loader_term_phys_dev_groups(struct loader_instance *inst) {
             }
         }
         total_count += cur_icd_group_count;
-        
     }
 
     if (total_count == 0) {