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

Fixing vendor-specific/tied memory property flag detection #771

Merged

Conversation

toomuchvoltage
Copy link
Contributor

Fixing vendor-specific/tied memory property flag detection for vkloadtests's VMA wrapper.

@toomuchvoltage
Copy link
Contributor Author

Dear @MarkCallow , this should fix the reported issue. Though would appreciate a quick smoke test on the M2 device as well just to be sure.

@MarkCallow
Copy link
Collaborator

Though would appreciate a quick smoke test on the M2 device

It works.

@MarkCallow
Copy link
Collaborator

For orthogonality please add `ktxTexture[12]_VkUploadEx_WithSuballocator wrappers to vkloader.c. I hadn't noticed they weren't there in the previous PR.

@toomuchvoltage
Copy link
Contributor Author

For orthogonality please add `ktxTexture[12]_VkUploadEx_WithSuballocator wrappers to vkloader.c. I hadn't noticed they weren't there in the previous PR.

All done! Let me know if this suffices. 🙏

@MarkCallow MarkCallow merged commit a100217 into KhronosGroup:main Sep 17, 2023
14 checks passed
@MarkCallow
Copy link
Collaborator

Thanks again.

@toomuchvoltage
Copy link
Contributor Author

Thanks again.

You're very welcome Mark. I feel privileged to have been able to contribute.

@toomuchvoltage toomuchvoltage deleted the vkloadtests-suballocator-fix branch September 18, 2023 02:10
KaperD pushed a commit to KaperD/KTX-Software that referenced this pull request Feb 21, 2024
KaperD pushed a commit to KaperD/KTX-Software that referenced this pull request Feb 22, 2024
KaperD pushed a commit to KaperD/KTX-Software that referenced this pull request Feb 22, 2024
KaperD pushed a commit to KaperD/KTX-Software that referenced this pull request Feb 22, 2024
KaperD pushed a commit to KaperD/KTX-Software that referenced this pull request Feb 22, 2024
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.

2 participants