Skip to content

Commit dd28162

Browse files
layers: VUID Churn 1.4.350
1 parent eb798fd commit dd28162

12 files changed

Lines changed: 73 additions & 38 deletions

layers/core_checks/cc_device_generated_commands.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -668,9 +668,8 @@ bool CoreChecks::ValidateGeneratedCommandsInfo(const vvl::CommandBuffer& cb_stat
668668
}
669669

670670
{
671-
// VUID being added in https://gitlab.khronos.org/vulkan/vulkan/-/merge_requests/8220
672671
BufferAddressValidation<1> buffer_address_validator = {
673-
{{{"UNASSIGNED-VkGeneratedCommandsInfoEXT-indirectAddress-usage",
672+
{{{"VUID-VkGeneratedCommandsInfoEXT-indirectAddress-12407",
674673
[](const vvl::Buffer& buffer_state) { return (buffer_state.usage & VK_BUFFER_USAGE_2_INDIRECT_BUFFER_BIT) == 0; },
675674
[]() { return "The following buffers are missing VK_BUFFER_USAGE_2_INDIRECT_BUFFER_BIT"; }, kUsageErrorMsgBuffer}}}};
676675

layers/core_checks/cc_drawdispatch.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2189,7 +2189,7 @@ bool CoreChecks::ValidateActionStatePushConstant(const LastBound& last_bound_sta
21892189
// Edge case where if the shader is using push constants statically and there never was a vkCmdPushConstants
21902190
if (!cb_state.push_constant_ranges_layout && !enabled_features.maintenance4) {
21912191
const LogObjectList objlist(cb_state.Handle(), pipeline_layout->Handle(), pipeline->Handle());
2192-
skip |= LogError(CreateActionVuid(loc.function, vvl::ActionVUID::PUSH_CONSTANT_08602), objlist, loc,
2192+
skip |= LogError(CreateActionVuid(loc.function, vvl::ActionVUID::PUSH_CONSTANT_08601), objlist, loc,
21932193
"Shader in %s uses push-constant statically but vkCmdPushConstants was not called yet for "
21942194
"%s bound by vkCmdBindPipeline.",
21952195
string_VkShaderStageFlags(stage.GetStage()).c_str(),
@@ -2212,7 +2212,7 @@ bool CoreChecks::ValidateActionStatePushConstant(const LastBound& last_bound_sta
22122212
// Edge case where if the shader is using push constants statically and there never was a vkCmdPushConstants
22132213
if (!cb_state.push_constant_ranges_layout && !enabled_features.maintenance4) {
22142214
const LogObjectList objlist(cb_state.Handle(), shader_object->Handle());
2215-
skip |= LogError(CreateActionVuid(loc.function, vvl::ActionVUID::PUSH_CONSTANT_08602), objlist, loc,
2215+
skip |= LogError(CreateActionVuid(loc.function, vvl::ActionVUID::PUSH_CONSTANT_08601), objlist, loc,
22162216
"Shader in %s uses push-constant statically but vkCmdPushConstants was not called yet.",
22172217
string_VkShaderStageFlags(shader_object->create_info.stage).c_str());
22182218
}

layers/core_checks/cc_external_object.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@ bool CoreChecks::PreCallValidateGetSemaphoreFdKHR(VkDevice device, const VkSemap
110110

111111
if (sem_state->ImportedHandleType().has_value() &&
112112
!CanSemaphoreExportFromImported(pGetFdInfo->handleType, sem_state->ImportedHandleType().value())) {
113+
// See https://gitlab.khronos.org/vulkan/vulkan/-/issues/3670 for more details
113114
skip |= LogError("VUID-VkSemaphoreGetFdInfoKHR-semaphore-01133", sem_state->Handle(), info_loc.dot(Field::handleType),
114115
"(%s) cannot be exported from semaphore with imported payload with handle type %s",
115116
string_VkExternalSemaphoreHandleTypeFlagBits(pGetFdInfo->handleType),

layers/core_checks/cc_spirv.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3247,9 +3247,7 @@ bool CoreChecks::ValidateShaderDescriptorSetAndBindingMappingInfo(const spirv::M
32473247
} info;
32483248

32493249
if (base_opcode == spv::OpTypeSampledImage) {
3250-
// VUID being added in
3251-
// https://gitlab.khronos.org/vulkan/vulkan/-/merge_requests/8242
3252-
info.vuid = "UNASSIGNED-VkDescriptorSetAndBindingMappingEXT-combined-image";
3250+
info.vuid = "VUID-VkDescriptorSetAndBindingMappingEXT-source-12406";
32533251
info.align = phys_dev_ext_props.descriptor_heap_props.imageDescriptorAlignment;
32543252
info.align_field = Field::imageDescriptorAlignment;
32553253
} else if (base_opcode == spv::OpTypeImage) {

layers/drawdispatch/drawdispatch_vuids.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -772,8 +772,8 @@ std::string CreateActionVuid(vvl::Func function, const ActionVUID id) {
772772
// ### VUID-vkCmdDraw-imageLayout-00344
773773
case ActionVUID::IMAGE_LAYOUT_00344: suffix = "imageLayout-00344"; break;
774774

775-
// ### VUID-vkCmdDraw-maintenance4-08602
776-
case ActionVUID::PUSH_CONSTANT_08602: suffix = "maintenance4-08602"; break;
775+
// ### VUID-vkCmdDraw-None-08601
776+
case ActionVUID::PUSH_CONSTANT_08601: suffix = "None-08601"; break;
777777

778778
// ### VUID-vkCmdDraw-None-08608
779779
case ActionVUID::DYNAMIC_STATE_ALL_SET_08608: suffix = "None-08608"; break;

layers/drawdispatch/drawdispatch_vuids.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ enum class ActionVUID {
132132
FILTER_CUBIC_02694,
133133
FILTER_CUBIC_02695,
134134
IMAGE_LAYOUT_00344,
135-
PUSH_CONSTANT_08602,
135+
PUSH_CONSTANT_08601,
136136

137137
DYNAMIC_STATE_ALL_SET_08608,
138138
DYNAMIC_DEPTH_COMPARE_OP_07845,

0 commit comments

Comments
 (0)