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

layers: Improve Fragment Density Map Offset messages #9510

Merged

Conversation

spencer-lunarg
Copy link
Contributor

This takes @ziga-lunarg tests (#9506) and improves the error messages for them

@spencer-lunarg spencer-lunarg requested a review from a team as a code owner February 23, 2025 15:40
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 378515.

"VkPhysicalDeviceFragmentDensityMapOffsetPropertiesQCOM": {},
"VkPhysicalDeviceFragmentDensityMapOffsetPropertiesQCOM": {
"fragmentDensityOffsetGranularity": {
"width": 8,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ziga-lunarg this is why the tests were failing, these were returning back 0, 0


if (fdm_non_zero_offsets && !has_offset_flag) {
const LogObjectList objlist(cb_state.Handle(), rp_state.Handle(), view_state->Handle());
skip |= LogError("VUID-VkFramebufferCreateInfo-renderPass-06502", objlist,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

new message

vkCmdEndRenderPass2KHR(): pSubpassEndInfo->pNext.pFragmentDensityOffsets[0] is x = 8, y = 8 (non-zero offsets) but pAttachments[0] was not created with VK_IMAGE_CREATE_FRAGMENT_DENSITY_MAP_OFFSET_BIT_QCOM.

(prints the actual offsets now)

if ((attachment != VK_ATTACHMENT_UNUSED) && (attachment == i) && !has_offset_flag) {
const LogObjectList objlist(cb_state.Handle(), rp_state.Handle(), view_state->Handle());
skip |=
LogError("VUID-VkSubpassFragmentDensityMapOffsetEndInfoQCOM-pInputAttachments-06506", objlist, offset_count_loc,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

new message

vkCmdEndRenderPass2KHR(): pSubpassEndInfo->pNext.fragmentDensityOffsetCount is 1 but pInputAttachments[0] (pAttachments[1] VkImageView 0xee647e0000000009) underlying VkImage 0xcb3ee80000000007 was not created with VK_IMAGE_CREATE_FRAGMENT_DENSITY_MAP_OFFSET_BIT_QCOM.

Helps connect which pInputAttachments to which renderpass pAttachments with some handles as well

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19164 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 19164 failed.

@spencer-lunarg
Copy link
Contributor Author

CI Vulkan-ValidationLayers build # 19164 failed.

Network issue with windows machine, passed internally on Android

@spencer-lunarg spencer-lunarg merged commit fc57881 into KhronosGroup:main Feb 23, 2025
20 of 21 checks passed
@spencer-lunarg spencer-lunarg deleted the spencer-lunarg-fdm-are-dense branch February 23, 2025 16:55
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