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

Vulkan Validation Errors During Instance Creation Don't Cause Test Failures #4039

Open
jimblandy opened this issue Aug 11, 2023 · 2 comments
Labels
area: infrastructure Testing, building, coordinating issues area: tests Improvements or issues with our test suite type: enhancement New feature or request

Comments

@jimblandy
Copy link
Member

The transcript in #4037 shows a test run getting a Vulkan validation error reported against it, yet the test still passes. This could mean that our CI is missing validation failures.

@jimblandy
Copy link
Member Author

@cwfitzgerald sagely observes:

of course, we add our validation layer hook in Instance::from_raw, which happens after the instance is created

@cwfitzgerald cwfitzgerald changed the title Vulkan validation errors don't seem to cause test failures Vulkan Validation Errors During Instance Creation Don't Cause Test Failures Aug 11, 2023
@cwfitzgerald cwfitzgerald added type: enhancement New feature or request area: infrastructure Testing, building, coordinating issues area: tests Improvements or issues with our test suite labels Aug 11, 2023
@jimblandy
Copy link
Member Author

It is actually possible to capture these events: https://registry.khronos.org/vulkan/specs/1.2-extensions/html/vkspec.html#_examples_10

To capture events that occur while creating or destroying an instance, an application can link a VkDebugReportCallbackCreateInfoEXT structure or a VkDebugUtilsMessengerCreateInfoEXT structure to the pNext element of the VkInstanceCreateInfo structure given to vkCreateInstance. This callback is only valid for the duration of the vkCreateInstance and the vkDestroyInstance call. Use vkCreateDebugReportCallbackEXT or vkCreateDebugUtilsMessengerEXT to create persistent callback objects.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: infrastructure Testing, building, coordinating issues area: tests Improvements or issues with our test suite type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants