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.

Sign up for GitHub

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

Jump to bottom

sync: Update renderpass layout transition messages #9540

Merged
merged 1 commit into from
Feb 28, 2025
Merged

sync: Update renderpass layout transition messages #9540

merged 1 commit into from
Feb 28, 2025

Conversation

artem-lunarg
Copy link
Contributor

vkCmdBeginRenderPass(): WRITE_AFTER_READ hazard detected. vkCmdBeginRenderPass performs image layout transition in subpass 0 on attachment 0 (VkImageView 0xcfef35000000000a, VkImage 0xf56c9b0000000004, oldLayout VK_IMAGE_LAYOUT_UNDEFINED, newLayout VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL), which was previously read by vkCmdCopyImage.
No sufficient synchronization is present to ensure that a layout transition does not conflict with a prior read (VK_ACCESS_2_TRANSFER_READ_BIT) at VK_PIPELINE_STAGE_2_COPY_BIT. An execution dependency is sufficient to prevent this hazard.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 381958.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19211 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19211 passed.

@artem-lunarg artem-lunarg marked this pull request as ready for review February 26, 2025 23:46
@artem-lunarg artem-lunarg requested a review from a team as a code owner February 26, 2025 23:46
@artem-lunarg artem-lunarg force-pushed the artem-sync-renderpass-layout-transition-messages branch from c9d5ece to ee05dcb Compare February 27, 2025 13:35
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 382425.

@artem-lunarg artem-lunarg changed the title sync: Update renderpass layout transition messages. Part 1 sync: Update renderpass layout transition messages Feb 27, 2025
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19212 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19212 passed.

@artem-lunarg artem-lunarg mentioned this pull request Feb 27, 2025
Merged
@artem-lunarg artem-lunarg force-pushed the artem-sync-renderpass-layout-transition-messages branch from ee05dcb to 385d694 Compare February 28, 2025 08:14
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 382956.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19215 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19215 passed.

@artem-lunarg artem-lunarg merged commit c7ee7cd into KhronosGroup:main Feb 28, 2025
21 checks passed
@artem-lunarg artem-lunarg deleted the artem-sync-renderpass-layout-transition-messages branch February 28, 2025 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spencer-lunarg spencer-lunarg spencer-lunarg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@artem-lunarg @ci-tester-lunarg @spencer-lunarg