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

tests: Remove invalid positive array layer tests #9006

Merged

Conversation

spencer-lunarg
Copy link
Contributor

closes #1998

from #8982

Was discussed in https://gitlab.khronos.org/vulkan/vulkan/-/issues/4110 that all image array layers in a single ImageView must be valid if any part of the descriptor is accessed. While this might not effect drivers that don't care about Image Layout, those that do care said this likely will break and the advice to either

  1. Full transition the entire Image View where doing a sub-transition
  2. Create a separate Image Views for each arrayLayer and use a descriptor indexing method on them (since things like Descriptor Indexing were designed to solve being able to have this dynamic access)
  3. For more future things like VK_EXT_descriptor_buffer there is actually a descriptorBufferImageLayoutIgnored feature that was designed for things like this

cc @Ralith

@spencer-lunarg spencer-lunarg requested a review from a team as a code owner December 12, 2024 02:12
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 322364.

@@ -93,79 +93,3 @@ TEST_F(PositiveGpuAVImageLayout, DescriptorArrayLayout) {
m_default_queue->Submit(m_command_buffer);
m_default_queue->Wait();
}

// Waiting to hear from https://gitlab.khronos.org/vulkan/vulkan/-/issues/4110
TEST_F(PositiveGpuAVImageLayout, DISABLED_MultiArrayLayers) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we have negative variation of the tests already when these were added

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18297 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18297 passed.

@spencer-lunarg spencer-lunarg merged commit 828d51a into KhronosGroup:main Dec 12, 2024
21 checks passed
@spencer-lunarg spencer-lunarg deleted the spencer-lunarg-array-tests-2 branch December 12, 2024 05:05
@Ralith
Copy link
Contributor

Ralith commented Dec 12, 2024

Was discussed in https://gitlab.khronos.org/vulkan/vulkan/-/issues/4110 that all image array layers in a single ImageView must be valid if any part of the descriptor is accessed

Where is this specified? What is the exact, formal, requirement? At face value, this seems to make it impossible to stream layers in an array texture concurrently with rendering, which is a painful constraint.

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.

UNASSIGNED-CoreValidation-DrawState-InvalidImageLayout false positive with array textures
4 participants