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: Include values in wsi logs #9208

Merged

Conversation

ziga-lunarg
Copy link
Contributor

No description provided.

@ziga-lunarg ziga-lunarg requested a review from a team as a code owner January 13, 2025 11:52
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 343563.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18623 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18623 failed.

swapchain_ci.clipped = VK_FALSE;
swapchain_ci.oldSwapchain = 0;

m_errorMonitor->SetAllowedFailureMsg("VUID-VkSwapchainCreateInfoKHR-compositeAlpha-01280");
Copy link
Contributor

Choose a reason for hiding this comment

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

so this is failing on AMD Windows, should this not be SetDesiredError because by having it SetAllowedFailureMsg it will still try to create the VkSwapchian which in this case is invalid and blowing up CI machine

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wrong copy paste!

@ziga-lunarg ziga-lunarg force-pushed the ziga-improve-wsi-log-msg branch from 7385242 to 01d7edb Compare January 13, 2025 13:56
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 343622.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18625 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18625 passed.

@spencer-lunarg spencer-lunarg merged commit 01808b6 into KhronosGroup:main Jan 13, 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.

3 participants