From e7bf86e164e88409324fa2c8b41e0775f9d48501 Mon Sep 17 00:00:00 2001 From: Mark Young Date: Mon, 7 Nov 2016 13:27:02 -0700 Subject: loader: gh1120/gh1134 - Object wrapping issues First issue was that we needed to override vkGetDeviceProcAddr. Instead of allowing this to always go directly to the ICD, we needed to intercept a few commands because they require a loader trampoline and terminator call. Most commands still return a pointer directly to ICD command. GH1120 - Unwrap both the physical device handles and the KHR_surface handles in the loader during both the trampoline and terminator calls for DebugMarker commands. This has to be done since the values given to an application are the loader trampoline versions, and the values given to the last layer is the loader terminator versions. GH1134 - We were passing down the wrong device object to the ICD functions when querying the ICD command function address and comparing it in the override functions. Thanks to Baldur (Mr. Renderdoc) for discovering this, testing my fixes, and resolving several bugs. Change-Id: I7618d71ffee6c53d9842758210a9261f6b3a1797 --- loader/extensions.c | 107 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 107 insertions(+) (limited to 'loader/extensions.c') diff --git a/loader/extensions.c b/loader/extensions.c index 9b59eb5f..643911dd 100644 --- a/loader/extensions.c +++ b/loader/extensions.c @@ -26,6 +26,100 @@ #include "loader.h" #include "extensions.h" #include +#include "wsi.h" + +// Definitions for the VK_EXT_debug_marker extension commands which +// need to have a terminator function + +VKAPI_ATTR VkResult VKAPI_CALL vkDebugMarkerSetObjectTagEXT( + VkDevice device, VkDebugMarkerObjectTagInfoEXT *pTagInfo) { + const VkLayerDispatchTable *disp = loader_get_dispatch(device); + // If this is a physical device, we have to replace it with the proper one + // for the next call. + if (pTagInfo->objectType == + VK_DEBUG_REPORT_OBJECT_TYPE_PHYSICAL_DEVICE_EXT) { + struct loader_physical_device_tramp *phys_dev_tramp = + (struct loader_physical_device_tramp *)pTagInfo->object; + pTagInfo->object = (uint64_t)phys_dev_tramp->phys_dev; + } + return disp->DebugMarkerSetObjectTagEXT(device, pTagInfo); +} + +VKAPI_ATTR VkResult VKAPI_CALL terminator_DebugMarkerSetObjectTagEXT( + VkDevice device, VkDebugMarkerObjectTagInfoEXT *pTagInfo) { + uint32_t icd_index = 0; + struct loader_device *dev; + struct loader_icd_term *icd_term = + loader_get_icd_and_device(device, &dev, &icd_index); + // If this is a physical device, we have to replace it with the proper one + // for the next call. + if (pTagInfo->objectType == + VK_DEBUG_REPORT_OBJECT_TYPE_PHYSICAL_DEVICE_EXT) { + struct loader_physical_device_term *phys_dev_term = + (struct loader_physical_device_term *)pTagInfo->object; + pTagInfo->object = (uint64_t)phys_dev_term->phys_dev; + + // If this is a KHR_surface, and the ICD has created its own, we have to + // replace it with the proper one for the next call. + } else if (pTagInfo->objectType == + VK_DEBUG_REPORT_OBJECT_TYPE_SURFACE_KHR_EXT) { + if (NULL != icd_term && NULL != icd_term->CreateSwapchainKHR) { + VkIcdSurface *icd_surface = (VkIcdSurface *)(pTagInfo->object); + if (NULL != icd_surface->real_icd_surfaces) { + pTagInfo->object = + (uint64_t)icd_surface->real_icd_surfaces[icd_index]; + } + } + } + assert(icd_term != NULL && icd_term->DebugMarkerSetObjectTagEXT && + "loader: null DebugMarkerSetObjectTagEXT ICD pointer"); + return icd_term->DebugMarkerSetObjectTagEXT(device, pTagInfo); +} + +VKAPI_ATTR VkResult VKAPI_CALL vkDebugMarkerSetObjectNameEXT( + VkDevice device, VkDebugMarkerObjectNameInfoEXT *pNameInfo) { + const VkLayerDispatchTable *disp = loader_get_dispatch(device); + // If this is a physical device, we have to replace it with the proper one + // for the next call. + if (pNameInfo->objectType == + VK_DEBUG_REPORT_OBJECT_TYPE_PHYSICAL_DEVICE_EXT) { + struct loader_physical_device_tramp *phys_dev_tramp = + (struct loader_physical_device_tramp *)pNameInfo->object; + pNameInfo->object = (uint64_t)phys_dev_tramp->phys_dev; + } + return disp->DebugMarkerSetObjectNameEXT(device, pNameInfo); +} + +VKAPI_ATTR VkResult VKAPI_CALL terminator_DebugMarkerSetObjectNameEXT( + VkDevice device, VkDebugMarkerObjectNameInfoEXT *pNameInfo) { + uint32_t icd_index = 0; + struct loader_device *dev; + struct loader_icd_term *icd_term = + loader_get_icd_and_device(device, &dev, &icd_index); + // If this is a physical device, we have to replace it with the proper one + // for the next call. + if (pNameInfo->objectType == + VK_DEBUG_REPORT_OBJECT_TYPE_PHYSICAL_DEVICE_EXT) { + struct loader_physical_device_term *phys_dev_term = + (struct loader_physical_device_term *)pNameInfo->object; + pNameInfo->object = (uint64_t)phys_dev_term->phys_dev; + + // If this is a KHR_surface, and the ICD has created its own, we have to + // replace it with the proper one for the next call. + } else if (pNameInfo->objectType == + VK_DEBUG_REPORT_OBJECT_TYPE_SURFACE_KHR_EXT) { + if (NULL != icd_term && NULL != icd_term->CreateSwapchainKHR) { + VkIcdSurface *icd_surface = (VkIcdSurface *)(pNameInfo->object); + if (NULL != icd_surface->real_icd_surfaces) { + pNameInfo->object = + (uint64_t)icd_surface->real_icd_surfaces[icd_index]; + } + } + } + assert(icd_term != NULL && icd_term->DebugMarkerSetObjectNameEXT && + "loader: null DebugMarkerSetObjectNameEXT ICD pointer"); + return icd_term->DebugMarkerSetObjectNameEXT(device, pNameInfo); +} // Definitions for the VK_NV_external_memory_capabilities extension @@ -118,6 +212,19 @@ bool extension_instance_gpa(struct loader_instance *ptr_instance, const char *name, void **addr) { *addr = NULL; + // Definitions for the VK_EXT_debug_marker extension commands which + // need to have a terminator function. Since these are device + // commands, we always need to return a valid value for them. + + if (!strcmp("vkDebugMarkerSetObjectTagEXT", name)) { + *addr = (void *)vkDebugMarkerSetObjectTagEXT; + return true; + } + if (!strcmp("vkDebugMarkerSetObjectNameEXT", name)) { + *addr = (void *)vkDebugMarkerSetObjectNameEXT; + return true; + } + // Functions for the VK_NV_external_memory_capabilities extension if (!strcmp("vkGetPhysicalDeviceExternalImageFormatPropertiesNV", name)) { -- cgit v1.2.3