Skip to content

gpuav: Fix validation#11986

Merged
arno-lunarg merged 2 commits intoKhronosGroup:mainfrom
arno-lunarg:arno-gpuav-fix-validation
Mar 27, 2026
Merged

gpuav: Fix validation#11986
arno-lunarg merged 2 commits intoKhronosGroup:mainfrom
arno-lunarg:arno-gpuav-fix-validation

Conversation

@arno-lunarg
Copy link
Copy Markdown
Contributor

@arno-lunarg arno-lunarg commented Mar 26, 2026

When picking a buffer from a device address range, filter by buffer usage AND buffer size

2nd commit closes #11879

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

CI Vulkan-ValidationLayers build queued with queue ID 690978.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22959 running.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22959 failed.

When picking a buffer from a device address range, filter by
buffer usage AND buffer size
@arno-lunarg arno-lunarg force-pushed the arno-gpuav-fix-validation branch from 8511c18 to f7add88 Compare March 26, 2026 13:51
@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 691027.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22960 running.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22960 aborted.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 691417.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22963 running.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22963 passed.


using BufferFilter = stdext::inplace_function<bool(vvl::Buffer*)>;
small_vector<vvl::Buffer*, 2> GetBuffersByAddressRange(const VkDeviceAddressRangeKHR& address_range,
const BufferFilter& filter) const;
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.

whey not just pass in a VkBufferUsageFlags2 instead of making this callback?

The main reason is this is implying there is another way to filter the buffers, but it should only be from the usage flag

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 can do that no problem

@arno-lunarg arno-lunarg force-pushed the arno-gpuav-fix-validation branch from 22e663b to a67dd5c Compare March 27, 2026 13:53
@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 692180.

Account for functions added with device address commands
@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22972 running.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22972 passed.

@arno-lunarg arno-lunarg merged commit a615461 into KhronosGroup:main Mar 27, 2026
20 checks passed
@arno-lunarg arno-lunarg deleted the arno-gpuav-fix-validation branch March 27, 2026 15:37
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.

Fix VK_KHR_device_address_command GPU-AV support

3 participants