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: Validate binary sync depenency on timeline wait #8739

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

artem-lunarg
Copy link
Contributor

@artem-lunarg artem-lunarg commented Oct 22, 2024

This partially addresses #8707

What's left for other PRs:

  • Provide different messages depending on which condition triggered VUs like VUID-vkQueueSubmit-pWaitSemaphores-03238: either missing binary signal or binary signal exists but depends on timeline wait.
  • Try to analyze entire synchronization chain, not only direct parent. - actually this works, added one more test to show this NegativeSyncObject.BinarySyncDependsOnTimelineWait3.

This validates "and any semaphore signal operations on which it depends
must have also been submitted for execution" from these VUIDs:
VUID-vkQueueSubmit-pWaitSemaphores-03238
VUID-vkQueueSubmit2-semaphore-03873
VUID-vkQueuePresentKHR-pWaitSemaphores-03268
VUID-VkSemaphoreGetFdInfoKHR-handleType-03254
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 284488.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17799 running.

@artem-lunarg artem-lunarg changed the title layers: Test binary sync depenency on timeline wait layers: Validate binary sync depenency on timeline wait Oct 22, 2024
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 284550.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17800 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17800 aborted.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 284712.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17805 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 17805 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.

2 participants