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 attachment loadOp vs layout transition message #9539

Merged
merged 1 commit into from
Feb 26, 2025
Merged

sync: Update attachment loadOp vs layout transition message #9539

merged 1 commit into from
Feb 26, 2025

Conversation

artem-lunarg
Copy link
Contributor

vkCmdBeginRenderPass(): WRITE_AFTER_WRITE hazard detected: attachment loadOp access is not synchronized with the attachment layout transition. vkCmdBeginRenderPass clears the color aspect of attachment 0 in subpass 0 (VkImageView 0x967dd1000000000e, loadOp VK_ATTACHMENT_LOAD_OP_CLEAR), which was previously written during an image layout transition initiated by the same command.
The current synchronization allows VK_ACCESS_2_INPUT_ATTACHMENT_READ_BIT|VK_ACCESS_2_SHADER_READ_BIT accesses at VK_PIPELINE_STAGE_2_FRAGMENT_SHADER_BIT, but to prevent this hazard, it must allow VK_ACCESS_2_COLOR_ATTACHMENT_WRITE_BIT accesses at VK_PIPELINE_STAGE_2_COLOR_ATTACHMENT_OUTPUT_BIT.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 381669.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19209 running.

@artem-lunarg artem-lunarg marked this pull request as ready for review February 26, 2025 16:54
@artem-lunarg artem-lunarg requested a review from a team as a code owner February 26, 2025 16:54
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19209 passed.

@artem-lunarg artem-lunarg merged commit 4da1db9 into KhronosGroup:main Feb 26, 2025
21 checks passed
@artem-lunarg artem-lunarg deleted the artem-sync-loadop-vs-layout-transition-message branch February 26, 2025 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arno-lunarg arno-lunarg arno-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 @arno-lunarg