KhronosGroup / KhronosGroup/Vulkan-ExtensionLayer

synchronization2 layer doesn't implement descriptor updates

Open
#131 0 comments 0 reactions 1 assignee View on GitHub

@jeremyg-lunarg is already working on this.

Since May 8, 2023.

Dominant language
C
Stars
172
Forks
57
Avg merge
59m
Merged PRs (30d)
5

Description

It's possible to write a sampled image descriptor which expects the image subresource to be in VK_IMAGE_LAYOUT_READ_ONLY_OPTIMAL, which was introduced in sync2. The sync2 layer doesn't implement vkUpdateDescriptorSets and other descriptor update functions and an invalid value will just be passed further down the dispatch chain.

The workaround is to not use sync2 layouts when an image subresource will be accessed through a descriptor.

A possible fix would need to implement vkCreateImageView so that it can tell the aspect bits when writing the descriptor. Another simpler, but possibly worse-performing, fix would map all read-only layouts to general.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.