mirrors/monado
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/monado/monado.git synced 2024-12-28 02:26:16 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

a/vk: Add/update comments

Browse source 
Part-of: <https://gitlab.freedesktop.org/monado/monado/-/merge_requests/2332>
This commit is contained in:
Rylie Pavlik 2024-11-05 12:41:08 -06:00
parent 869054403f
commit 7a4ba078d9
1 changed files with 14 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/xrt/auxiliary/vk/vk_helpers.c
View file

@ -1063,6 +1063,13 @@ err_image:
return ret;
}
// - vk_csci_get_image_external_handle_type (usually but not always a constant)
// - vk_csci_get_image_external_support
// - vkGetPhysicalDeviceImageFormatProperties2
// - vkCreateImage
// - vkGetImageMemoryRequirements
// - maybe vkGetAndroidHardwareBufferPropertiesANDROID
// - vk_alloc_and_bind_image_memory
XRT_CHECK_RESULT VkResult
vk_create_image_from_native(struct vk_bundle *vk,
const struct xrt_swapchain_create_info *info,
@ -1078,13 +1085,13 @@ vk_create_image_from_native(struct vk_bundle *vk,
#ifdef XRT_GRAPHICS_BUFFER_HANDLE_IS_AHARDWAREBUFFER
/*
* Some Vulkan drivers will natively support importing and exporting
* SRGB formats (Qualcomm Adreno) even tho technically that's not intended
* by the AHardwareBuffer since they don't support sRGB formats.
* While others (arm Mali) does not support importing and exporting sRGB
* formats. So we need to create the image without sRGB and then create
* the image views with sRGB which is allowed by the Vulkan spec. It
* seems to be safe to do with on all drivers, so to reduce the logic
* do that instead.
* SRGB formats (Qualcomm Adreno) even though technically the
* AHardwareBuffer support for sRGB is... awkward (not inherent).
* While others (arm Mali) does not support importing and exporting
* sRGB formats. So we need to create the image without sRGB and
* then create the image views with sRGB which is allowed by the
* Vulkan spec. It seems to be safe to do with on all drivers,
* so to reduce the logic do that instead.
*/
if (image_format == VK_FORMAT_R8G8B8A8_SRGB) {
image_format = VK_FORMAT_R8G8B8A8_UNORM;