From a06e15bee6e74732312772f65679cb0fea152e3a Mon Sep 17 00:00:00 2001 From: Slawomir Cygan Date: Wed, 5 Oct 2016 18:38:02 +0200 Subject: loader: gh1014- Fix physical device dispatch table Instead of initializing dispatch tabled for up to *pPhysicalDeviceCount devices, initialize always all dispatch tables. This fixes corruption _after_ VulkanCTS dEQP-VK.api.info.instance.physical_devices test. Change-Id: I29ff42456bed9acb6d2c4016fe7f1940b7f4bd3b --- loader/loader.c | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) (limited to 'loader') diff --git a/loader/loader.c b/loader/loader.c index f2ebb58e..a950ea1d 100644 --- a/loader/loader.c +++ b/loader/loader.c @@ -4449,30 +4449,31 @@ terminator_EnumeratePhysicalDevices(VkInstance instance, inst->phys_devs_term = NULL; } - if (copy_count > 0) { + if (inst->total_gpu_count > 0) { inst->phys_devs_term = loader_instance_heap_alloc( - inst, sizeof(struct loader_physical_device) * copy_count, + inst, sizeof(struct loader_physical_device) * inst->total_gpu_count, VK_SYSTEM_ALLOCATION_SCOPE_INSTANCE); if (!inst->phys_devs_term) { return VK_ERROR_OUT_OF_HOST_MEMORY; } } - for (i = 0; idx < copy_count && i < inst->total_icd_count; i++) { - for (j = 0; j < phys_devs[i].count && idx < copy_count; j++) { + for (i = 0; idx < inst->total_gpu_count && i < inst->total_icd_count; i++) { + for (j = 0; j < phys_devs[i].count && idx < inst->total_gpu_count; j++) { loader_set_dispatch((void *)&inst->phys_devs_term[idx], inst->disp); inst->phys_devs_term[idx].this_icd = phys_devs[i].this_icd; inst->phys_devs_term[idx].icd_index = i; inst->phys_devs_term[idx].phys_dev = phys_devs[i].phys_devs[j]; - pPhysicalDevices[idx] = - (VkPhysicalDevice)&inst->phys_devs_term[idx]; + if (idx < copy_count) { + pPhysicalDevices[idx] = + (VkPhysicalDevice)&inst->phys_devs_term[idx]; + } idx++; } } if (copy_count < inst->total_gpu_count) { - inst->total_gpu_count = copy_count; res = VK_INCOMPLETE; } } -- cgit v1.2.3