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

Add missing swapchain maintenance validation #9107

Merged
merged 2 commits into from
Jan 17, 2025

Conversation

ziga-lunarg
Copy link
Contributor

Closes #9099

@ziga-lunarg ziga-lunarg requested a review from a team as a code owner December 28, 2024 21:48
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 333769.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18467 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 333786.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18468 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18468 failed.

Copy link
Contributor

@spencer-lunarg spencer-lunarg left a comment

Choose a reason for hiding this comment

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

LGTM with nits

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 336756.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18499 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18499 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 336815.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18502 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18502 failed.

@spencer-lunarg
Copy link
Contributor

@ziga-lunarg rebase and I will look into CI failures (don't want these to go stale)

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 343529.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18621 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18621 failed.

@@ -1875,6 +1875,7 @@ TEST_F(NegativeWsi, SwapchainMaintenance1ExtensionAcquire) {
AddRequiredExtensions(VK_KHR_GET_SURFACE_CAPABILITIES_2_EXTENSION_NAME);
AddRequiredExtensions(VK_EXT_SWAPCHAIN_MAINTENANCE_1_EXTENSION_NAME);
AddSurfaceExtension();
AddRequiredFeature(vkt::Feature::swapchainMaintenance1);
Copy link
Contributor

Choose a reason for hiding this comment

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

this is needed for PositiveWsi.CreateSwapchainWithPresentModeInfo and PositiveWsi.PresentFenceWaitsForSubmission

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 343638.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18626 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18626 aborted.

@@ -1351,6 +1352,7 @@ TEST_F(PositiveWsi, PresentFenceWaitsForSubmission) {
AddSurfaceExtension();
AddRequiredExtensions(VK_EXT_SURFACE_MAINTENANCE_1_EXTENSION_NAME);
AddRequiredExtensions(VK_EXT_SWAPCHAIN_MAINTENANCE_1_EXTENSION_NAME);
AddRequiredFeature(vkt::Feature::swapchainMaintenance1);
Copy link
Contributor

Choose a reason for hiding this comment

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

PresentFenceRetiresPresentQueueOperation also needs this

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 346809.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18690 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18690 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 346852.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18691 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18691 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 346925.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18692 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18692 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 347008.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18696 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18696 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 347072.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18697 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18697 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 347357.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18703 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18703 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 348173.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18731 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18731 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 348472.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18734 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18734 passed.

@spencer-lunarg
Copy link
Contributor

CI Vulkan-ValidationLayers build # 18734 passed.

NEED TO HURRY NOW!

@spencer-lunarg spencer-lunarg merged commit ff19771 into KhronosGroup:main Jan 17, 2025
21 checks passed
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.

Missing VK_EXT_swapchain_maintenance1
3 participants