Skip to content

Commit

Permalink
build: Update to header 1.3.268
Browse files Browse the repository at this point in the history
- Update known-good
- Replace VUID-00378 with consolidated version VUID-05137
- Generate source
  • Loading branch information
mikes-lunarg authored and spencer-lunarg committed Oct 15, 2023
1 parent 7f2a9db commit 0d09c6f
Show file tree
Hide file tree
Showing 9 changed files with 90 additions and 74 deletions.
2 changes: 1 addition & 1 deletion layers/VkLayer_khronos_validation.json.in
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"name": "VK_LAYER_KHRONOS_validation",
"type": "GLOBAL",
"library_path": "@JSON_LIBRARY_PATH@",
"api_version": "1.3.267",
"api_version": "1.3.268",
"implementation_version": "1",
"description": "Khronos Validation Layer",
"introduction": "The main, comprehensive Khronos validation layer.\n\nVulkan is an Explicit API, enabling direct control over how GPUs actually work. By design, minimal error checking is done inside a Vulkan driver. Applications have full control and responsibility for correct operation. Any errors in how Vulkan is used can result in a crash. \n\nThe Khronos Valiation Layer can be enabled to assist development by enabling developers to verify their applications correctly use the Vulkan API.",
Expand Down
2 changes: 1 addition & 1 deletion layers/vulkan/generated/enum_flag_bits.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
#include <array>
#include "vulkan/vulkan.h"
// clang-format off
const uint32_t GeneratedVulkanHeaderVersion = 267;
const uint32_t GeneratedVulkanHeaderVersion = 268;
const VkAccessFlags AllVkAccessFlagBits = VK_ACCESS_INDIRECT_COMMAND_READ_BIT|VK_ACCESS_INDEX_READ_BIT|VK_ACCESS_VERTEX_ATTRIBUTE_READ_BIT|VK_ACCESS_UNIFORM_READ_BIT|VK_ACCESS_INPUT_ATTACHMENT_READ_BIT|VK_ACCESS_SHADER_READ_BIT|VK_ACCESS_SHADER_WRITE_BIT|VK_ACCESS_COLOR_ATTACHMENT_READ_BIT|VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT|VK_ACCESS_DEPTH_STENCIL_ATTACHMENT_READ_BIT|VK_ACCESS_DEPTH_STENCIL_ATTACHMENT_WRITE_BIT|VK_ACCESS_TRANSFER_READ_BIT|VK_ACCESS_TRANSFER_WRITE_BIT|VK_ACCESS_HOST_READ_BIT|VK_ACCESS_HOST_WRITE_BIT|VK_ACCESS_MEMORY_READ_BIT|VK_ACCESS_MEMORY_WRITE_BIT|VK_ACCESS_NONE|VK_ACCESS_TRANSFORM_FEEDBACK_WRITE_BIT_EXT|VK_ACCESS_TRANSFORM_FEEDBACK_COUNTER_READ_BIT_EXT|VK_ACCESS_TRANSFORM_FEEDBACK_COUNTER_WRITE_BIT_EXT|VK_ACCESS_CONDITIONAL_RENDERING_READ_BIT_EXT|VK_ACCESS_COLOR_ATTACHMENT_READ_NONCOHERENT_BIT_EXT|VK_ACCESS_ACCELERATION_STRUCTURE_READ_BIT_KHR|VK_ACCESS_ACCELERATION_STRUCTURE_WRITE_BIT_KHR|VK_ACCESS_FRAGMENT_DENSITY_MAP_READ_BIT_EXT|VK_ACCESS_FRAGMENT_SHADING_RATE_ATTACHMENT_READ_BIT_KHR|VK_ACCESS_COMMAND_PREPROCESS_READ_BIT_NV|VK_ACCESS_COMMAND_PREPROCESS_WRITE_BIT_NV;
const VkImageAspectFlags AllVkImageAspectFlagBits = VK_IMAGE_ASPECT_COLOR_BIT|VK_IMAGE_ASPECT_DEPTH_BIT|VK_IMAGE_ASPECT_STENCIL_BIT|VK_IMAGE_ASPECT_METADATA_BIT|VK_IMAGE_ASPECT_PLANE_0_BIT|VK_IMAGE_ASPECT_PLANE_1_BIT|VK_IMAGE_ASPECT_PLANE_2_BIT|VK_IMAGE_ASPECT_NONE|VK_IMAGE_ASPECT_MEMORY_PLANE_0_BIT_EXT|VK_IMAGE_ASPECT_MEMORY_PLANE_1_BIT_EXT|VK_IMAGE_ASPECT_MEMORY_PLANE_2_BIT_EXT|VK_IMAGE_ASPECT_MEMORY_PLANE_3_BIT_EXT;
const VkInstanceCreateFlags AllVkInstanceCreateFlagBits = VK_INSTANCE_CREATE_ENUMERATE_PORTABILITY_BIT_KHR;
Expand Down
2 changes: 1 addition & 1 deletion layers/vulkan/generated/object_tracker.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ bool ObjectLifetimes::ReportUndestroyedInstanceObjects(VkInstance instance, cons

bool ObjectLifetimes::ReportUndestroyedDeviceObjects(VkDevice device, const Location& loc) const {
bool skip = false;
const std::string error_code = "VUID-vkDestroyDevice-device-00378";
const std::string error_code = "VUID-vkDestroyDevice-device-05137";
skip |= ReportLeakedDeviceObjects(device, kVulkanObjectTypeCommandBuffer, error_code, loc);
skip |= ReportLeakedDeviceObjects(device, kVulkanObjectTypeBuffer, error_code, loc);
skip |= ReportLeakedDeviceObjects(device, kVulkanObjectTypeImage, error_code, loc);
Expand Down
140 changes: 78 additions & 62 deletions layers/vulkan/generated/vk_validation_error_messages.h

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion scripts/generators/object_tracker_generator.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ def getUndestroyedObjectVUID(targetApiName: str, scope: str) -> str:
case 'vulkan':
per_scope = {
'instance': 'VUID-vkDestroyInstance-instance-00629',
'device': 'VUID-vkDestroyDevice-device-00378'
'device': 'VUID-vkDestroyDevice-device-05137'
}

return per_scope[scope]
Expand Down
4 changes: 2 additions & 2 deletions scripts/known_good.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,15 +21,15 @@
"sub_dir": "Vulkan-Headers",
"build_dir": "Vulkan-Headers/build",
"install_dir": "Vulkan-Headers/build/install",
"commit": "v1.3.267"
"commit": "v1.3.268"
},
{
"name": "Vulkan-Utility-Libraries",
"url": "https://github.com/KhronosGroup/Vulkan-Utility-Libraries.git",
"sub_dir": "Vulkan-Utility-Libraries",
"build_dir": "Vulkan-Utility-Libraries/build",
"install_dir": "Vulkan-Utility-Libraries/build/install",
"commit": "v1.3.267",
"commit": "v1.3.268",
"deps": [
{
"var_name": "VULKAN_HEADERS_INSTALL_DIR",
Expand Down
2 changes: 1 addition & 1 deletion tests/layers/VkLayer_device_profile_api.json.in
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"name": "VK_LAYER_LUNARG_device_profile_api",
"type": "GLOBAL",
"library_path": "@JSON_LIBRARY_PATH@",
"api_version": "1.3.267",
"api_version": "1.3.268",
"implementation_version": "2",
"description": "LunarG Device Profile Api Layer",
"device_extensions": [
Expand Down
8 changes: 4 additions & 4 deletions tests/unit/others.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1198,13 +1198,13 @@ TEST_F(VkLayerTest, LeakAnObject) {
VkFence leaked_fence;
ASSERT_EQ(VK_SUCCESS, vk::CreateFence(leaky_device, &fence_ci, nullptr, &leaked_fence));

m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkDestroyDevice-device-00378");
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkDestroyDevice-device-05137");
vk::DestroyDevice(leaky_device, nullptr);
m_errorMonitor->VerifyFound();

// There's no way we can destroy the fence at this point. Even though DestroyDevice failed, the loader has already removed
// references to the device
m_errorMonitor->SetUnexpectedError("VUID-vkDestroyDevice-device-00378");
m_errorMonitor->SetUnexpectedError("VUID-vkDestroyDevice-device-05137");
m_errorMonitor->SetUnexpectedError("UNASSIGNED-ObjectTracker-ObjectLeak");
}

Expand Down Expand Up @@ -1249,13 +1249,13 @@ TEST_F(VkLayerTest, LeakABuffer) {
VkBuffer buffer{};
ASSERT_EQ(VK_SUCCESS, vk::CreateBuffer(leaky_device, &buffer_create_info, nullptr, &buffer));

m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkDestroyDevice-device-00378");
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkDestroyDevice-device-05137");
vk::DestroyDevice(leaky_device, nullptr);
m_errorMonitor->VerifyFound();

// There's no way we can destroy the buffer at this point.
// Even though DestroyDevice failed, the loader has already removed references to the device
m_errorMonitor->SetUnexpectedError("VUID-vkDestroyDevice-device-00378");
m_errorMonitor->SetUnexpectedError("VUID-vkDestroyDevice-device-05137");
m_errorMonitor->SetUnexpectedError("UNASSIGNED-ObjectTracker-ObjectLeak");
}

Expand Down
2 changes: 1 addition & 1 deletion tests/unit/wsi.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1637,7 +1637,7 @@ TEST_F(NegativeWsi, LeakASwapchain) {

// Warn about the surface/swapchain not being destroyed
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkDestroyInstance-instance-00629");
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkDestroyDevice-device-00378");
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkDestroyDevice-device-05137");
ShutdownFramework(); // Destroy Instance/Device
DestroySurfaceContext(surface_context);
m_errorMonitor->VerifyFound();
Expand Down

0 comments on commit 0d09c6f

Please sign in to comment.