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

Print differences in supported extensions and features #1221

Merged
merged 2 commits into from
Aug 28, 2023

Conversation

panos-lunarg
Copy link
Contributor

In case vkCreateInstance and vkCreateDevice fail at replay time because of an EXTENSION_NOT_PRESENT error, should now print which were the extensions that were not supported on the replay gpu and triggered the error.

Similarly features requested with vkCreateDevice during capture will be compared with what the replay device offers and print a warning for each feature not supported.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 19802.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3060 running.

}
else
{
GFXRECON_LOG_WARNING("Feature storageBuffer16BitAccess requested at catpure is not supported by the replay device");
Copy link
Contributor

Choose a reason for hiding this comment

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

"catpure" - I love this typo ;-)

I would prepare part of the string in advance and save quite a few conditionals here, would make code easier to read.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops missed your comment. Will fix

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3060 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 20000.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3061 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3061 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 20278.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3062 running.


if (remove_unsupported)
{
const_cast<VkPhysicalDevice16BitStorageFeatures*>(currentNext)->storageBuffer16BitAccess = VK_FALSE;
Copy link
Contributor

Choose a reason for hiding this comment

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

If you want to save even more lines of code, you could do "... = !remove_unsupported;" here instead of the if clause :-)

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3062 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 21035.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3071 running.

@panos-lunarg
Copy link
Contributor Author

panos-lunarg commented Aug 10, 2023

I realized that RemoveIgnorableExtensions already implements the logic which prints the unsupported extensions both for vkCreateInstance and vkCreateDevice, so i removed that code from my patch. The unsupported features are not listed though

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3071 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 29575.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3127 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3127 passed.

panos-lunarg and others added 2 commits August 25, 2023 08:56
vkCreateDevice can fail with FEATURE_NOT_PRESENT error when
features were requested during capture that are not present at the
replay device. In this case the features that are not supported will be
printed as a warning message.
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 30087.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3132 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3132 passed.

@panos-lunarg panos-lunarg merged commit 8a5f3d5 into LunarG:dev Aug 28, 2023
5 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