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

Fix tracking queue family properties #1810

Merged

Conversation

ziga-lunarg
Copy link
Contributor

Ran into a case where an app did:
vkGetPhysicalDeviceQueueFamilyProperties() to get the count
vkGetPhysicalDeviceQueueFamilyProperties() with the count
vkGetPhysicalDeviceQueueFamilyProperties() with 1

And we just stored the 1.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 278563.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5064 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5064 passed.

@bradgrantham-lunarg
Copy link
Contributor

@MarkY-LunarG if I reviewed this and it looks good, are you okay with @ziga-lunarg merging it or do you want a chance to review?

@MarkY-LunarG
Copy link
Contributor

I'm ok with this. This might be something we have to do in other API entrypoints (like EnumPhysDevs). But let's start with this for now.

Copy link
Contributor

@MarkY-LunarG MarkY-LunarG left a comment

Choose a reason for hiding this comment

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

Just put a comment around each instance explaining why you're doing it. No ones going to see the commit and know why just by looking at the code.

@ziga-lunarg ziga-lunarg force-pushed the ziga-fix-tracking-queue-family-properties branch from 3b94abb to a663a1b Compare October 15, 2024 21:52
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 279427.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5081 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5081 passed.

@ziga-lunarg ziga-lunarg merged commit ba579f0 into LunarG:dev Oct 15, 2024
9 checks passed
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.

4 participants