Skip to content

layers: Add 11845#11996

Merged
spencer-lunarg merged 1 commit intoKhronosGroup:mainfrom
arno-lunarg:arno-rt-11845
Mar 30, 2026
Merged

layers: Add 11845#11996
spencer-lunarg merged 1 commit intoKhronosGroup:mainfrom
arno-lunarg:arno-rt-11845

Conversation

@arno-lunarg
Copy link
Copy Markdown
Contributor

No description provided.

@arno-lunarg arno-lunarg requested a review from a team as a code owner March 27, 2026 15:57
@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 692341.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22974 running.

"VUID-VkMemoryDedicatedAllocateInfoTensorARM-tensor-09859",

// VUID-VkDeviceAddress-size-11364 is used instead
"VUID-vkCmdBuildAccelerationStructuresKHR-pInfos-11845"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I agree with this, so why I am approving, but this has been a constant issue with the spec,

Before we just approve and forget, I want to grasp where this is going wrong

VUID-vkBuildAccelerationStructuresKHR-pInfos-11820 is the matching version for host device,

also VUID-vkCmdBuildAccelerationStructuresIndirectKHR-pInfos-11845 is the same as well

... maybe just make this

    // These are caught by VUID-VkDeviceAddress-size-11364 already
    // TODO - these should be removed from the spec and they are redundant
    "VUID-vkCmdBuildAccelerationStructuresKHR-pInfos-11845",
    "VUID-vkCmdBuildAccelerationStructuresIndirectKHR-pInfos-11845",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think both of these are "wrong" because nvidia does not support host builds (VUID-vkCmdBuildAccelerationStructuresIndirectKHR-pInfos-11845) nor indirect builds (vkCmdBuildAccelerationStructuresIndirectKHR) so they don't really make sense

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updating this file

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 692356.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22975 running.

// VUID-VkDeviceAddress-size-11364 is used instead
"VUID-vkCmdBuildAccelerationStructuresKHR-pInfos-11845",
// TODO - these should be removed from the spec
"VUID-vkCmdBuildAccelerationStructuresKHR-pInfos-11845",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  1. Did you mean VUID-vkBuildAccelerationStructuresKHR-pInfos-11820?
  2. leave the "reason" why in the comments here (basically we describe it well in layers: Add 11845 #11996 (comment) and it is getting lost from here)

Comment thread layers/error_message/unimplementable_validation.h
@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22975 passed.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 694239.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22987 running.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22987 passed.

@spencer-lunarg spencer-lunarg merged commit 9fc498c into KhronosGroup:main Mar 30, 2026
20 checks passed
@arno-lunarg arno-lunarg deleted the arno-rt-11845 branch April 1, 2026 08:17
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.

3 participants