Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

layers: Add Extension Checks for Layer Extensions #8765

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

spencer-lunarg
Copy link
Contributor

From #8760 (comment) and matches spec change in https://gitlab.khronos.org/vulkan/vulkan/-/merge_requests/6979

This makes sure if you are using an extension like VK_EXT_layer_settings you need the VkInstanceCreateInfo::ppEnabledExtensionNames like any other extension

cc @charles-lunarg to make sure I am not overlooking anything obvious

@spencer-lunarg spencer-lunarg requested a review from a team as a code owner October 28, 2024 18:24
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 289587.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17847 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17847 failed.

// good to have for completeness
if (!local_instance_extensions.vk_ext_layer_settings &&
vku::FindStructInPNextChain<VkLayerSettingsCreateInfoEXT>(pCreateInfo->pNext)) {
skip |= LogError("UNASSIGNED-VkInstanceCreateInfo-layer-settings", instance,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

self note these are VUID-VkInstanceCreateInfo-pNext-10242/VUID-VkInstanceCreateInfo-pNext-10243/VUID-VkInstanceCreateInfo-pNext-10244 now

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 296446.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17925 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17925 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 296502.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17926 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17926 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 297014.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17933 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17933 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 297394.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17938 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17938 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 302021.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17998 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17998 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 302333.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18003 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 302449.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18011 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18011 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 302852.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18027 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18027 failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants