| /* Copyright (c) 2015-2023 The Khronos Group Inc. |
| * Copyright (c) 2015-2023 Valve Corporation |
| * Copyright (c) 2015-2023 LunarG, Inc. |
| * Copyright (C) 2015-2023 Google Inc. |
| * Modifications Copyright (C) 2020 Advanced Micro Devices, Inc. All rights reserved. |
| * |
| * Licensed under the Apache License, Version 2.0 (the "License"); |
| * you may not use this file except in compliance with the License. |
| * You may obtain a copy of the License at |
| * |
| * http://www.apache.org/licenses/LICENSE-2.0 |
| * |
| * Unless required by applicable law or agreed to in writing, software |
| * distributed under the License is distributed on an "AS IS" BASIS, |
| * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| * See the License for the specific language governing permissions and |
| * limitations under the License. |
| */ |
| |
| #include "cc_shader.h" |
| |
| #include <cassert> |
| #include <cinttypes> |
| #include <sstream> |
| #include <string> |
| #include <vector> |
| |
| #include <spirv/unified1/spirv.hpp> |
| #include "generated/vk_enum_string_helper.h" |
| #include "generated/chassis.h" |
| #include "core_validation.h" |
| #include "generated/spirv_grammar_helper.h" |
| #include "external/xxhash.h" |
| |
| bool CoreChecks::ValidateViAgainstVsInputs(const PIPELINE_STATE &pipeline, const SHADER_MODULE_STATE &module_state, |
| const EntryPoint &entrypoint) const { |
| bool skip = false; |
| safe_VkPipelineVertexInputStateCreateInfo const *vi = pipeline.vertex_input_state->input_state; |
| |
| // Build index by location |
| std::map<uint32_t, const VkVertexInputAttributeDescription *> attribs; |
| if (vi) { |
| for (uint32_t i = 0; i < vi->vertexAttributeDescriptionCount; ++i) { |
| // Vertex input attributes use VkFormat, but only to make use of how they define sizes, things such as |
| // depth/multi-plane/compressed will never be used here because they would mean nothing. So we can ensure these are |
| // "standard" color formats being used |
| const VkFormat format = vi->pVertexAttributeDescriptions[i].format; |
| const uint32_t format_size = FormatElementSize(format); |
| // Vulkan Spec: Location is made up of 16 bytes, never can have 0 Locations |
| const uint32_t bytes_in_location = 16; |
| const uint32_t num_locations = ((format_size - 1) / bytes_in_location) + 1; |
| for (uint32_t j = 0; j < num_locations; ++j) { |
| attribs[vi->pVertexAttributeDescriptions[i].location + j] = &vi->pVertexAttributeDescriptions[i]; |
| } |
| } |
| } |
| |
| struct AttribInputPair { |
| const VkVertexInputAttributeDescription *attrib = nullptr; |
| const StageInteraceVariable *input = nullptr; |
| }; |
| std::map<uint32_t, AttribInputPair> location_map; |
| for (const auto &attrib_it : attribs) location_map[attrib_it.first].attrib = attrib_it.second; |
| for (const auto *variable : entrypoint.user_defined_interface_variables) { |
| if ((variable->storage_class != spv::StorageClassInput)) { |
| continue; // not an input interface |
| } |
| for (const auto &slot : variable->interface_slots) { |
| location_map[slot.Location()].input = variable; |
| } |
| } |
| |
| for (const auto &location_it : location_map) { |
| const auto location = location_it.first; |
| const auto attrib = location_it.second.attrib; |
| const auto input = location_it.second.input; |
| |
| if (attrib && !input) { |
| skip |= LogPerformanceWarning(module_state.vk_shader_module(), kVUID_Core_Shader_OutputNotConsumed, |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Vertex attribute at location %" PRIu32 " not consumed by vertex shader", |
| pipeline.create_index, location); |
| } else if (!attrib && input) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-Input-07904", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Vertex shader consumes input at location %" PRIu32 " but not provided", |
| pipeline.create_index, location); |
| } else if (attrib && input) { |
| const VkFormat attribute_format = attrib->format; |
| const auto attrib_type = GetFormatType(attribute_format); |
| const uint32_t input_base_type_id = input->base_type.ResultId(); |
| const auto input_type = module_state.GetNumericType(input_base_type_id); |
| |
| // TODO 5906 - This means there is a struct and we are not currently searching inside of it |
| if (input_type == NumericTypeUnknown) { |
| continue; |
| } |
| |
| // Type checking |
| if (!(attrib_type & input_type)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-Input-08733", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Attribute type of `%s` at location %" PRIu32 " does not match vertex shader input type of `%s`", |
| pipeline.create_index, string_VkFormat(attribute_format), location, |
| module_state.DescribeType(input_base_type_id).c_str()); |
| } else { |
| // 64-bit can't be used if the other is not also 64-bit. |
| const bool attribute64 = FormatIs64bit(attribute_format); |
| const bool input64 = module_state.GetBaseTypeInstruction(input_base_type_id)->GetBitWidth() == 64; |
| if (attribute64 && !input64) { |
| skip |= LogError(module_state.vk_shader_module(), "UNASSIGNED-VkGraphicsPipelineCreateInfo-Attribute-64bit", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Attribute at location %" PRIu32 |
| " is a 64-bit format (%s) but vertex shader input is 32-bit type (%s)", |
| pipeline.create_index, location, string_VkFormat(attribute_format), |
| module_state.DescribeType(input_base_type_id).c_str()); |
| } else if (!attribute64 && input64) { |
| skip |= LogError(module_state.vk_shader_module(), "UNASSIGNED-VkGraphicsPipelineCreateInfo-Input-64bit", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Attribute at location %" PRIu32 |
| " is a not a 64-bit format (%s) but vertex shader input is 64-bit type (%s)", |
| pipeline.create_index, location, string_VkFormat(attribute_format), |
| module_state.DescribeType(input_base_type_id).c_str()); |
| } else if (attribute64 && input64) { |
| // Unlike 32-bit, the components for 64-bit inputs have to match exactly |
| const uint32_t attribute_components = FormatComponentCount(attribute_format); |
| const uint32_t input_components = module_state.GetNumComponentsInBaseType(&input->base_type); |
| if (attribute_components != input_components) { |
| skip |= LogError(module_state.vk_shader_module(), "UNASSIGNED-VkGraphicsPipelineCreateInfo-Component-64bit", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Attribute at location %" PRIu32 |
| " is a %" PRIu32 "-wide 64-bit format (%s) but vertex shader input is %" PRIu32 |
| "-wide 64-bit type (%s), 64-bit vertex input don't have default values and require " |
| "matching components.", |
| pipeline.create_index, location, attribute_components, string_VkFormat(attribute_format), |
| input_components, module_state.DescribeType(input_base_type_id).c_str()); |
| } |
| } |
| } |
| } else { // !attrib && !input |
| assert(false); // at least one exists in the map |
| } |
| } |
| |
| return skip; |
| } |
| |
| // Validate use of input attachments against subpass structure |
| bool CoreChecks::ValidateShaderInputAttachment(const SHADER_MODULE_STATE &module_state, const PIPELINE_STATE &pipeline, |
| const ResourceInterfaceVariable &variable) const { |
| bool skip = false; |
| assert(variable.is_input_attachment); |
| |
| const auto &rp_state = pipeline.RenderPassState(); |
| // Dynamic Rendering guards this with VUID 06061 |
| if (!rp_state || rp_state->UsesDynamicRendering()) { |
| return skip; |
| } |
| |
| for (uint32_t i = 0; i < variable.input_attachment_index_read.size(); i++) { |
| // If the attachment is not read from, nothing to validate |
| if (!variable.input_attachment_index_read[i]) { |
| continue; |
| } |
| const auto rpci = rp_state->createInfo.ptr(); |
| const uint32_t subpass = pipeline.Subpass(); |
| const auto subpass_description = rpci->pSubpasses[subpass]; |
| const auto input_attachments = subpass_description.pInputAttachments; |
| // offsets by the InputAttachmentIndex decoration |
| const uint32_t input_attachment_index = variable.decorations.input_attachment_index_start + i; |
| |
| // Same error, but provide more useful message 'how' VK_ATTACHMENT_UNUSED is derived |
| if (!input_attachments) { |
| const LogObjectList objlist(module_state.vk_shader_module(), rp_state->renderPass()); |
| skip |= |
| LogError(objlist, "VUID-VkGraphicsPipelineCreateInfo-renderPass-06038", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Shader consumes input attachment index %" PRIu32 |
| " but pSubpasses[%" PRIu32 "].pInputAttachments is null", |
| pipeline.create_index, input_attachment_index, subpass); |
| } else if (input_attachment_index >= subpass_description.inputAttachmentCount) { |
| const LogObjectList objlist(module_state.vk_shader_module(), rp_state->renderPass()); |
| skip |= |
| LogError(objlist, "VUID-VkGraphicsPipelineCreateInfo-renderPass-06038", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Shader consumes input attachment index %" PRIu32 |
| " but that is greater than the pSubpasses[%" PRIu32 "].inputAttachmentCount (%" PRIu32 ")", |
| pipeline.create_index, input_attachment_index, subpass, subpass_description.inputAttachmentCount); |
| } else if (input_attachments[input_attachment_index].attachment == VK_ATTACHMENT_UNUSED) { |
| const LogObjectList objlist(module_state.vk_shader_module(), rp_state->renderPass()); |
| skip |= |
| LogError(objlist, "VUID-VkGraphicsPipelineCreateInfo-renderPass-06038", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Shader consumes input attachment index %" PRIu32 |
| " but pSubpasses[%" PRIu32 "].pInputAttachments[%" PRIu32 "].attachment is VK_ATTACHMENT_UNUSED", |
| pipeline.create_index, input_attachment_index, subpass, input_attachment_index); |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateConservativeRasterization(const SHADER_MODULE_STATE &module_state, const EntryPoint &entrypoint, |
| const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| |
| // only new to validate if property is not enabled |
| if (phys_dev_ext_props.conservative_rasterization_props.conservativeRasterizationPostDepthCoverage) { |
| return skip; |
| } |
| |
| // skipped here, don't need to check later |
| if (!entrypoint.execution_mode.Has(ExecutionModeSet::post_depth_coverage_bit)) { |
| return skip; |
| } |
| |
| if (module_state.static_data_.has_builtin_fully_covered) { |
| const LogObjectList objlist(module_state.vk_shader_module(), pipeline.PipelineLayoutState()->layout()); |
| skip |= LogError(objlist, "VUID-FullyCoveredEXT-conservativeRasterizationPostDepthCoverage-04235", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] has a fragment shader with a\nOpExecutionMode EarlyFragmentTests\nOpDecorate BuiltIn " |
| "FullyCoveredEXT\nbut conservativeRasterizationPostDepthCoverage is not enabled", |
| pipeline.create_index); |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidatePushConstantUsage(const PIPELINE_STATE &pipeline, const SHADER_MODULE_STATE &module_state, |
| const EntryPoint &entrypoint) const { |
| bool skip = false; |
| |
| // TODO - Workaround for https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/5911 |
| if (module_state.static_data_.has_specialization_constants) { |
| return skip; |
| } |
| |
| const VkShaderStageFlagBits stage = entrypoint.stage; |
| const auto push_constant_variable = entrypoint.push_constant_variable; |
| if (!push_constant_variable) { |
| return skip; |
| } |
| const auto &pipeline_layout = pipeline.PipelineLayoutState(); |
| std::vector<VkPushConstantRange> const *push_constant_ranges = pipeline_layout->push_constant_ranges.get(); |
| |
| std::string vuid; |
| switch (pipeline.GetCreateInfoSType()) { |
| case VK_STRUCTURE_TYPE_GRAPHICS_PIPELINE_CREATE_INFO: |
| vuid = "VUID-VkGraphicsPipelineCreateInfo-layout-07987"; |
| break; |
| case VK_STRUCTURE_TYPE_COMPUTE_PIPELINE_CREATE_INFO: |
| vuid = "VUID-VkComputePipelineCreateInfo-layout-07987"; |
| break; |
| case VK_STRUCTURE_TYPE_RAY_TRACING_PIPELINE_CREATE_INFO_KHR: |
| vuid = "VUID-VkRayTracingPipelineCreateInfoKHR-layout-07987"; |
| break; |
| case VK_STRUCTURE_TYPE_RAY_TRACING_PIPELINE_CREATE_INFO_NV: |
| vuid = "VUID-VkRayTracingPipelineCreateInfoNV-layout-07987"; |
| break; |
| default: |
| assert(false); |
| break; |
| } |
| |
| bool found_stage = false; |
| for (auto const &range : *push_constant_ranges) { |
| if (range.stageFlags & stage) { |
| found_stage = true; |
| const uint32_t range_end = range.offset + range.size; |
| const uint32_t push_constant_end = push_constant_variable->offset + push_constant_variable->size; |
| // spec: "If a push constant block is declared in a shader" |
| // Is checked regardless if element in Block is not statically used |
| if ((push_constant_variable->offset < range.offset) | (push_constant_end > range_end)) { |
| const LogObjectList objlist(module_state.vk_shader_module(), pipeline_layout->layout()); |
| skip |= LogError(objlist, vuid, |
| "%s(): pCreateInfos[%" PRIu32 "] %s has a push constant buffer Block with range [%" PRIu32 |
| ", %" PRIu32 "] which outside the pipeline layout range of [%" PRIu32 ", %" PRIu32 "].", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, string_VkShaderStageFlags(stage).c_str(), |
| push_constant_variable->offset, push_constant_end, range.offset, range_end); |
| break; |
| } |
| } |
| } |
| |
| if (!found_stage) { |
| const LogObjectList objlist(module_state.vk_shader_module(), pipeline_layout->layout()); |
| skip |= LogError(objlist, vuid, "%s(): pCreateInfos[%" PRIu32 "] Push constant is used in %s of %s. But %s doesn't set %s.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, string_VkShaderStageFlags(stage).c_str(), |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| report_data->FormatHandle(pipeline_layout->layout()).c_str(), string_VkShaderStageFlags(stage).c_str()); |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateBuiltinLimits(const SHADER_MODULE_STATE &module_state, const EntryPoint &entrypoint, |
| const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| |
| // Currently all builtin tested are only found in fragment shaders |
| if (entrypoint.execution_model != spv::ExecutionModelFragment) { |
| return skip; |
| } |
| |
| for (const auto *variable : entrypoint.built_in_variables) { |
| // Currently don't need to search in structs |
| if (variable->decorations.builtin == spv::BuiltInSampleMask) { |
| if (variable->base_type.Opcode() == spv::OpTypeArray) { |
| uint32_t length = module_state.GetConstantValueById(variable->base_type.Word(3)); |
| // Handles both the input and output sampleMask |
| if (length > phys_dev_props.limits.maxSampleMaskWords) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-maxSampleMaskWords-00711", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] The BuiltIns SampleMask array sizes is %u which exceeds " |
| "maxSampleMaskWords of %u in %s.", |
| pipeline.create_index, length, phys_dev_props.limits.maxSampleMaskWords, |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str()); |
| } |
| break; |
| } |
| } |
| } |
| |
| return skip; |
| } |
| |
| // Validate that data for each specialization entry is fully contained within the buffer. |
| bool CoreChecks::ValidateSpecializations(const SHADER_MODULE_STATE &module_state, const safe_VkSpecializationInfo *spec, |
| const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| if (spec) { |
| for (auto i = 0u; i < spec->mapEntryCount; i++) { |
| if (spec->pMapEntries[i].offset >= spec->dataSize) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkSpecializationInfo-offset-00773", |
| "%s(): pCreateInfos[%" PRIu32 |
| "] Specialization entry %u (for constant id %u) references memory outside provided specialization " |
| "data (bytes %u..%zu; %zu bytes provided).", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, i, spec->pMapEntries[i].constantID, |
| spec->pMapEntries[i].offset, spec->pMapEntries[i].offset + spec->dataSize - 1, spec->dataSize); |
| |
| continue; |
| } |
| if (spec->pMapEntries[i].offset + spec->pMapEntries[i].size > spec->dataSize) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkSpecializationInfo-pMapEntries-00774", |
| "%s(): pCreateInfos[%" PRIu32 |
| "] Specialization entry %u (for constant id %u) references memory outside provided specialization " |
| "data (bytes %u..%zu; %zu bytes provided).", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, i, spec->pMapEntries[i].constantID, |
| spec->pMapEntries[i].offset, spec->pMapEntries[i].offset + spec->pMapEntries[i].size - 1, |
| spec->dataSize); |
| } |
| for (uint32_t j = i + 1; j < spec->mapEntryCount; ++j) { |
| if (spec->pMapEntries[i].constantID == spec->pMapEntries[j].constantID) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-VkSpecializationInfo-constantID-04911", |
| "%s(): pCreateInfos[%" PRIu32 "] Specialization entry %" PRIu32 " and %" PRIu32 |
| " have the same constantID (%" PRIu32 ").", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, i, j, spec->pMapEntries[i].constantID); |
| } |
| } |
| } |
| } |
| |
| return skip; |
| } |
| |
| // TODO (jbolz): Can this return a const reference? |
| static std::set<uint32_t> TypeToDescriptorTypeSet(const SHADER_MODULE_STATE &module_state, uint32_t type_id, |
| uint32_t &descriptor_count, bool is_khr) { |
| const Instruction *type = module_state.FindDef(type_id); |
| bool is_storage_buffer = false; |
| descriptor_count = 1; |
| std::set<uint32_t> ret; |
| |
| // Strip off any array or ptrs. Where we remove array levels, adjust the descriptor count for each dimension. |
| while (type->Opcode() == spv::OpTypeArray || type->Opcode() == spv::OpTypePointer || |
| type->Opcode() == spv::OpTypeRuntimeArray) { |
| if (type->Opcode() == spv::OpTypeRuntimeArray) { |
| descriptor_count = 0; |
| type = module_state.FindDef(type->Word(2)); |
| } else if (type->Opcode() == spv::OpTypeArray) { |
| descriptor_count *= module_state.GetConstantValueById(type->Word(3)); |
| type = module_state.FindDef(type->Word(2)); |
| } else { |
| if (type->StorageClass() == spv::StorageClassStorageBuffer) { |
| is_storage_buffer = true; |
| } |
| type = module_state.FindDef(type->Word(3)); |
| } |
| } |
| |
| switch (type->Opcode()) { |
| case spv::OpTypeStruct: { |
| for (const Instruction *insn : module_state.static_data_.decoration_inst) { |
| if (insn->Word(1) == type->Word(1)) { |
| if (insn->Word(2) == spv::DecorationBlock) { |
| if (is_storage_buffer) { |
| ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER); |
| ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC); |
| return ret; |
| } else { |
| ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER); |
| ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC); |
| ret.insert(VK_DESCRIPTOR_TYPE_INLINE_UNIFORM_BLOCK_EXT); |
| return ret; |
| } |
| } else if (insn->Word(2) == spv::DecorationBufferBlock) { |
| ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER); |
| ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC); |
| return ret; |
| } |
| } |
| } |
| |
| // Invalid |
| return ret; |
| } |
| |
| case spv::OpTypeSampler: |
| ret.insert(VK_DESCRIPTOR_TYPE_SAMPLER); |
| ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER); |
| return ret; |
| |
| case spv::OpTypeSampledImage: { |
| // Slight relaxation for some GLSL historical madness: samplerBuffer doesn't really have a sampler, and a texel |
| // buffer descriptor doesn't really provide one. Allow this slight mismatch. |
| const Instruction *image_type = module_state.FindDef(type->Word(2)); |
| auto dim = image_type->Word(3); |
| auto sampled = image_type->Word(7); |
| if (dim == spv::DimBuffer && sampled == 1) { |
| ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER); |
| return ret; |
| } |
| } |
| ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER); |
| return ret; |
| |
| case spv::OpTypeImage: { |
| // Many descriptor types backing image types-- depends on dimension and whether the image will be used with a sampler. |
| // SPIRV for Vulkan requires that sampled be 1 or 2 -- leaving the decision to runtime is unacceptable. |
| auto dim = type->Word(3); |
| auto sampled = type->Word(7); |
| |
| if (dim == spv::DimSubpassData) { |
| ret.insert(VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT); |
| return ret; |
| } else if (dim == spv::DimBuffer) { |
| if (sampled == 1) { |
| ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER); |
| return ret; |
| } else { |
| ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER); |
| return ret; |
| } |
| } else if (sampled == 1) { |
| ret.insert(VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE); |
| ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER); |
| return ret; |
| } else { |
| ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_IMAGE); |
| return ret; |
| } |
| } |
| case spv::OpTypeAccelerationStructureNV: |
| is_khr ? ret.insert(VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_KHR) |
| : ret.insert(VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_NV); |
| return ret; |
| |
| // We shouldn't really see any other junk types -- but if we do, they're a mismatch. |
| default: |
| return ret; // Matches nothing |
| } |
| } |
| |
| static std::string string_descriptorTypeSet(const std::set<uint32_t> &descriptor_type_set) { |
| std::stringstream ss; |
| for (auto it = descriptor_type_set.begin(); it != descriptor_type_set.end(); ++it) { |
| if (ss.tellp()) ss << ", "; |
| ss << string_VkDescriptorType(VkDescriptorType(*it)); |
| } |
| return ss.str(); |
| } |
| |
| bool CoreChecks::RequirePropertyFlag(const SHADER_MODULE_STATE &module_state, VkBool32 check, char const *flag, |
| char const *structure, const char *vuid) const { |
| if (!check) { |
| if (LogError(module_state.vk_shader_module(), vuid, "Shader requires flag %s set in %s but it is not set on the device", |
| flag, structure)) { |
| return true; |
| } |
| } |
| |
| return false; |
| } |
| |
| bool CoreChecks::RequireFeature(const SHADER_MODULE_STATE &module_state, VkBool32 feature, char const *feature_name, |
| const char *vuid) const { |
| if (!feature) { |
| if (LogError(module_state.vk_shader_module(), vuid, "Shader requires %s but is not enabled on the device", feature_name)) { |
| return true; |
| } |
| } |
| |
| return false; |
| } |
| |
| bool CoreChecks::ValidateShaderStageGroupNonUniform(const SHADER_MODULE_STATE &module_state, VkShaderStageFlagBits stage) const { |
| bool skip = false; |
| |
| // Check anything using a group operation (which currently is only OpGroupNonUnifrom* operations) |
| for (const Instruction *group_inst : module_state.static_data_.group_inst) { |
| const Instruction &insn = *group_inst; |
| // Check the quad operations. |
| if ((insn.Opcode() == spv::OpGroupNonUniformQuadBroadcast) || (insn.Opcode() == spv::OpGroupNonUniformQuadSwap)) { |
| if ((stage != VK_SHADER_STAGE_FRAGMENT_BIT) && (stage != VK_SHADER_STAGE_COMPUTE_BIT)) { |
| skip |= |
| RequireFeature(module_state, phys_dev_props_core11.subgroupQuadOperationsInAllStages, |
| "VkPhysicalDeviceSubgroupProperties::quadOperationsInAllStages", "VUID-RuntimeSpirv-None-06342"); |
| } |
| } |
| |
| uint32_t scope_type = spv::ScopeMax; |
| if (insn.Opcode() == spv::OpGroupNonUniformPartitionNV) { |
| // OpGroupNonUniformPartitionNV always assumed subgroup as missing operand |
| scope_type = spv::ScopeSubgroup; |
| } else { |
| // "All <id> used for Scope <id> must be of an OpConstant" |
| const Instruction *scope_id = module_state.FindDef(insn.Word(3)); |
| scope_type = scope_id->Word(3); |
| } |
| |
| if (scope_type == spv::ScopeSubgroup) { |
| // "Group operations with subgroup scope" must have stage support |
| const VkSubgroupFeatureFlags supported_stages = phys_dev_props_core11.subgroupSupportedStages; |
| skip |= RequirePropertyFlag(module_state, supported_stages & stage, string_VkShaderStageFlagBits(stage), |
| "VkPhysicalDeviceSubgroupProperties::supportedStages", "VUID-RuntimeSpirv-None-06343"); |
| } |
| |
| if (!enabled_features.core12.shaderSubgroupExtendedTypes) { |
| const Instruction *type = module_state.FindDef(insn.Word(1)); |
| |
| if (type->Opcode() == spv::OpTypeVector) { |
| // Get the element type |
| type = module_state.FindDef(type->Word(2)); |
| } |
| |
| if (type->Opcode() != spv::OpTypeBool) { |
| // Both OpTypeInt and OpTypeFloat the width is in the 2nd word. |
| const uint32_t width = type->Word(2); |
| |
| if ((type->Opcode() == spv::OpTypeFloat && width == 16) || |
| (type->Opcode() == spv::OpTypeInt && (width == 8 || width == 16 || width == 64))) { |
| skip |= RequireFeature(module_state, enabled_features.core12.shaderSubgroupExtendedTypes, |
| "VkPhysicalDeviceShaderSubgroupExtendedTypesFeatures::shaderSubgroupExtendedTypes", |
| "VUID-RuntimeSpirv-None-06275"); |
| } |
| } |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateMemoryScope(const SHADER_MODULE_STATE &module_state, const Instruction &insn) const { |
| bool skip = false; |
| |
| const auto &entry = OpcodeMemoryScopePosition(insn.Opcode()); |
| if (entry > 0) { |
| const uint32_t scope_id = insn.Word(entry); |
| const Instruction *scope_def = module_state.GetConstantDef(scope_id); |
| if (scope_def) { |
| const auto scope_type = scope_def->GetConstantValue(); |
| if (enabled_features.core12.vulkanMemoryModel && !enabled_features.core12.vulkanMemoryModelDeviceScope && |
| scope_type == spv::Scope::ScopeDevice) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-vulkanMemoryModel-06265", |
| "VkPhysicalDeviceVulkan12Features::vulkanMemoryModel is enabled and " |
| "VkPhysicalDeviceVulkan12Features::vulkanMemoryModelDeviceScope is disabled, but\n%s\nuses " |
| "Device memory scope.", |
| insn.Describe().c_str()); |
| } else if (!enabled_features.core12.vulkanMemoryModel && scope_type == spv::Scope::ScopeQueueFamily) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-vulkanMemoryModel-06266", |
| "VkPhysicalDeviceVulkan12Features::vulkanMemoryModel is not enabled, but\n%s\nuses " |
| "QueueFamily memory scope.", |
| insn.Describe().c_str()); |
| } |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderStageInputOutputLimits(const SHADER_MODULE_STATE &module_state, VkShaderStageFlagBits stage, |
| const PIPELINE_STATE &pipeline, const EntryPoint &entrypoint) const { |
| if (stage == VK_SHADER_STAGE_COMPUTE_BIT || stage == VK_SHADER_STAGE_ALL_GRAPHICS || stage == VK_SHADER_STAGE_ALL) { |
| return false; |
| } |
| |
| bool skip = false; |
| auto const &limits = phys_dev_props.limits; |
| |
| const uint32_t num_vertices = entrypoint.execution_mode.output_vertices; |
| const uint32_t num_primitives = entrypoint.execution_mode.output_primitives; |
| const bool is_iso_lines = entrypoint.execution_mode.Has(ExecutionModeSet::iso_lines_bit); |
| const bool is_point_mode = entrypoint.execution_mode.Has(ExecutionModeSet::point_mode_bit); |
| const bool is_xfb_execution_mode = entrypoint.execution_mode.Has(ExecutionModeSet::xfb_bit); |
| |
| if (is_xfb_execution_mode && |
| ((pipeline.create_info_shaders & (VK_SHADER_STAGE_MESH_BIT_EXT | VK_SHADER_STAGE_TASK_BIT_EXT)) != 0)) { |
| skip |= LogError(pipeline.pipeline(), "VUID-VkGraphicsPipelineCreateInfo-None-02322", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] If the pipeline is being created with pre-rasterization shader state, and there are any mesh shader " |
| "stages in the pipeline there must not be any shader stage in the pipeline with a Xfb execution mode", |
| pipeline.create_index); |
| } |
| |
| // The max is a combiniation of both the user defined variables largest values |
| // and |
| // The total components used by built ins |
| const auto max_input_slot = |
| (entrypoint.max_input_slot_variable && entrypoint.max_input_slot) ? *entrypoint.max_input_slot : InterfaceSlot(0, 0, 0, 0); |
| const auto max_output_slot = (entrypoint.max_output_slot_variable && entrypoint.max_output_slot) ? *entrypoint.max_output_slot |
| : InterfaceSlot(0, 0, 0, 0); |
| |
| const uint32_t total_input_components = max_input_slot.slot + entrypoint.builtin_input_components; |
| const uint32_t total_output_components = max_output_slot.slot + entrypoint.builtin_output_components; |
| |
| switch (stage) { |
| case VK_SHADER_STAGE_VERTEX_BIT: |
| if (total_output_components >= limits.maxVertexOutputComponents) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Vertex shader output variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxVertexOutputComponents (%" PRIu32 ")", |
| pipeline.create_index, max_output_slot.Describe().c_str(), entrypoint.builtin_output_components, |
| limits.maxVertexOutputComponents); |
| } |
| break; |
| |
| case VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT: |
| if (total_input_components >= limits.maxTessellationControlPerVertexInputComponents) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Tessellation control shader input variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxTessellationControlPerVertexInputComponents (%" PRIu32 ")", |
| pipeline.create_index, max_input_slot.Describe().c_str(), entrypoint.builtin_input_components, |
| limits.maxTessellationControlPerVertexInputComponents); |
| } |
| if (entrypoint.max_input_slot_variable) { |
| if (entrypoint.max_input_slot_variable->is_patch && |
| total_output_components >= limits.maxTessellationControlPerPatchOutputComponents) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Tessellation control shader output variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxTessellationControlPerPatchOutputComponents (%" PRIu32 |
| ")", |
| pipeline.create_index, max_output_slot.Describe().c_str(), entrypoint.builtin_output_components, |
| limits.maxTessellationControlPerPatchOutputComponents); |
| } |
| if (!entrypoint.max_input_slot_variable->is_patch && |
| total_output_components >= limits.maxTessellationControlPerVertexOutputComponents) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Tessellation control shader output variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxTessellationControlPerVertexOutputComponents (%" PRIu32 |
| ")", |
| pipeline.create_index, max_output_slot.Describe().c_str(), entrypoint.builtin_output_components, |
| limits.maxTessellationControlPerVertexOutputComponents); |
| } |
| } |
| break; |
| |
| case VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT: |
| if (total_input_components >= limits.maxTessellationEvaluationInputComponents) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Tessellation evaluation shader input variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxTessellationEvaluationInputComponents (%" PRIu32 ")", |
| pipeline.create_index, max_input_slot.Describe().c_str(), entrypoint.builtin_input_components, |
| limits.maxTessellationEvaluationInputComponents); |
| } |
| if (total_output_components >= limits.maxTessellationEvaluationOutputComponents) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Tessellation evaluation shader output variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxTessellationEvaluationOutputComponents (%" PRIu32 ")", |
| pipeline.create_index, max_output_slot.Describe().c_str(), entrypoint.builtin_output_components, |
| limits.maxTessellationEvaluationOutputComponents); |
| } |
| // Portability validation |
| if (IsExtEnabled(device_extensions.vk_khr_portability_subset)) { |
| if (is_iso_lines && (VK_FALSE == enabled_features.portability_subset_features.tessellationIsolines)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-tessellationShader-06326", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "](portability error): Tessellation evaluation shader" |
| " is using abstract patch type IsoLines, but this is not supported on this platform", |
| pipeline.create_index); |
| } |
| if (is_point_mode && (VK_FALSE == enabled_features.portability_subset_features.tessellationPointMode)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-tessellationShader-06327", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "](portability error): Tessellation evaluation shader" |
| " is using abstract patch type PointMode, but this is not supported on this platform", |
| pipeline.create_index); |
| } |
| } |
| break; |
| |
| case VK_SHADER_STAGE_GEOMETRY_BIT: |
| if (total_input_components >= limits.maxGeometryInputComponents) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Geometry shader input variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxGeometryInputComponents (%" PRIu32 ")", |
| pipeline.create_index, max_input_slot.Describe().c_str(), entrypoint.builtin_input_components, |
| limits.maxGeometryInputComponents); |
| } |
| if (total_output_components >= limits.maxGeometryOutputComponents) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Geometry shader output variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxGeometryOutputComponents (%" PRIu32 ")", |
| pipeline.create_index, max_output_slot.Describe().c_str(), entrypoint.builtin_output_components, |
| limits.maxGeometryOutputComponents); |
| } |
| break; |
| |
| case VK_SHADER_STAGE_FRAGMENT_BIT: |
| if (total_input_components >= limits.maxFragmentInputComponents) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-Location-06272", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Fragment shader input variable (%s) along with %" PRIu32 |
| " built-in components, " |
| "exceeds component limit VkPhysicalDeviceLimits::maxFragmentInputComponents (%" PRIu32 ")", |
| pipeline.create_index, max_input_slot.Describe().c_str(), entrypoint.builtin_input_components, |
| limits.maxFragmentInputComponents); |
| } |
| break; |
| |
| case VK_SHADER_STAGE_RAYGEN_BIT_KHR: |
| case VK_SHADER_STAGE_ANY_HIT_BIT_KHR: |
| case VK_SHADER_STAGE_CLOSEST_HIT_BIT_KHR: |
| case VK_SHADER_STAGE_MISS_BIT_KHR: |
| case VK_SHADER_STAGE_INTERSECTION_BIT_KHR: |
| case VK_SHADER_STAGE_CALLABLE_BIT_KHR: |
| case VK_SHADER_STAGE_TASK_BIT_EXT: |
| break; |
| |
| // Shader stage is an alias, but the ExecutionModel is not |
| case VK_SHADER_STAGE_MESH_BIT_EXT: |
| if (entrypoint.execution_model == spv::ExecutionModelMeshNV) { |
| if (num_vertices > phys_dev_ext_props.mesh_shader_props_nv.maxMeshOutputVertices) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-MeshNV-07113", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Mesh shader output vertices count exceeds the " |
| "maxMeshOutputVertices of %" PRIu32 " by %" PRIu32, |
| pipeline.create_index, phys_dev_ext_props.mesh_shader_props_nv.maxMeshOutputVertices, |
| num_vertices - phys_dev_ext_props.mesh_shader_props_nv.maxMeshOutputVertices); |
| } |
| if (num_primitives > phys_dev_ext_props.mesh_shader_props_nv.maxMeshOutputPrimitives) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-MeshNV-07114", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Mesh shader output primitives count exceeds the " |
| "maxMeshOutputPrimitives of %" PRIu32 " by %" PRIu32, |
| pipeline.create_index, phys_dev_ext_props.mesh_shader_props_nv.maxMeshOutputPrimitives, |
| num_primitives - phys_dev_ext_props.mesh_shader_props_nv.maxMeshOutputPrimitives); |
| } |
| } else if (entrypoint.execution_model == spv::ExecutionModelMeshEXT) { |
| if (num_vertices > phys_dev_ext_props.mesh_shader_props_ext.maxMeshOutputVertices) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-MeshEXT-07115", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Mesh shader output vertices count exceeds the " |
| "maxMeshOutputVertices of %" PRIu32 " by %" PRIu32, |
| pipeline.create_index, phys_dev_ext_props.mesh_shader_props_ext.maxMeshOutputVertices, |
| num_vertices - phys_dev_ext_props.mesh_shader_props_ext.maxMeshOutputVertices); |
| } |
| if (num_primitives > phys_dev_ext_props.mesh_shader_props_ext.maxMeshOutputPrimitives) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-MeshEXT-07116", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Mesh shader output primitives count exceeds the " |
| "maxMeshOutputPrimitives of %u by %u ", |
| pipeline.create_index, phys_dev_ext_props.mesh_shader_props_ext.maxMeshOutputPrimitives, |
| num_primitives - phys_dev_ext_props.mesh_shader_props_ext.maxMeshOutputPrimitives); |
| } |
| } |
| break; |
| |
| default: |
| assert(false); // This should never happen |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderStorageImageFormatsVariables(const SHADER_MODULE_STATE &module_state, |
| const Instruction *insn) const { |
| bool skip = false; |
| // Go through all variables for images and check decorations |
| // Note: Tried to move to ResourceInterfaceVariable but the issue is the variables don't need to be accessed in the entrypoint |
| // to trigger the error. |
| assert(insn->Opcode() == spv::OpVariable); |
| // spirv-val validates this is an OpTypePointer |
| const Instruction *pointer_def = module_state.FindDef(insn->Word(1)); |
| if (pointer_def->Word(2) != spv::StorageClassUniformConstant) { |
| return skip; // Vulkan Spec says storage image must be UniformConstant |
| } |
| const Instruction *type_def = module_state.FindDef(pointer_def->Word(3)); |
| |
| // Unpack an optional level of arraying |
| if (type_def && (type_def->Opcode() == spv::OpTypeArray || type_def->Opcode() == spv::OpTypeRuntimeArray)) { |
| type_def = module_state.FindDef(type_def->Word(2)); |
| } |
| |
| if (type_def && type_def->Opcode() == spv::OpTypeImage) { |
| // Only check if the Image Dim operand is not SubpassData |
| const uint32_t dim = type_def->Word(3); |
| // Only check storage images |
| const uint32_t sampled = type_def->Word(7); |
| const uint32_t image_format = type_def->Word(8); |
| if ((dim == spv::DimSubpassData) || (sampled != 2) || (image_format != spv::ImageFormatUnknown)) { |
| return skip; |
| } |
| |
| const uint32_t var_id = insn->Word(2); |
| const auto decorations = module_state.GetDecorationSet(var_id); |
| |
| if (!enabled_features.core.shaderStorageImageReadWithoutFormat && !decorations.Has(DecorationSet::nonreadable_bit)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpTypeImage-06270", |
| "shaderStorageImageReadWithoutFormat is not supported but\n%s\nhas an Image\n%s\nwith Unknown " |
| "format and is not decorated with NonReadable", |
| module_state.FindDef(var_id)->Describe().c_str(), type_def->Describe().c_str()); |
| } |
| |
| if (!enabled_features.core.shaderStorageImageWriteWithoutFormat && !decorations.Has(DecorationSet::nonwritable_bit)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpTypeImage-06269", |
| "shaderStorageImageWriteWithoutFormat is not supported but\n%s\nhas an Image\n%s\nwith " |
| "Unknown format and is not decorated with NonWritable", |
| module_state.FindDef(var_id)->Describe().c_str(), type_def->Describe().c_str()); |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderStageMaxResources(const SHADER_MODULE_STATE &module_state, VkShaderStageFlagBits stage, |
| const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| uint32_t total_resources = 0; |
| |
| const auto &rp_state = pipeline.RenderPassState(); |
| if ((stage == VK_SHADER_STAGE_FRAGMENT_BIT) && rp_state) { |
| if (rp_state->UsesDynamicRendering()) { |
| total_resources += rp_state->dynamic_rendering_pipeline_create_info.colorAttachmentCount; |
| } else { |
| // "For the fragment shader stage the framebuffer color attachments also count against this limit" |
| total_resources += rp_state->createInfo.pSubpasses[pipeline.Subpass()].colorAttachmentCount; |
| } |
| } |
| |
| // TODO: This reuses a lot of GetDescriptorCountMaxPerStage but currently would need to make it agnostic in a way to handle |
| // input from CreatePipeline and CreatePipelineLayout level |
| const auto &layout_state = pipeline.PipelineLayoutState(); |
| if (layout_state) { |
| for (const auto &set_layout : layout_state->set_layouts) { |
| if (!set_layout) { |
| continue; |
| } |
| |
| if ((set_layout->GetCreateFlags() & VK_DESCRIPTOR_SET_LAYOUT_CREATE_UPDATE_AFTER_BIND_POOL_BIT) != 0) { |
| continue; |
| } |
| |
| for (uint32_t binding_idx = 0; binding_idx < set_layout->GetBindingCount(); binding_idx++) { |
| const VkDescriptorSetLayoutBinding *binding = set_layout->GetDescriptorSetLayoutBindingPtrFromIndex(binding_idx); |
| // Bindings with a descriptorCount of 0 are "reserved" and should be skipped |
| if (((stage & binding->stageFlags) != 0) && (binding->descriptorCount > 0)) { |
| // Check only descriptor types listed in maxPerStageResources description in spec |
| switch (binding->descriptorType) { |
| case VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER: |
| case VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE: |
| case VK_DESCRIPTOR_TYPE_STORAGE_IMAGE: |
| case VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER: |
| case VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER: |
| case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER: |
| case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER: |
| case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC: |
| case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC: |
| case VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT: |
| total_resources += binding->descriptorCount; |
| break; |
| default: |
| break; |
| } |
| } |
| } |
| } |
| } |
| |
| if (total_resources > phys_dev_props.limits.maxPerStageResources) { |
| const char *vuid = nullptr; |
| if (stage == VK_SHADER_STAGE_COMPUTE_BIT) { |
| vuid = "VUID-VkComputePipelineCreateInfo-layout-01687"; |
| } else if ((stage & VK_SHADER_STAGE_ALL_GRAPHICS) == 0) { |
| vuid = "VUID-VkRayTracingPipelineCreateInfoKHR-layout-03428"; |
| } else { |
| vuid = "VUID-VkGraphicsPipelineCreateInfo-layout-01688"; |
| } |
| skip |= LogError(module_state.vk_shader_module(), vuid, |
| "%s(): pCreateInfos[%" PRIu32 |
| "] Shader Stage %s exceeds component limit " |
| "VkPhysicalDeviceLimits::maxPerStageResources (%" PRIu32 ")", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, string_VkShaderStageFlagBits(stage), |
| phys_dev_props.limits.maxPerStageResources); |
| } |
| |
| return skip; |
| } |
| |
| // copy the specialization constant value into buf, if it is present |
| void GetSpecConstantValue(const safe_VkSpecializationInfo *spec, uint32_t spec_id, void *buf) { |
| if (spec && spec_id < spec->mapEntryCount) { |
| memcpy(buf, (uint8_t *)spec->pData + spec->pMapEntries[spec_id].offset, spec->pMapEntries[spec_id].size); |
| } |
| } |
| |
| // Fill in value with the constant or specialization constant value, if available. |
| // Returns true if the value has been accurately filled out. |
| static bool GetIntConstantValue(const Instruction *insn, const SHADER_MODULE_STATE &module_state, |
| const safe_VkSpecializationInfo *spec, const vvl::unordered_map<uint32_t, uint32_t> &id_to_spec_id, |
| uint32_t *value) { |
| const Instruction *type_id = module_state.FindDef(insn->Word(1)); |
| if (type_id->Opcode() != spv::OpTypeInt || type_id->Word(2) != 32) { |
| return false; |
| } |
| switch (insn->Opcode()) { |
| case spv::OpSpecConstant: |
| *value = insn->Word(3); |
| GetSpecConstantValue(spec, id_to_spec_id.at(insn->Word(2)), value); |
| return true; |
| case spv::OpConstant: |
| *value = insn->Word(3); |
| return true; |
| default: |
| return false; |
| } |
| } |
| |
| // Map SPIR-V type to VK_COMPONENT_TYPE enum |
| VkComponentTypeNV GetComponentType(const Instruction *insn) { |
| switch (insn->Opcode()) { |
| case spv::OpTypeInt: |
| switch (insn->Word(2)) { |
| case 8: |
| return insn->Word(3) != 0 ? VK_COMPONENT_TYPE_SINT8_NV : VK_COMPONENT_TYPE_UINT8_NV; |
| case 16: |
| return insn->Word(3) != 0 ? VK_COMPONENT_TYPE_SINT16_NV : VK_COMPONENT_TYPE_UINT16_NV; |
| case 32: |
| return insn->Word(3) != 0 ? VK_COMPONENT_TYPE_SINT32_NV : VK_COMPONENT_TYPE_UINT32_NV; |
| case 64: |
| return insn->Word(3) != 0 ? VK_COMPONENT_TYPE_SINT64_NV : VK_COMPONENT_TYPE_UINT64_NV; |
| default: |
| return VK_COMPONENT_TYPE_MAX_ENUM_NV; |
| } |
| case spv::OpTypeFloat: |
| switch (insn->Word(2)) { |
| case 16: |
| return VK_COMPONENT_TYPE_FLOAT16_NV; |
| case 32: |
| return VK_COMPONENT_TYPE_FLOAT32_NV; |
| case 64: |
| return VK_COMPONENT_TYPE_FLOAT64_NV; |
| default: |
| return VK_COMPONENT_TYPE_MAX_ENUM_NV; |
| } |
| default: |
| return VK_COMPONENT_TYPE_MAX_ENUM_NV; |
| } |
| } |
| |
| // Validate SPV_NV_cooperative_matrix behavior that can't be statically validated |
| // in SPIRV-Tools (e.g. due to specialization constant usage). |
| bool CoreChecks::ValidateCooperativeMatrix(const SHADER_MODULE_STATE &module_state, |
| safe_VkPipelineShaderStageCreateInfo const *create_info) const { |
| bool skip = false; |
| |
| // Map SPIR-V result ID to specialization constant id (SpecId decoration value) |
| vvl::unordered_map<uint32_t, uint32_t> id_to_spec_id; |
| // Map SPIR-V result ID to the ID of its type. |
| vvl::unordered_map<uint32_t, uint32_t> id_to_type_id; |
| const safe_VkSpecializationInfo *spec = create_info->pSpecializationInfo; |
| |
| struct CoopMatType { |
| uint32_t scope, rows, cols; |
| VkComponentTypeNV component_type; |
| bool all_constant; |
| |
| CoopMatType() : scope(0), rows(0), cols(0), component_type(VK_COMPONENT_TYPE_MAX_ENUM_NV), all_constant(false) {} |
| |
| void Init(uint32_t id, const SHADER_MODULE_STATE &module_state, const safe_VkSpecializationInfo *spec, |
| const vvl::unordered_map<uint32_t, uint32_t> &id_to_spec_id) { |
| const Instruction *insn = module_state.FindDef(id); |
| uint32_t component_type_id = insn->Word(2); |
| uint32_t scope_id = insn->Word(3); |
| uint32_t rows_id = insn->Word(4); |
| uint32_t cols_id = insn->Word(5); |
| const Instruction *component_type_insn = module_state.FindDef(component_type_id); |
| const Instruction *scope_insn = module_state.FindDef(scope_id); |
| const Instruction *rows_insn = module_state.FindDef(rows_id); |
| const Instruction *cols_insn = module_state.FindDef(cols_id); |
| |
| all_constant = true; |
| if (!GetIntConstantValue(scope_insn, module_state, spec, id_to_spec_id, &scope)) { |
| all_constant = false; |
| } |
| if (!GetIntConstantValue(rows_insn, module_state, spec, id_to_spec_id, &rows)) { |
| all_constant = false; |
| } |
| if (!GetIntConstantValue(cols_insn, module_state, spec, id_to_spec_id, &cols)) { |
| all_constant = false; |
| } |
| component_type = GetComponentType(component_type_insn); |
| } |
| }; |
| |
| bool seen_coopmat_capability = false; |
| |
| for (const Instruction &insn : module_state.GetInstructions()) { |
| if (OpcodeHasType(insn.Opcode()) && OpcodeHasResult(insn.Opcode())) { |
| id_to_type_id[insn.Word(2)] = insn.Word(1); |
| } |
| |
| switch (insn.Opcode()) { |
| case spv::OpDecorate: |
| if (insn.Word(2) == spv::DecorationSpecId) { |
| id_to_spec_id[insn.Word(1)] = insn.Word(3); |
| } |
| break; |
| case spv::OpCapability: |
| if (insn.Word(1) == spv::CapabilityCooperativeMatrixNV) { |
| seen_coopmat_capability = true; |
| |
| if (!(create_info->stage & phys_dev_ext_props.cooperative_matrix_props.cooperativeMatrixSupportedStages)) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpTypeCooperativeMatrixNV-06322", |
| "OpTypeCooperativeMatrixNV used in shader stage not in cooperativeMatrixSupportedStages (= %u)", |
| phys_dev_ext_props.cooperative_matrix_props.cooperativeMatrixSupportedStages); |
| } |
| } |
| break; |
| case spv::OpMemoryModel: |
| // If the capability isn't enabled, don't bother with the rest of this function. |
| // OpMemoryModel is the first required instruction after all OpCapability instructions. |
| if (!seen_coopmat_capability) { |
| return skip; |
| } |
| break; |
| case spv::OpTypeCooperativeMatrixNV: { |
| CoopMatType m; |
| m.Init(insn.Word(1), module_state, spec, id_to_spec_id); |
| |
| if (m.all_constant) { |
| // Validate that the type parameters are all supported for one of the |
| // operands of a cooperative matrix property. |
| bool valid = false; |
| for (uint32_t i = 0; i < cooperative_matrix_properties.size(); ++i) { |
| if (cooperative_matrix_properties[i].AType == m.component_type && |
| cooperative_matrix_properties[i].MSize == m.rows && cooperative_matrix_properties[i].KSize == m.cols && |
| cooperative_matrix_properties[i].scope == m.scope) { |
| valid = true; |
| break; |
| } |
| if (cooperative_matrix_properties[i].BType == m.component_type && |
| cooperative_matrix_properties[i].KSize == m.rows && cooperative_matrix_properties[i].NSize == m.cols && |
| cooperative_matrix_properties[i].scope == m.scope) { |
| valid = true; |
| break; |
| } |
| if (cooperative_matrix_properties[i].CType == m.component_type && |
| cooperative_matrix_properties[i].MSize == m.rows && cooperative_matrix_properties[i].NSize == m.cols && |
| cooperative_matrix_properties[i].scope == m.scope) { |
| valid = true; |
| break; |
| } |
| if (cooperative_matrix_properties[i].DType == m.component_type && |
| cooperative_matrix_properties[i].MSize == m.rows && cooperative_matrix_properties[i].NSize == m.cols && |
| cooperative_matrix_properties[i].scope == m.scope) { |
| valid = true; |
| break; |
| } |
| } |
| if (!valid) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpTypeCooperativeMatrixNV-06316", |
| "OpTypeCooperativeMatrixNV (result id = %u) operands don't match a supported matrix type", |
| insn.Word(1)); |
| } |
| } |
| break; |
| } |
| case spv::OpCooperativeMatrixMulAddNV: { |
| CoopMatType a, b, c, d; |
| if (id_to_type_id.find(insn.Word(2)) == id_to_type_id.end() || |
| id_to_type_id.find(insn.Word(3)) == id_to_type_id.end() || |
| id_to_type_id.find(insn.Word(4)) == id_to_type_id.end() || |
| id_to_type_id.find(insn.Word(5)) == id_to_type_id.end()) { |
| // Couldn't find type of matrix |
| assert(false); |
| break; |
| } |
| d.Init(id_to_type_id[insn.Word(2)], module_state, spec, id_to_spec_id); |
| a.Init(id_to_type_id[insn.Word(3)], module_state, spec, id_to_spec_id); |
| b.Init(id_to_type_id[insn.Word(4)], module_state, spec, id_to_spec_id); |
| c.Init(id_to_type_id[insn.Word(5)], module_state, spec, id_to_spec_id); |
| |
| if (a.all_constant && b.all_constant && c.all_constant && d.all_constant) { |
| // Validate that the type parameters are all supported for the same |
| // cooperative matrix property. |
| bool valid_a = false; |
| bool valid_b = false; |
| bool valid_c = false; |
| bool valid_d = false; |
| for (uint32_t i = 0; i < cooperative_matrix_properties.size(); ++i) { |
| valid_a = cooperative_matrix_properties[i].AType == a.component_type && |
| cooperative_matrix_properties[i].MSize == a.rows && |
| cooperative_matrix_properties[i].KSize == a.cols && |
| cooperative_matrix_properties[i].scope == a.scope; |
| valid_b = cooperative_matrix_properties[i].BType == b.component_type && |
| cooperative_matrix_properties[i].KSize == b.rows && |
| cooperative_matrix_properties[i].NSize == b.cols && |
| cooperative_matrix_properties[i].scope == b.scope; |
| valid_c = cooperative_matrix_properties[i].CType == c.component_type && |
| cooperative_matrix_properties[i].MSize == c.rows && |
| cooperative_matrix_properties[i].NSize == c.cols && |
| cooperative_matrix_properties[i].scope == c.scope; |
| valid_d = cooperative_matrix_properties[i].DType == d.component_type && |
| cooperative_matrix_properties[i].MSize == d.rows && |
| cooperative_matrix_properties[i].NSize == d.cols && |
| cooperative_matrix_properties[i].scope == d.scope; |
| if (valid_a && valid_b && valid_c && valid_d) { |
| break; |
| } |
| } |
| if (!valid_a) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpCooperativeMatrixMulAddNV-06317", |
| "OpCooperativeMatrixMulAddNV (result id = %u) operands don't match a supported matrix " |
| "VkCooperativeMatrixPropertiesNV", |
| insn.Word(2)); |
| } else if (!valid_b) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpCooperativeMatrixMulAddNV-06318", |
| "OpCooperativeMatrixMulAddNV (result id = %u) operands don't match a supported matrix " |
| "VkCooperativeMatrixPropertiesNV", |
| insn.Word(2)); |
| } else if (!valid_c) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpCooperativeMatrixMulAddNV-06319", |
| "OpCooperativeMatrixMulAddNV (result id = %u) operands don't match a supported matrix " |
| "VkCooperativeMatrixPropertiesNV", |
| insn.Word(2)); |
| } else if (!valid_d) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpCooperativeMatrixMulAddNV-06320", |
| "OpCooperativeMatrixMulAddNV (result id = %u) operands don't match a supported matrix " |
| "VkCooperativeMatrixPropertiesNV", |
| insn.Word(2)); |
| } |
| } |
| break; |
| } |
| default: |
| break; |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderResolveQCOM(const SHADER_MODULE_STATE &module_state, VkShaderStageFlagBits stage, |
| const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| |
| // If the pipeline's subpass description contains flag VK_SUBPASS_DESCRIPTION_FRAGMENT_REGION_BIT_QCOM, |
| // then the fragment shader must not enable the SPIRV SampleRateShading capability. |
| if (stage == VK_SHADER_STAGE_FRAGMENT_BIT && module_state.HasCapability(spv::CapabilitySampleRateShading)) { |
| const auto &rp_state = pipeline.RenderPassState(); |
| auto subpass_flags = (!rp_state) ? 0 : rp_state->createInfo.pSubpasses[pipeline.Subpass()].flags; |
| if ((subpass_flags & VK_SUBPASS_DESCRIPTION_FRAGMENT_REGION_BIT_QCOM) != 0) { |
| const LogObjectList objlist(module_state.vk_shader_module(), rp_state->renderPass()); |
| skip |= LogError(objlist, "VUID-RuntimeSpirv-SampleRateShading-06378", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "]: fragment shader enables SampleRateShading capability " |
| "and the subpass flags includes VK_SUBPASS_DESCRIPTION_FRAGMENT_REGION_BIT_QCOM.", |
| pipeline.create_index); |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderSubgroupSizeControl(const SHADER_MODULE_STATE &module_state, |
| VkPipelineShaderStageCreateFlags flags) const { |
| bool skip = false; |
| |
| if ((flags & VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT) != 0 && |
| !enabled_features.core13.subgroupSizeControl) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-flags-02784", |
| "VkPipelineShaderStageCreateInfo flags contain VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT, " |
| "but the VkPhysicalDeviceSubgroupSizeControlFeaturesEXT::subgroupSizeControl feature is not enabled."); |
| } |
| |
| if ((flags & VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT) != 0 && |
| !enabled_features.core13.computeFullSubgroups) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-flags-02785", |
| "VkPipelineShaderStageCreateInfo flags contain VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT, but the " |
| "VkPhysicalDeviceSubgroupSizeControlFeaturesEXT::computeFullSubgroups feature is not enabled"); |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateAtomicsTypes(const SHADER_MODULE_STATE &module_state) const { |
| bool skip = false; |
| |
| // "If sparseImageInt64Atomics is enabled, shaderImageInt64Atomics must be enabled" |
| const bool valid_image_64_int = enabled_features.shader_image_atomic_int64_features.shaderImageInt64Atomics == VK_TRUE; |
| |
| const VkPhysicalDeviceShaderAtomicFloatFeaturesEXT &float_features = enabled_features.shader_atomic_float_features; |
| const VkPhysicalDeviceShaderAtomicFloat2FeaturesEXT &float2_features = enabled_features.shader_atomic_float2_features; |
| |
| const bool valid_storage_buffer_float = |
| ((float_features.shaderBufferFloat32Atomics == VK_TRUE) || (float_features.shaderBufferFloat32AtomicAdd == VK_TRUE) || |
| (float_features.shaderBufferFloat64Atomics == VK_TRUE) || (float_features.shaderBufferFloat64AtomicAdd == VK_TRUE) || |
| (float2_features.shaderBufferFloat16Atomics == VK_TRUE) || (float2_features.shaderBufferFloat16AtomicAdd == VK_TRUE) || |
| (float2_features.shaderBufferFloat16AtomicMinMax == VK_TRUE) || |
| (float2_features.shaderBufferFloat32AtomicMinMax == VK_TRUE) || |
| (float2_features.shaderBufferFloat64AtomicMinMax == VK_TRUE)); |
| |
| const bool valid_workgroup_float = |
| ((float_features.shaderSharedFloat32Atomics == VK_TRUE) || (float_features.shaderSharedFloat32AtomicAdd == VK_TRUE) || |
| (float_features.shaderSharedFloat64Atomics == VK_TRUE) || (float_features.shaderSharedFloat64AtomicAdd == VK_TRUE) || |
| (float2_features.shaderSharedFloat16Atomics == VK_TRUE) || (float2_features.shaderSharedFloat16AtomicAdd == VK_TRUE) || |
| (float2_features.shaderSharedFloat16AtomicMinMax == VK_TRUE) || |
| (float2_features.shaderSharedFloat32AtomicMinMax == VK_TRUE) || |
| (float2_features.shaderSharedFloat64AtomicMinMax == VK_TRUE)); |
| |
| const bool valid_image_float = |
| ((float_features.shaderImageFloat32Atomics == VK_TRUE) || (float_features.shaderImageFloat32AtomicAdd == VK_TRUE) || |
| (float2_features.shaderImageFloat32AtomicMinMax == VK_TRUE)); |
| |
| const bool valid_16_float = |
| ((float2_features.shaderBufferFloat16Atomics == VK_TRUE) || (float2_features.shaderBufferFloat16AtomicAdd == VK_TRUE) || |
| (float2_features.shaderBufferFloat16AtomicMinMax == VK_TRUE) || (float2_features.shaderSharedFloat16Atomics == VK_TRUE) || |
| (float2_features.shaderSharedFloat16AtomicAdd == VK_TRUE) || (float2_features.shaderSharedFloat16AtomicMinMax == VK_TRUE)); |
| |
| const bool valid_32_float = |
| ((float_features.shaderBufferFloat32Atomics == VK_TRUE) || (float_features.shaderBufferFloat32AtomicAdd == VK_TRUE) || |
| (float_features.shaderSharedFloat32Atomics == VK_TRUE) || (float_features.shaderSharedFloat32AtomicAdd == VK_TRUE) || |
| (float_features.shaderImageFloat32Atomics == VK_TRUE) || (float_features.shaderImageFloat32AtomicAdd == VK_TRUE) || |
| (float2_features.shaderBufferFloat32AtomicMinMax == VK_TRUE) || |
| (float2_features.shaderSharedFloat32AtomicMinMax == VK_TRUE) || |
| (float2_features.shaderImageFloat32AtomicMinMax == VK_TRUE)); |
| |
| const bool valid_64_float = |
| ((float_features.shaderBufferFloat64Atomics == VK_TRUE) || (float_features.shaderBufferFloat64AtomicAdd == VK_TRUE) || |
| (float_features.shaderSharedFloat64Atomics == VK_TRUE) || (float_features.shaderSharedFloat64AtomicAdd == VK_TRUE) || |
| (float2_features.shaderBufferFloat64AtomicMinMax == VK_TRUE) || |
| (float2_features.shaderSharedFloat64AtomicMinMax == VK_TRUE)); |
| // clang-format on |
| |
| for (const Instruction *atomic_def : module_state.static_data_.atomic_inst) { |
| const AtomicInstructionInfo &atomic = atomic_def->GetAtomicInfo(module_state); |
| const uint32_t opcode = atomic_def->Opcode(); |
| |
| if ((atomic.bit_width == 64) && (atomic.type == spv::OpTypeInt)) { |
| // Validate 64-bit image atomics |
| if (((atomic.storage_class == spv::StorageClassStorageBuffer) || (atomic.storage_class == spv::StorageClassUniform)) && |
| (enabled_features.core12.shaderBufferInt64Atomics == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06278", |
| "%s: Can't use 64-bit int atomics operations\n%s\nwith %s storage class without " |
| "shaderBufferInt64Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str(), |
| string_SpvStorageClass(atomic.storage_class)); |
| } else if ((atomic.storage_class == spv::StorageClassWorkgroup) && |
| (enabled_features.core12.shaderSharedInt64Atomics == VK_FALSE)) { |
| skip |= |
| LogError(device, "VUID-RuntimeSpirv-None-06279", |
| "%s: Can't use 64-bit int atomics operations\n%s\nwith Workgroup storage class without " |
| "shaderSharedInt64Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str()); |
| } else if ((atomic.storage_class == spv::StorageClassImage) && (valid_image_64_int == false)) { |
| skip |= |
| LogError(device, "VUID-RuntimeSpirv-None-06288", |
| "%s: Can't use 64-bit int atomics operations\n%s\nwith Image storage class without " |
| "shaderImageInt64Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str()); |
| } |
| } else if (atomic.type == spv::OpTypeFloat) { |
| // Validate Floats |
| if (atomic.storage_class == spv::StorageClassStorageBuffer) { |
| if (valid_storage_buffer_float == false) { |
| const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06284" |
| : "VUID-RuntimeSpirv-None-06280"; |
| skip |= LogError(device, vuid, |
| "%s: Can't use float atomics operations\n%s\nwith StorageBuffer storage class without " |
| "shaderBufferFloat32Atomics or shaderBufferFloat32AtomicAdd or shaderBufferFloat64Atomics or " |
| "shaderBufferFloat64AtomicAdd or shaderBufferFloat16Atomics or shaderBufferFloat16AtomicAdd " |
| "or shaderBufferFloat16AtomicMinMax or shaderBufferFloat32AtomicMinMax or " |
| "shaderBufferFloat64AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if (opcode == spv::OpAtomicFAddEXT) { |
| if ((atomic.bit_width == 16) && (float2_features.shaderBufferFloat16AtomicAdd == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06337", |
| "%s: Can't use 16-bit float atomics for add operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat16AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 32) && (float_features.shaderBufferFloat32AtomicAdd == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06338", |
| "%s: Can't use 32-bit float atomics for add operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat32AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 64) && (float_features.shaderBufferFloat64AtomicAdd == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06339", |
| "%s: Can't use 64-bit float atomics for add operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat64AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } |
| } else if (opcode == spv::OpAtomicFMinEXT || opcode == spv::OpAtomicFMaxEXT) { |
| if ((atomic.bit_width == 16) && (float2_features.shaderBufferFloat16AtomicMinMax == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06337", |
| "%s: Can't use 16-bit float atomics for min/max operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat16AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 32) && (float2_features.shaderBufferFloat32AtomicMinMax == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06338", |
| "%s: Can't use 32-bit float atomics for min/max operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat32AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 64) && (float2_features.shaderBufferFloat64AtomicMinMax == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06339", |
| "%s: Can't use 64-bit float atomics for min/max operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat64AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } |
| } else { |
| // Assume is valid load/store/exchange (rest of supported atomic operations) or else spirv-val will catch |
| if ((atomic.bit_width == 16) && (float2_features.shaderBufferFloat16Atomics == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06338", |
| "%s: Can't use 16-bit float atomics for load/store/exhange operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat16Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 32) && (float_features.shaderBufferFloat32Atomics == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06338", |
| "%s: Can't use 32-bit float atomics for load/store/exhange operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat32Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 64) && (float_features.shaderBufferFloat64Atomics == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06339", |
| "%s: Can't use 64-bit float atomics for load/store/exhange operations\n%s\nwith " |
| "StorageBuffer storage class without shaderBufferFloat64Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } |
| } |
| } else if (atomic.storage_class == spv::StorageClassWorkgroup) { |
| if (valid_workgroup_float == false) { |
| const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06285" |
| : "VUID-RuntimeSpirv-None-06281"; |
| skip |= LogError( |
| device, vuid, |
| "%s: Can't use float atomics operations\n%s\nwith Workgroup storage class without " |
| "shaderSharedFloat32Atomics or " |
| "shaderSharedFloat32AtomicAdd or shaderSharedFloat64Atomics or shaderSharedFloat64AtomicAdd or " |
| "shaderSharedFloat16Atomics or shaderSharedFloat16AtomicAdd or shaderSharedFloat16AtomicMinMax or " |
| "shaderSharedFloat32AtomicMinMax or shaderSharedFloat64AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str()); |
| } else if (opcode == spv::OpAtomicFAddEXT) { |
| if ((atomic.bit_width == 16) && (float2_features.shaderSharedFloat16AtomicAdd == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06337", |
| "%s: Can't use 16-bit float atomics for add operations\n%s\nwith Workgroup " |
| "storage class without shaderSharedFloat16AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 32) && (float_features.shaderSharedFloat32AtomicAdd == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06338", |
| "%s: Can't use 32-bit float atomics for add operations\n%s\nwith Workgroup " |
| "storage class without shaderSharedFloat32AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 64) && (float_features.shaderSharedFloat64AtomicAdd == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06339", |
| "%s: Can't use 64-bit float atomics for add operations\n%s\nwith Workgroup " |
| "storage class without shaderSharedFloat64AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } |
| } else if (opcode == spv::OpAtomicFMinEXT || opcode == spv::OpAtomicFMaxEXT) { |
| if ((atomic.bit_width == 16) && (float2_features.shaderSharedFloat16AtomicMinMax == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06337", |
| "%s: Can't use 16-bit float atomics for min/max operations\n%s\nwith " |
| "Workgroup storage class without shaderSharedFloat16AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 32) && (float2_features.shaderSharedFloat32AtomicMinMax == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06338", |
| "%s: Can't use 32-bit float atomics for min/max operations\n%s\nwith " |
| "Workgroup storage class without shaderSharedFloat32AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 64) && (float2_features.shaderSharedFloat64AtomicMinMax == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06339", |
| "%s: Can't use 64-bit float atomics for min/max operations\n%s\nwith " |
| "Workgroup storage class without shaderSharedFloat64AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } |
| } else { |
| // Assume is valid load/store/exchange (rest of supported atomic operations) or else spirv-val will catch |
| if ((atomic.bit_width == 16) && (float2_features.shaderSharedFloat16Atomics == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06337", |
| "%s: Can't use 16-bit float atomics for load/store/exhange operations\n%s\nwith Workgroup " |
| "storage class without shaderSharedFloat16Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 32) && (float_features.shaderSharedFloat32Atomics == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06338", |
| "%s: Can't use 32-bit float atomics for load/store/exhange operations\n%s\nwith Workgroup " |
| "storage class without shaderSharedFloat32Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 64) && (float_features.shaderSharedFloat64Atomics == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-None-06339", |
| "%s: Can't use 64-bit float atomics for load/store/exhange operations\n%s\nwith Workgroup " |
| "storage class without shaderSharedFloat64Atomics enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| atomic_def->Describe().c_str()); |
| } |
| } |
| } else if ((atomic.storage_class == spv::StorageClassImage) && (valid_image_float == false)) { |
| const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06286" |
| : "VUID-RuntimeSpirv-None-06282"; |
| skip |= LogError( |
| device, vuid, |
| "%s: Can't use float atomics operations\n%s\nwith Image storage class without shaderImageFloat32Atomics or " |
| "shaderImageFloat32AtomicAdd or shaderImageFloat32AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 16) && (valid_16_float == false)) { |
| skip |= |
| LogError(device, "VUID-RuntimeSpirv-None-06337", |
| "%s: Can't use 16-bit float atomics operations\n%s\nwithout shaderBufferFloat16Atomics, " |
| "shaderBufferFloat16AtomicAdd, shaderBufferFloat16AtomicMinMax, shaderSharedFloat16Atomics, " |
| "shaderSharedFloat16AtomicAdd or shaderSharedFloat16AtomicMinMax enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 32) && (valid_32_float == false)) { |
| const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06338" |
| : "VUID-RuntimeSpirv-None-06335"; |
| skip |= |
| LogError(device, vuid, |
| "%s: Can't use 32-bit float atomics operations\n%s\nwithout shaderBufferFloat32AtomicMinMax, " |
| "shaderSharedFloat32AtomicMinMax, shaderImageFloat32AtomicMinMax, sparseImageFloat32AtomicMinMax, " |
| "shaderBufferFloat32Atomics, shaderBufferFloat32AtomicAdd, shaderSharedFloat32Atomics, " |
| "shaderSharedFloat32AtomicAdd, shaderImageFloat32Atomics, shaderImageFloat32AtomicAdd, " |
| "sparseImageFloat32Atomics or sparseImageFloat32AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str()); |
| } else if ((atomic.bit_width == 64) && (valid_64_float == false)) { |
| const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06339" |
| : "VUID-RuntimeSpirv-None-06336"; |
| skip |= |
| LogError(device, vuid, |
| "%s: Can't use 64-bit float atomics operations\n%s\nwithout shaderBufferFloat64AtomicMinMax, " |
| "shaderSharedFloat64AtomicMinMax, shaderBufferFloat64Atomics, shaderBufferFloat64AtomicAdd, " |
| "shaderSharedFloat64Atomics or shaderSharedFloat64AtomicAdd enabled.", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), atomic_def->Describe().c_str()); |
| } |
| } |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateExecutionModes(const SHADER_MODULE_STATE &module_state, const EntryPoint &entrypoint, |
| VkShaderStageFlagBits stage, const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| |
| // Need to wrap otherwise phys_dev_props_core12 can be junk |
| if (IsExtEnabled(device_extensions.vk_khr_shader_float_controls)) { |
| if (entrypoint.execution_mode.Has(ExecutionModeSet::signed_zero_inf_nan_preserve_width_16) && |
| !phys_dev_props_core12.shaderSignedZeroInfNanPreserveFloat16) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderSignedZeroInfNanPreserveFloat16-06293", |
| "Shader requires SignedZeroInfNanPreserve for bit width 16 but it is not enabled on the device"); |
| } else if (entrypoint.execution_mode.Has(ExecutionModeSet::signed_zero_inf_nan_preserve_width_32) && |
| !phys_dev_props_core12.shaderSignedZeroInfNanPreserveFloat32) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderSignedZeroInfNanPreserveFloat32-06294", |
| "Shader requires SignedZeroInfNanPreserve for bit width 32 but it is not enabled on the device"); |
| } else if (entrypoint.execution_mode.Has(ExecutionModeSet::signed_zero_inf_nan_preserve_width_64) && |
| !phys_dev_props_core12.shaderSignedZeroInfNanPreserveFloat64) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderSignedZeroInfNanPreserveFloat64-06295", |
| "Shader requires SignedZeroInfNanPreserve for bit width 64 but it is not enabled on the device"); |
| } |
| |
| const bool has_denorm_preserve_width_16 = entrypoint.execution_mode.Has(ExecutionModeSet::denorm_preserve_width_16); |
| const bool has_denorm_preserve_width_32 = entrypoint.execution_mode.Has(ExecutionModeSet::denorm_preserve_width_32); |
| const bool has_denorm_preserve_width_64 = entrypoint.execution_mode.Has(ExecutionModeSet::denorm_preserve_width_64); |
| if (has_denorm_preserve_width_16 && !phys_dev_props_core12.shaderDenormPreserveFloat16) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderDenormPreserveFloat16-06296", |
| "Shader requires DenormPreserve for bit width 16 but it is not enabled on the device"); |
| } else if (has_denorm_preserve_width_32 && !phys_dev_props_core12.shaderDenormPreserveFloat32) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderDenormPreserveFloat32-06297", |
| "Shader requires DenormPreserve for bit width 32 but it is not enabled on the device"); |
| } else if (has_denorm_preserve_width_64 && !phys_dev_props_core12.shaderDenormPreserveFloat64) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderDenormPreserveFloat64-06298", |
| "Shader requires DenormPreserve for bit width 64 but it is not enabled on the device"); |
| } |
| |
| const bool has_denorm_flush_to_zero_width_16 = |
| entrypoint.execution_mode.Has(ExecutionModeSet::denorm_flush_to_zero_width_16); |
| const bool has_denorm_flush_to_zero_width_32 = |
| entrypoint.execution_mode.Has(ExecutionModeSet::denorm_flush_to_zero_width_32); |
| const bool has_denorm_flush_to_zero_width_64 = |
| entrypoint.execution_mode.Has(ExecutionModeSet::denorm_flush_to_zero_width_64); |
| if (has_denorm_flush_to_zero_width_16 && !phys_dev_props_core12.shaderDenormFlushToZeroFloat16) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderDenormFlushToZeroFloat16-06299", |
| "Shader requires DenormFlushToZero for bit width 16 but it is not enabled on the device"); |
| } else if (has_denorm_flush_to_zero_width_32 && !phys_dev_props_core12.shaderDenormFlushToZeroFloat32) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderDenormFlushToZeroFloat32-06300", |
| "Shader requires DenormFlushToZero for bit width 32 but it is not enabled on the device"); |
| } else if (has_denorm_flush_to_zero_width_64 && !phys_dev_props_core12.shaderDenormFlushToZeroFloat64) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderDenormFlushToZeroFloat64-06301", |
| "Shader requires DenormFlushToZero for bit width 64 but it is not enabled on the device"); |
| } |
| |
| const bool has_rounding_mode_rte_width_16 = entrypoint.execution_mode.Has(ExecutionModeSet::rounding_mode_rte_width_16); |
| const bool has_rounding_mode_rte_width_32 = entrypoint.execution_mode.Has(ExecutionModeSet::rounding_mode_rte_width_32); |
| const bool has_rounding_mode_rte_width_64 = entrypoint.execution_mode.Has(ExecutionModeSet::rounding_mode_rte_width_64); |
| if (has_rounding_mode_rte_width_16 && !phys_dev_props_core12.shaderRoundingModeRTEFloat16) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderRoundingModeRTEFloat16-06302", |
| "Shader requires RoundingModeRTE for bit width 16 but it is not enabled on the device"); |
| } else if (has_rounding_mode_rte_width_32 && !phys_dev_props_core12.shaderRoundingModeRTEFloat32) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderRoundingModeRTEFloat32-06303", |
| "Shader requires RoundingModeRTE for bit width 32 but it is not enabled on the device"); |
| } else if (has_rounding_mode_rte_width_64 && !phys_dev_props_core12.shaderRoundingModeRTEFloat64) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderRoundingModeRTEFloat64-06304", |
| "Shader requires RoundingModeRTE for bit width 64 but it is not enabled on the device"); |
| } |
| |
| const bool has_rounding_mode_rtz_width_16 = entrypoint.execution_mode.Has(ExecutionModeSet::rounding_mode_rtz_width_16); |
| const bool has_rounding_mode_rtz_width_32 = entrypoint.execution_mode.Has(ExecutionModeSet::rounding_mode_rtz_width_32); |
| const bool has_rounding_mode_rtz_width_64 = entrypoint.execution_mode.Has(ExecutionModeSet::rounding_mode_rtz_width_64); |
| if (has_rounding_mode_rtz_width_16 && !phys_dev_props_core12.shaderRoundingModeRTZFloat16) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderRoundingModeRTZFloat16-06305", |
| "Shader requires RoundingModeRTZ for bit width 16 but it is not enabled on the device"); |
| } else if (has_rounding_mode_rtz_width_32 && !phys_dev_props_core12.shaderRoundingModeRTZFloat32) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderRoundingModeRTZFloat32-06306", |
| "Shader requires RoundingModeRTZ for bit width 32 but it is not enabled on the device"); |
| } else if (has_rounding_mode_rtz_width_64 && !phys_dev_props_core12.shaderRoundingModeRTZFloat64) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-shaderRoundingModeRTZFloat64-06307", |
| "Shader requires RoundingModeRTZ for bit width 64 but it is not enabled on the device"); |
| } |
| } |
| |
| if (entrypoint.execution_mode.Has(ExecutionModeSet::local_size_id_bit)) { |
| if (!enabled_features.core13.maintenance4) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-LocalSizeId-06434", |
| "LocalSizeId execution mode used but maintenance4 feature not enabled"); |
| } |
| if (!IsExtEnabled(device_extensions.vk_khr_maintenance4)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-LocalSizeId-06433", |
| "LocalSizeId execution mode used but maintenance4 extension is not enabled and used " |
| "Vulkan api version is 1.2 or less"); |
| } |
| } |
| |
| if (entrypoint.execution_mode.Has(ExecutionModeSet::subgroup_uniform_control_flow_bit)) { |
| if (!enabled_features.shader_subgroup_uniform_control_flow_features.shaderSubgroupUniformControlFlow || |
| (phys_dev_ext_props.subgroup_props.supportedStages & stage) == 0 || |
| module_state.static_data_.has_invocation_repack_instruction) { |
| std::stringstream msg; |
| if (!enabled_features.shader_subgroup_uniform_control_flow_features.shaderSubgroupUniformControlFlow) { |
| msg << "shaderSubgroupUniformControlFlow feature must be enabled"; |
| } else if ((phys_dev_ext_props.subgroup_props.supportedStages & stage) == 0) { |
| msg << "stage" << string_VkShaderStageFlagBits(stage) |
| << " must be in VkPhysicalDeviceSubgroupProperties::supportedStages(" |
| << string_VkShaderStageFlags(phys_dev_ext_props.subgroup_props.supportedStages) << ")"; |
| } else { |
| msg << "the shader must not use any invocation repack instructions"; |
| } |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-SubgroupUniformControlFlowKHR-06379", |
| "If ExecutionModeSubgroupUniformControlFlowKHR is used %s.", msg.str().c_str()); |
| } |
| } |
| |
| if (entrypoint.stage == VK_SHADER_STAGE_GEOMETRY_BIT) { |
| const uint32_t vertices_out = entrypoint.execution_mode.output_vertices; |
| const uint32_t invocations = entrypoint.execution_mode.invocations; |
| if (vertices_out == 0 || vertices_out > phys_dev_props.limits.maxGeometryOutputVertices) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-stage-00714", |
| "Geometry shader entry point must have an OpExecutionMode instruction that " |
| "specifies a maximum output vertex count that is greater than 0 and less " |
| "than or equal to maxGeometryOutputVertices. " |
| "OutputVertices=%d, maxGeometryOutputVertices=%d", |
| vertices_out, phys_dev_props.limits.maxGeometryOutputVertices); |
| } |
| |
| if (invocations == 0 || invocations > phys_dev_props.limits.maxGeometryShaderInvocations) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-stage-00715", |
| "Geometry shader entry point must have an OpExecutionMode instruction that " |
| "specifies an invocation count that is greater than 0 and less " |
| "than or equal to maxGeometryShaderInvocations. " |
| "Invocations=%d, maxGeometryShaderInvocations=%d", |
| invocations, phys_dev_props.limits.maxGeometryShaderInvocations); |
| } |
| } else if (entrypoint.stage == VK_SHADER_STAGE_FRAGMENT_BIT && |
| entrypoint.execution_mode.Has(ExecutionModeSet::early_fragment_test_bit)) { |
| const auto *ds_state = pipeline.DepthStencilState(); |
| if ((ds_state && (ds_state->flags & |
| (VK_PIPELINE_DEPTH_STENCIL_STATE_CREATE_RASTERIZATION_ORDER_ATTACHMENT_DEPTH_ACCESS_BIT_EXT | |
| VK_PIPELINE_DEPTH_STENCIL_STATE_CREATE_RASTERIZATION_ORDER_ATTACHMENT_STENCIL_ACCESS_BIT_EXT)) != 0)) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-flags-06591", |
| "The fragment shader enables early fragment tests, but VkPipelineDepthStencilStateCreateInfo::flags == " |
| "%s", |
| string_VkPipelineDepthStencilStateCreateFlags(ds_state->flags).c_str()); |
| } |
| } |
| |
| return skip; |
| } |
| |
| // For given pipelineLayout verify that the set_layout_node at slot.first |
| // has the requested binding at slot.second and return ptr to that binding |
| static VkDescriptorSetLayoutBinding const *GetDescriptorBinding(PIPELINE_LAYOUT_STATE const *pipelineLayout, uint32_t set, |
| uint32_t binding) { |
| if (!pipelineLayout) return nullptr; |
| |
| if (set >= pipelineLayout->set_layouts.size()) return nullptr; |
| |
| return pipelineLayout->set_layouts[set]->GetDescriptorSetLayoutBindingPtrFromBinding(binding); |
| } |
| |
| bool CoreChecks::ValidatePointSizeShaderState(const PIPELINE_STATE &pipeline, const SHADER_MODULE_STATE &module_state, |
| const EntryPoint &entrypoint, VkShaderStageFlagBits stage) const { |
| bool skip = false; |
| // vkspec.html#primsrast-points describes which is the final stage that needs to check for points |
| // |
| // Vertex - Need to read input topology in pipeline |
| // Geo/Tess - Need to know the feature bit is on |
| // Mesh - are checked in spirv-val as they don't require any runtime information |
| if (stage != VK_SHADER_STAGE_VERTEX_BIT && stage != VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT && |
| stage != VK_SHADER_STAGE_GEOMETRY_BIT) { |
| return skip; |
| } |
| |
| const bool output_points = entrypoint.execution_mode.Has(ExecutionModeSet::output_points_bit); |
| const bool point_mode = entrypoint.execution_mode.Has(ExecutionModeSet::point_mode_bit); |
| |
| if (stage == VK_SHADER_STAGE_GEOMETRY_BIT && output_points) { |
| if (enabled_features.core.shaderTessellationAndGeometryPointSize && !entrypoint.written_builtin_point_size && |
| entrypoint.emit_vertex_geometry) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-Geometry-07725", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shaderTessellationAndGeometryPointSize is enabled, but PointSize is not " |
| "written in the Geometry shader.", |
| pipeline.create_index); |
| } else if (!enabled_features.core.shaderTessellationAndGeometryPointSize && entrypoint.written_builtin_point_size) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-Geometry-07726", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shaderTessellationAndGeometryPointSize is not enabled, but PointSize is " |
| "written to in the Geometry shader (gl_PointSize must NOT be written and a default of 1.0 is assumed).", |
| pipeline.create_index); |
| } |
| } else if (stage == VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT && |
| ((pipeline.create_info_shaders & VK_SHADER_STAGE_GEOMETRY_BIT) == 0) && point_mode) { |
| if (enabled_features.core.shaderTessellationAndGeometryPointSize && !entrypoint.written_builtin_point_size) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-TessellationEvaluation-07723", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shaderTessellationAndGeometryPointSize is enabled, but PointSize is not " |
| "written in the Tessellation Evaluation shader.", |
| pipeline.create_index); |
| } else if (!enabled_features.core.shaderTessellationAndGeometryPointSize && entrypoint.written_builtin_point_size) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-TessellationEvaluation-07724", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shaderTessellationAndGeometryPointSize is not enabled, but PointSize is written to " |
| "in the Tessellation Evaluation shader (gl_PointSize must NOT be written and a default of 1.0 is assumed).", |
| pipeline.create_index); |
| } |
| } else if (stage == VK_SHADER_STAGE_VERTEX_BIT && |
| ((pipeline.create_info_shaders & (VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT | VK_SHADER_STAGE_GEOMETRY_BIT)) == |
| 0) && |
| pipeline.topology_at_rasterizer == VK_PRIMITIVE_TOPOLOGY_POINT_LIST) { |
| if (!entrypoint.written_builtin_point_size) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-VkGraphicsPipelineCreateInfo-Vertex-07722", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] Pipeline topology is set to VK_PRIMITIVE_TOPOLOGY_POINT_LIST, but " |
| "PointSize is not written in the Vertex shader.", |
| pipeline.create_index); |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidatePrimitiveRateShaderState(const PIPELINE_STATE &pipeline, const SHADER_MODULE_STATE &module_state, |
| const EntryPoint &entrypoint, VkShaderStageFlagBits stage) const { |
| bool skip = false; |
| |
| const auto viewport_state = pipeline.ViewportState(); |
| if (!phys_dev_ext_props.fragment_shading_rate_props.primitiveFragmentShadingRateWithMultipleViewports && |
| (pipeline.pipeline_type == VK_PIPELINE_BIND_POINT_GRAPHICS) && viewport_state) { |
| if (!pipeline.IsDynamic(VK_DYNAMIC_STATE_VIEWPORT_WITH_COUNT) && viewport_state->viewportCount > 1 && |
| entrypoint.written_builtin_primitive_shading_rate_khr) { |
| skip |= LogError(module_state.vk_shader_module(), |
| "VUID-VkGraphicsPipelineCreateInfo-primitiveFragmentShadingRateWithMultipleViewports-04503", |
| "vkCreateGraphicsPipelines: pCreateInfos[%" PRIu32 |
| "] %s shader statically writes to PrimitiveShadingRateKHR built-in, but " |
| "multiple viewports " |
| "are used and the primitiveFragmentShadingRateWithMultipleViewports limit is not supported.", |
| pipeline.create_index, string_VkShaderStageFlagBits(stage)); |
| } |
| |
| if (entrypoint.written_builtin_primitive_shading_rate_khr && entrypoint.written_builtin_viewport_index) { |
| skip |= LogError(module_state.vk_shader_module(), |
| "VUID-VkGraphicsPipelineCreateInfo-primitiveFragmentShadingRateWithMultipleViewports-04504", |
| "vkCreateGraphicsPipelines: pCreateInfos[%" PRIu32 |
| "] %s shader statically writes to both PrimitiveShadingRateKHR and " |
| "ViewportIndex built-ins," |
| "but the primitiveFragmentShadingRateWithMultipleViewports limit is not supported.", |
| pipeline.create_index, string_VkShaderStageFlagBits(stage)); |
| } |
| |
| if (entrypoint.written_builtin_primitive_shading_rate_khr && entrypoint.written_builtin_viewport_mask_nv) { |
| skip |= LogError(module_state.vk_shader_module(), |
| "VUID-VkGraphicsPipelineCreateInfo-primitiveFragmentShadingRateWithMultipleViewports-04505", |
| "vkCreateGraphicsPipelines: pCreateInfos[%" PRIu32 |
| "] %s shader statically writes to both PrimitiveShadingRateKHR and " |
| "ViewportMaskNV built-ins," |
| "but the primitiveFragmentShadingRateWithMultipleViewports limit is not supported.", |
| pipeline.create_index, string_VkShaderStageFlagBits(stage)); |
| } |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateTransformFeedbackDecorations(const SHADER_MODULE_STATE &module_state, |
| const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| |
| std::vector<const Instruction *> xfb_streams; |
| std::vector<const Instruction *> xfb_buffers; |
| std::vector<const Instruction *> xfb_offsets; |
| |
| for (const Instruction *op_decorate : module_state.static_data_.decoration_inst) { |
| uint32_t decoration = op_decorate->Word(2); |
| if (decoration == spv::DecorationXfbStride) { |
| uint32_t stride = op_decorate->Word(3); |
| if (stride > phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataStride) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-XfbStride-06313", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shader uses transform feedback with xfb_stride (%" PRIu32 |
| ") greater than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackBufferDataStride (%" PRIu32 |
| ").", |
| pipeline.create_index, stride, |
| phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataStride); |
| } |
| } |
| if (decoration == spv::DecorationStream) { |
| xfb_streams.push_back(op_decorate); |
| uint32_t stream = op_decorate->Word(3); |
| if (stream >= phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Stream-06312", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] shader uses transform feedback with stream (%" PRIu32 |
| ") not less than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackStreams (%" PRIu32 ").", |
| pipeline.create_index, stream, phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams); |
| } |
| } |
| if (decoration == spv::DecorationXfbBuffer) { |
| xfb_buffers.push_back(op_decorate); |
| } |
| if (decoration == spv::DecorationOffset) { |
| xfb_offsets.push_back(op_decorate); |
| } |
| } |
| |
| // XfbBuffer, buffer data size |
| std::vector<std::pair<uint32_t, uint32_t>> buffer_data_sizes; |
| for (const Instruction *op_decorate : xfb_offsets) { |
| for (const Instruction *xfb_buffer : xfb_buffers) { |
| if (xfb_buffer->Word(1) == op_decorate->Word(1)) { |
| const auto offset = op_decorate->Word(3); |
| const Instruction *def = module_state.FindDef(xfb_buffer->Word(1)); |
| const auto size = module_state.GetTypeBytesSize(def); |
| const uint32_t buffer_data_size = offset + size; |
| if (buffer_data_size > phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataSize) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Offset-06308", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shader uses transform feedback with xfb_offset (%" PRIu32 ") + size of variable (%" PRIu32 |
| ") greater than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackBufferDataSize " |
| "(%" PRIu32 ").", |
| pipeline.create_index, offset, size, |
| phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataSize); |
| } |
| |
| bool found = false; |
| for (auto &bds : buffer_data_sizes) { |
| if (bds.first == xfb_buffer->Word(1)) { |
| bds.second = std::max(bds.second, buffer_data_size); |
| found = true; |
| break; |
| } |
| } |
| if (!found) { |
| buffer_data_sizes.emplace_back(xfb_buffer->Word(1), buffer_data_size); |
| } |
| |
| break; |
| } |
| } |
| } |
| |
| std::unordered_map<uint32_t, uint32_t> stream_data_size; |
| for (const Instruction *xfb_stream : xfb_streams) { |
| for (const auto &bds : buffer_data_sizes) { |
| if (xfb_stream->Word(1) == bds.first) { |
| uint32_t stream = xfb_stream->Word(3); |
| const auto itr = stream_data_size.find(stream); |
| if (itr != stream_data_size.end()) { |
| itr->second += bds.second; |
| } else { |
| stream_data_size.insert({stream, bds.second}); |
| } |
| } |
| } |
| } |
| |
| for (const auto &stream : stream_data_size) { |
| if (stream.second > phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreamDataSize) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-XfbBuffer-06309", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shader uses transform feedback with stream (%" PRIu32 ") having the sum of buffer data sizes (%" PRIu32 |
| ") not less than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackBufferDataSize " |
| "(%" PRIu32 ").", |
| pipeline.create_index, stream.first, stream.second, |
| phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataSize); |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateWorkgroupSharedMemory(const SHADER_MODULE_STATE &module_state, VkShaderStageFlagBits stage, |
| uint32_t total_workgroup_shared_memory) const { |
| bool skip = false; |
| |
| // If not found before with spec constants, find here |
| if (total_workgroup_shared_memory == 0) { |
| total_workgroup_shared_memory = module_state.CalculateWorkgroupSharedMemory(); |
| } |
| |
| switch (stage) { |
| case VK_SHADER_STAGE_COMPUTE_BIT: { |
| if (total_workgroup_shared_memory > phys_dev_props.limits.maxComputeSharedMemorySize) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-Workgroup-06530", |
| "Shader uses %" PRIu32 |
| " bytes of shared memory, more than allowed by physicalDeviceLimits::maxComputeSharedMemorySize (%" PRIu32 ")", |
| total_workgroup_shared_memory, phys_dev_props.limits.maxComputeSharedMemorySize); |
| } |
| break; |
| } |
| case VK_SHADER_STAGE_MESH_BIT_EXT: { |
| if (total_workgroup_shared_memory > phys_dev_ext_props.mesh_shader_props_ext.maxMeshSharedMemorySize) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-maxMeshSharedMemorySize-08754", |
| "Shader uses %" PRIu32 |
| " bytes of shared memory, more than allowed by maxMeshSharedMemorySize (%" PRIu32 ")", |
| total_workgroup_shared_memory, phys_dev_ext_props.mesh_shader_props_ext.maxMeshSharedMemorySize); |
| } |
| break; |
| } |
| case VK_SHADER_STAGE_TASK_BIT_EXT: { |
| if (total_workgroup_shared_memory > phys_dev_ext_props.mesh_shader_props_ext.maxTaskSharedMemorySize) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-maxTaskSharedMemorySize-08759", |
| "Shader uses %" PRIu32 |
| " bytes of shared memory, more than allowed by maxTaskSharedMemorySize (%" PRIu32 ")", |
| total_workgroup_shared_memory, phys_dev_ext_props.mesh_shader_props_ext.maxTaskSharedMemorySize); |
| } |
| break; |
| } |
| default: |
| assert(false); // other stages should not have called this function |
| break; |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderModuleId(const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| for (const auto &stage_ci : pipeline.shader_stages_ci) { |
| const auto module_identifier = LvlFindInChain<VkPipelineShaderStageModuleIdentifierCreateInfoEXT>(stage_ci.pNext); |
| const auto module_create_info = LvlFindInChain<VkShaderModuleCreateInfo>(stage_ci.pNext); |
| if (module_identifier) { |
| if (module_identifier->identifierSize > 0) { |
| if (!(enabled_features.shader_module_identifier_features.shaderModuleIdentifier)) { |
| skip |= LogError(device, "VUID-VkPipelineShaderStageModuleIdentifierCreateInfoEXT-pNext-06850", |
| "%s pCreateInfos[%" PRIu32 |
| "] module (stage %s) VkPipelineShaderStageCreateInfo has a " |
| "VkPipelineShaderStageModuleIdentifierCreateInfoEXT " |
| "struct in the pNext chain but the shaderModuleIdentifier feature is not enabled", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, |
| string_VkShaderStageFlagBits(stage_ci.stage)); |
| } |
| if (!(pipeline.create_flags & VK_PIPELINE_CREATE_FAIL_ON_PIPELINE_COMPILE_REQUIRED_BIT)) { |
| skip |= LogError( |
| device, "VUID-VkPipelineShaderStageModuleIdentifierCreateInfoEXT-pNext-06851", |
| "%s pCreateInfos[%" PRIu32 |
| "] module (stage %s) VkPipelineShaderStageCreateInfo has a " |
| "VkPipelineShaderStageModuleIdentifierCreateInfoEXT " |
| "struct in the pNext chain whose identifierSize is > 0 (%" PRIu32 |
| "), but the " |
| "VK_PIPELINE_CREATE_FAIL_ON_PIPELINE_COMPILE_REQUIRED_BIT bit is not set in the pipeline create flags", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, string_VkShaderStageFlagBits(stage_ci.stage), |
| module_identifier->identifierSize); |
| } |
| if (module_identifier->identifierSize > VK_MAX_SHADER_MODULE_IDENTIFIER_SIZE_EXT) { |
| skip |= LogError(device, "VUID-VkPipelineShaderStageModuleIdentifierCreateInfoEXT-identifierSize-06852", |
| "%s pCreateInfos[%" PRIu32 |
| "] module (stage %s) VkPipelineShaderStageCreateInfo has a " |
| "VkPipelineShaderStageModuleIdentifierCreateInfoEXT " |
| "struct in the pNext chain whose identifierSize (%" PRIu32 |
| ") is > VK_MAX_SHADER_MODULE_IDENTIFIER_SIZE_EXT (%" PRIu32 ")", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, |
| string_VkShaderStageFlagBits(stage_ci.stage), module_identifier->identifierSize, |
| VK_MAX_SHADER_MODULE_IDENTIFIER_SIZE_EXT); |
| } |
| } |
| if (module_create_info) { |
| skip |= |
| LogError(device, "VUID-VkPipelineShaderStageCreateInfo-stage-06844", |
| "%s pCreateInfos[%" PRIu32 |
| "] module (stage %s) VkPipelineShaderStageCreateInfo has both a " |
| "VkPipelineShaderStageModuleIdentifierCreateInfoEXT " |
| "struct and a VkShaderModuleCreateInfo struct in the pNext chain", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, string_VkShaderStageFlagBits(stage_ci.stage)); |
| } |
| if (stage_ci.module != VK_NULL_HANDLE) { |
| skip |= LogError( |
| device, "VUID-VkPipelineShaderStageCreateInfo-stage-06848", |
| "%s pCreateInfos[%" PRIu32 |
| "] module (stage %s) VkPipelineShaderStageCreateInfo has a VkPipelineShaderStageModuleIdentifierCreateInfoEXT " |
| "struct in the pNext chain, and module is not VK_NULL_HANDLE", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, string_VkShaderStageFlagBits(stage_ci.stage)); |
| } |
| } else { |
| if (enabled_features.graphics_pipeline_library_features.graphicsPipelineLibrary) { |
| if (stage_ci.module == VK_NULL_HANDLE && !module_create_info) { |
| skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-stage-06845", |
| "%s pCreateInfos[%" PRIu32 |
| "] module (stage %s) VkPipelineShaderStageCreateInfo has no " |
| "VkPipelineShaderStageModuleIdentifierCreateInfoEXT " |
| "struct and no VkShaderModuleCreateInfo struct in the pNext chain, and module is not a valid " |
| "VkShaderModule", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, |
| string_VkShaderStageFlagBits(stage_ci.stage)); |
| } |
| } else if (stage_ci.module == VK_NULL_HANDLE) { |
| const char *vuid = IsExtEnabled(device_extensions.vk_khr_pipeline_library) |
| ? "VUID-VkPipelineShaderStageCreateInfo-stage-06846" |
| : "VUID-VkPipelineShaderStageCreateInfo-stage-06847"; |
| skip |= LogError( |
| device, vuid, |
| "%s pCreateInfos[%" PRIu32 |
| "] module (stage %s) VkPipelineShaderStageCreateInfo has no VkPipelineShaderStageModuleIdentifierCreateInfoEXT " |
| "struct in the pNext chain, the graphicsPipelineLibrary feature is not enabled, and module is not a valid " |
| "VkShaderModule", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, string_VkShaderStageFlagBits(stage_ci.stage)); |
| } |
| } |
| } |
| return skip; |
| } |
| |
| // Temporary data of a OpVariable when validating it. |
| // If found useful in another location, can move out to the header |
| struct VariableInstInfo { |
| bool has_8bit = false; |
| bool has_16bit = false; |
| }; |
| |
| // easier to use recursion to traverse the OpTypeStruct |
| static void GetVariableInfo(const SHADER_MODULE_STATE &module_state, const Instruction *insn, VariableInstInfo &info) { |
| if (!insn) { |
| return; |
| } else if (insn->Opcode() == spv::OpTypeFloat || insn->Opcode() == spv::OpTypeInt) { |
| const uint32_t bit_width = insn->Word(2); |
| info.has_8bit |= (bit_width == 8); |
| info.has_16bit |= (bit_width == 16); |
| } else if (insn->Opcode() == spv::OpTypeStruct) { |
| for (uint32_t i = 2; i < insn->Length(); i++) { |
| const Instruction *base_insn = module_state.GetBaseTypeInstruction(insn->Word(i)); |
| GetVariableInfo(module_state, base_insn, info); |
| } |
| } |
| } |
| |
| bool CoreChecks::ValidateVariables(const SHADER_MODULE_STATE &module_state) const { |
| bool skip = false; |
| |
| for (const Instruction *insn : module_state.static_data_.variable_inst) { |
| const uint32_t storage_class = insn->StorageClass(); |
| |
| if (storage_class == spv::StorageClassWorkgroup) { |
| // If Workgroup variable is initalized, make sure the feature is enabled |
| if (insn->Length() > 4 && !enabled_features.core13.shaderZeroInitializeWorkgroupMemory) { |
| const char *vuid = IsExtEnabled(device_extensions.vk_khr_zero_initialize_workgroup_memory) |
| ? "VUID-RuntimeSpirv-shaderZeroInitializeWorkgroupMemory-06372" |
| : "VUID-RuntimeSpirv-OpVariable-06373"; |
| skip |= LogError( |
| module_state.vk_shader_module(), vuid, |
| "vkCreateShaderModule(): " |
| "VkPhysicalDeviceZeroInitializeWorkgroupMemoryFeaturesKHR::shaderZeroInitializeWorkgroupMemory is not enabled, " |
| "but shader contains an OpVariable with Workgroup Storage Class with an Initializer operand.\n%s", |
| insn->Describe().c_str()); |
| } |
| } |
| |
| const Instruction *type_pointer = module_state.FindDef(insn->Word(1)); |
| const Instruction *type = module_state.FindDef(type_pointer->Word(3)); |
| // type will either be a float, int, or struct and if struct need to traverse it |
| VariableInstInfo info; |
| GetVariableInfo(module_state, type, info); |
| |
| if (info.has_8bit) { |
| if (!enabled_features.core12.storageBuffer8BitAccess && |
| (storage_class == spv::StorageClassStorageBuffer || storage_class == spv::StorageClassShaderRecordBufferKHR || |
| storage_class == spv::StorageClassPhysicalStorageBuffer)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-storageBuffer8BitAccess-06328", |
| "vkCreateShaderModule(): storageBuffer8BitAccess is not enabled, but shader contains an 8-bit " |
| "OpVariable with %s Storage Class.\n%s", |
| string_SpvStorageClass(storage_class), insn->Describe().c_str()); |
| } |
| if (!enabled_features.core12.uniformAndStorageBuffer8BitAccess && storage_class == spv::StorageClassUniform) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-uniformAndStorageBuffer8BitAccess-06329", |
| "vkCreateShaderModule(): uniformAndStorageBuffer8BitAccess is not enabled, but shader contains an " |
| "8-bit OpVariable with Uniform Storage Class.\n%s", |
| insn->Describe().c_str()); |
| } |
| if (!enabled_features.core12.storagePushConstant8 && storage_class == spv::StorageClassPushConstant) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-storagePushConstant8-06330", |
| "vkCreateShaderModule(): storagePushConstant8 is not enabled, but shader contains an 8-bit " |
| "OpVariable with PushConstant Storage Class.\n%s", |
| insn->Describe().c_str()); |
| } |
| } |
| |
| if (info.has_16bit) { |
| if (!enabled_features.core11.storageBuffer16BitAccess && |
| (storage_class == spv::StorageClassStorageBuffer || storage_class == spv::StorageClassShaderRecordBufferKHR || |
| storage_class == spv::StorageClassPhysicalStorageBuffer)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-storageBuffer16BitAccess-06331", |
| "vkCreateShaderModule(): storageBuffer16BitAccess is not enabled, but shader contains an 16-bit " |
| "OpVariable with %s Storage Class.\n%s", |
| string_SpvStorageClass(storage_class), insn->Describe().c_str()); |
| } |
| if (!enabled_features.core11.uniformAndStorageBuffer16BitAccess && storage_class == spv::StorageClassUniform) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-uniformAndStorageBuffer16BitAccess-06332", |
| "vkCreateShaderModule(): uniformAndStorageBuffer16BitAccess is not enabled, but shader contains an " |
| "16-bit OpVariable with Uniform Storage Class.\n%s", |
| insn->Describe().c_str()); |
| } |
| if (!enabled_features.core11.storagePushConstant16 && storage_class == spv::StorageClassPushConstant) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-storagePushConstant16-06333", |
| "vkCreateShaderModule(): storagePushConstant16 is not enabled, but shader contains an 16-bit " |
| "OpVariable with PushConstant Storage Class.\n%s", |
| insn->Describe().c_str()); |
| } |
| if (!enabled_features.core11.storageInputOutput16 && |
| (storage_class == spv::StorageClassInput || storage_class == spv::StorageClassOutput)) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-storageInputOutput16-06334", |
| "vkCreateShaderModule(): storageInputOutput16 is not enabled, but shader contains an 16-bit " |
| "OpVariable with %s Storage Class.\n%s", |
| string_SpvStorageClass(storage_class), insn->Describe().c_str()); |
| } |
| } |
| |
| // Checks based off shaderStorageImage(Read|Write)WithoutFormat are |
| // disabled if VK_KHR_format_feature_flags2 is supported. |
| // |
| // https://github.com/KhronosGroup/Vulkan-Docs/blob/6177645341afc/appendices/spirvenv.txt#L553 |
| // |
| // The other checks need to take into account the format features and so |
| // we apply that in the descriptor set matching validation code (see |
| // descriptor_sets.cpp). |
| if (!has_format_feature2) { |
| skip |= ValidateShaderStorageImageFormatsVariables(module_state, insn); |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderDescriptorVariable(const SHADER_MODULE_STATE &module_state, VkShaderStageFlagBits stage, |
| const PIPELINE_STATE &pipeline, const EntryPoint &entrypoint) const { |
| bool skip = false; |
| |
| std::string vuid_07988; |
| std::string vuid_07989; |
| std::string vuid_07990; |
| std::string vuid_07991; |
| switch (pipeline.GetCreateInfoSType()) { |
| case VK_STRUCTURE_TYPE_GRAPHICS_PIPELINE_CREATE_INFO: |
| vuid_07988 = "VUID-VkGraphicsPipelineCreateInfo-layout-07988"; |
| vuid_07989 = "VUID-VkGraphicsPipelineCreateInfo-layout-07989"; |
| vuid_07990 = "VUID-VkGraphicsPipelineCreateInfo-layout-07990"; |
| vuid_07991 = "VUID-VkGraphicsPipelineCreateInfo-layout-07991"; |
| break; |
| case VK_STRUCTURE_TYPE_COMPUTE_PIPELINE_CREATE_INFO: |
| vuid_07988 = "VUID-VkComputePipelineCreateInfo-layout-07988"; |
| vuid_07989 = "VUID-VkComputePipelineCreateInfo-layout-07989"; |
| vuid_07990 = "VUID-VkComputePipelineCreateInfo-layout-07990"; |
| vuid_07991 = "VUID-VkComputePipelineCreateInfo-layout-07991"; |
| break; |
| case VK_STRUCTURE_TYPE_RAY_TRACING_PIPELINE_CREATE_INFO_KHR: |
| vuid_07988 = "VUID-VkRayTracingPipelineCreateInfoKHR-layout-07988"; |
| vuid_07989 = "VUID-VkRayTracingPipelineCreateInfoKHR-layout-07989"; |
| vuid_07990 = "VUID-VkRayTracingPipelineCreateInfoKHR-layout-07990"; |
| vuid_07991 = "VUID-VkRayTracingPipelineCreateInfoKHR-layout-07991"; |
| break; |
| case VK_STRUCTURE_TYPE_RAY_TRACING_PIPELINE_CREATE_INFO_NV: |
| vuid_07988 = "VUID-VkRayTracingPipelineCreateInfoNV-layout-07988"; |
| vuid_07989 = "VUID-VkRayTracingPipelineCreateInfoNV-layout-07989"; |
| vuid_07990 = "VUID-VkRayTracingPipelineCreateInfoNV-layout-07990"; |
| vuid_07991 = "VUID-VkRayTracingPipelineCreateInfoNV-layout-07991"; |
| break; |
| default: |
| assert(false); |
| break; |
| } |
| |
| for (const auto &variable : entrypoint.resource_interface_variables) { |
| const auto &binding = |
| GetDescriptorBinding(pipeline.PipelineLayoutState().get(), variable.decorations.set, variable.decorations.binding); |
| uint32_t required_descriptor_count; |
| const bool is_khr = binding && binding->descriptorType == VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_KHR; |
| const auto descriptor_type_set = TypeToDescriptorTypeSet(module_state, variable.type_id, required_descriptor_count, is_khr); |
| |
| if (!binding) { |
| const LogObjectList objlist(module_state.vk_shader_module(), pipeline.PipelineLayoutState()->layout()); |
| skip |= LogError(objlist, vuid_07988, |
| "%s(): pCreateInfos[%" PRIu32 "] Set %" PRIu32 " Binding %" PRIu32 |
| " in shader (%s) uses descriptor slot (expected `%s`) but not declared in pipeline layout", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, variable.decorations.set, |
| variable.decorations.binding, string_VkShaderStageFlagBits(variable.stage), |
| string_descriptorTypeSet(descriptor_type_set).c_str()); |
| } else if (~binding->stageFlags & stage) { |
| const LogObjectList objlist(module_state.vk_shader_module(), pipeline.PipelineLayoutState()->layout()); |
| skip |= LogError(objlist, vuid_07988, |
| "%s(): pCreateInfos[%" PRIu32 "] Set %" PRIu32 " Binding %" PRIu32 |
| " in shader (%s) uses descriptor slot but descriptor not accessible from stage %s", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, variable.decorations.set, |
| variable.decorations.binding, string_VkShaderStageFlagBits(variable.stage), |
| string_VkShaderStageFlagBits(stage)); |
| } else if ((binding->descriptorType != VK_DESCRIPTOR_TYPE_MUTABLE_EXT) && |
| (descriptor_type_set.find(binding->descriptorType) == descriptor_type_set.end())) { |
| std::string vuid = (IsExtEnabled(device_extensions.vk_valve_mutable_descriptor_type) || |
| IsExtEnabled(device_extensions.vk_ext_mutable_descriptor_type)) |
| ? vuid_07990 |
| : vuid_07989; |
| const LogObjectList objlist(module_state.vk_shader_module(), pipeline.PipelineLayoutState()->layout()); |
| skip |= |
| LogError(objlist, vuid, |
| "%s(): pCreateInfos[%" PRIu32 "] Set %" PRIu32 " Binding %" PRIu32 |
| " type mismatch on descriptor slot in shader (%s), uses type %s but expected %s", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, variable.decorations.set, |
| variable.decorations.binding, string_VkShaderStageFlagBits(variable.stage), |
| string_VkDescriptorType(binding->descriptorType), string_descriptorTypeSet(descriptor_type_set).c_str()); |
| } else if (binding->descriptorCount < required_descriptor_count) { |
| const LogObjectList objlist(module_state.vk_shader_module(), pipeline.PipelineLayoutState()->layout()); |
| skip |= LogError(objlist, vuid_07991, |
| "%s(): pCreateInfos[%" PRIu32 "] Set %" PRIu32 " Binding %" PRIu32 |
| " in shader (%s) expects at least %" PRIu32 " descriptors, but only %" PRIu32 " provided", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, variable.decorations.set, |
| variable.decorations.binding, string_VkShaderStageFlagBits(variable.stage), required_descriptor_count, |
| binding->descriptorCount); |
| } |
| |
| if ((variable.is_storage_image || variable.is_storage_texel_buffer || variable.is_storage_buffer) && |
| !variable.decorations.Has(DecorationSet::nonwritable_bit)) { |
| switch (variable.stage) { |
| case VK_SHADER_STAGE_FRAGMENT_BIT: |
| skip |= RequireFeature(module_state, enabled_features.core.fragmentStoresAndAtomics, "fragmentStoresAndAtomics", |
| "VUID-RuntimeSpirv-NonWritable-06340"); |
| break; |
| case VK_SHADER_STAGE_VERTEX_BIT: |
| case VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT: |
| case VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT: |
| case VK_SHADER_STAGE_GEOMETRY_BIT: |
| skip |= RequireFeature(module_state, enabled_features.core.vertexPipelineStoresAndAtomics, |
| "vertexPipelineStoresAndAtomics", "VUID-RuntimeSpirv-NonWritable-06341"); |
| break; |
| default: |
| // No feature requirements for writes and atomics for other stages |
| break; |
| } |
| } |
| |
| if (variable.decorations.Has(DecorationSet::input_attachment_bit)) { |
| skip |= ValidateShaderInputAttachment(module_state, pipeline, variable); |
| } |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateTransformFeedback(const SHADER_MODULE_STATE &module_state, const EntryPoint &entrypoint, |
| const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| |
| if (!enabled_features.transform_feedback_features.transformFeedback) { |
| return skip; // most apps will not use transform feedback, so only check if enabled |
| } |
| skip |= ValidateTransformFeedbackDecorations(module_state, pipeline); |
| |
| if (entrypoint.stage != VK_SHADER_STAGE_GEOMETRY_BIT) { |
| return skip; // GeometryStreams are only used in Geomtry Shaders |
| } |
| |
| vvl::unordered_set<uint32_t> emitted_streams; |
| for (const Instruction *insn : module_state.static_data_.transform_feedback_stream_inst) { |
| const uint32_t opcode = insn->Opcode(); |
| if (opcode == spv::OpEmitStreamVertex) { |
| emitted_streams.emplace(module_state.GetConstantValueById(insn->Word(1))); |
| } |
| if (opcode == spv::OpEmitStreamVertex || opcode == spv::OpEndStreamPrimitive) { |
| uint32_t stream = module_state.GetConstantValueById(insn->Word(1)); |
| if (stream >= phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpEmitStreamVertex-06310", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] shader uses transform feedback stream\n%s\nwith index %" PRIu32 |
| ", which is not less than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackStreams (%" PRIu32 |
| ").", |
| pipeline.create_index, insn->Describe().c_str(), stream, |
| phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams); |
| } |
| } |
| } |
| |
| const bool output_points = entrypoint.execution_mode.Has(ExecutionModeSet::output_points_bit); |
| const uint32_t emitted_streams_size = static_cast<uint32_t>(emitted_streams.size()); |
| if (emitted_streams_size > 1 && !output_points && |
| phys_dev_ext_props.transform_feedback_props.transformFeedbackStreamsLinesTriangles == VK_FALSE) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-transformFeedbackStreamsLinesTriangles-06311", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] shader emits to %" PRIu32 |
| " vertex streams and VkPhysicalDeviceTransformFeedbackPropertiesEXT::transformFeedbackStreamsLinesTriangles " |
| "is VK_FALSE, but execution mode is not OutputPoints.", |
| pipeline.create_index, emitted_streams_size); |
| } |
| |
| return skip; |
| } |
| |
| // Checks for both TexelOffset and TexelGatherOffset limits |
| bool CoreChecks::ValidateTexelOffsetLimits(const SHADER_MODULE_STATE &module_state, const Instruction &insn) const { |
| bool skip = false; |
| |
| const uint32_t opcode = insn.Opcode(); |
| if (ImageGatherOperation(opcode) || ImageSampleOperation(opcode) || ImageFetchOperation(opcode)) { |
| uint32_t image_operand_position = OpcodeImageOperandsPosition(opcode); |
| // Image operands can be optional |
| if (image_operand_position != 0 && insn.Length() > image_operand_position) { |
| auto image_operand = insn.Word(image_operand_position); |
| // Bits we are validating (sample/fetch only check ConstOffset) |
| uint32_t offset_bits = |
| ImageGatherOperation(opcode) |
| ? (spv::ImageOperandsOffsetMask | spv::ImageOperandsConstOffsetMask | spv::ImageOperandsConstOffsetsMask) |
| : (spv::ImageOperandsConstOffsetMask); |
| if (image_operand & (offset_bits)) { |
| // Operand values follow |
| uint32_t index = image_operand_position + 1; |
| // Each bit has it's own operand, starts with the smallest set bit and loop to the highest bit among |
| // ImageOperandsOffsetMask, ImageOperandsConstOffsetMask and ImageOperandsConstOffsetsMask |
| for (uint32_t i = 1; i < spv::ImageOperandsConstOffsetsMask; i <<= 1) { |
| if (image_operand & i) { // If the bit is set, consume operand |
| if (insn.Length() > index && (i & offset_bits)) { |
| uint32_t constant_id = insn.Word(index); |
| const Instruction *constant = module_state.FindDef(constant_id); |
| const bool is_dynamic_offset = constant == nullptr; |
| if (!is_dynamic_offset && constant->Opcode() == spv::OpConstantComposite) { |
| for (uint32_t j = 3; j < constant->Length(); ++j) { |
| uint32_t comp_id = constant->Word(j); |
| const Instruction *comp = module_state.FindDef(comp_id); |
| const Instruction *comp_type = module_state.FindDef(comp->Word(1)); |
| // Get operand value |
| const uint32_t offset = comp->Word(3); |
| // spec requires minTexelGatherOffset/minTexelOffset to be -8 or less so never can compare if |
| // unsigned spec requires maxTexelGatherOffset/maxTexelOffset to be 7 or greater so never can |
| // compare if signed is less then zero |
| const int32_t signed_offset = static_cast<int32_t>(offset); |
| const bool use_signed = (comp_type->Opcode() == spv::OpTypeInt && comp_type->Word(3) != 0); |
| |
| // There are 2 sets of VU being covered where the only main difference is the opcode |
| if (ImageGatherOperation(opcode)) { |
| // min/maxTexelGatherOffset |
| if (use_signed && (signed_offset < phys_dev_props.limits.minTexelGatherOffset)) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpImage-06376", |
| "vkCreateShaderModule(): Shader uses\n%s\nwith offset (%" PRIi32 |
| ") less than VkPhysicalDeviceLimits::minTexelGatherOffset (%" PRIi32 ").", |
| insn.Describe().c_str(), signed_offset, phys_dev_props.limits.minTexelGatherOffset); |
| } else if ((offset > phys_dev_props.limits.maxTexelGatherOffset) && |
| (!use_signed || (use_signed && signed_offset > 0))) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpImage-06377", |
| "vkCreateShaderModule(): Shader uses\n%s\nwith offset (%" PRIu32 |
| ") greater than VkPhysicalDeviceLimits::maxTexelGatherOffset (%" PRIu32 ").", |
| insn.Describe().c_str(), offset, phys_dev_props.limits.maxTexelGatherOffset); |
| } |
| } else { |
| // min/maxTexelOffset |
| if (use_signed && (signed_offset < phys_dev_props.limits.minTexelOffset)) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpImageSample-06435", |
| "vkCreateShaderModule(): Shader uses\n%s\nwith offset (%" PRIi32 |
| ") less than VkPhysicalDeviceLimits::minTexelOffset (%" PRIi32 ").", |
| insn.Describe().c_str(), signed_offset, phys_dev_props.limits.minTexelOffset); |
| } else if ((offset > phys_dev_props.limits.maxTexelOffset) && |
| (!use_signed || (use_signed && signed_offset > 0))) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-OpImageSample-06436", |
| "vkCreateShaderModule(): Shader uses\n%s\nwith offset (%" PRIu32 |
| ") greater than VkPhysicalDeviceLimits::maxTexelOffset (%" PRIu32 ").", |
| insn.Describe().c_str(), offset, phys_dev_props.limits.maxTexelOffset); |
| } |
| } |
| } |
| } |
| } |
| index += ImageOperandsParamCount(i); |
| } |
| } |
| } |
| } |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateShaderClock(const SHADER_MODULE_STATE &module_state) const { |
| bool skip = false; |
| |
| for (const Instruction *group_inst : module_state.static_data_.read_clock_inst) { |
| const Instruction &insn = *group_inst; |
| const Instruction *scope_id = module_state.FindDef(insn.Word(3)); |
| auto scope_type = scope_id->Word(3); |
| // if scope isn't Subgroup or Device, spirv-val will catch |
| if ((scope_type == spv::ScopeSubgroup) && (enabled_features.shader_clock_features.shaderSubgroupClock == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-shaderSubgroupClock-06267", |
| "%s: OpReadClockKHR is used with a Subgroup scope but shaderSubgroupClock was not enabled.\n%s", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), insn.Describe().c_str()); |
| } else if ((scope_type == spv::ScopeDevice) && (enabled_features.shader_clock_features.shaderDeviceClock == VK_FALSE)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-shaderDeviceClock-06268", |
| "%s: OpReadClockKHR is used with a Device scope but shaderDeviceClock was not enabled.\n%s", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), insn.Describe().c_str()); |
| } |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateImageWrite(const SHADER_MODULE_STATE &module_state) const { |
| bool skip = false; |
| for (const auto &image_write : module_state.static_data_.image_write_load_id_map) { |
| const Instruction &insn = *image_write.first; |
| // guaranteed by spirv-val to be an OpTypeImage |
| const uint32_t image = module_state.GetTypeId(image_write.second); |
| const Instruction *image_def = module_state.FindDef(image); |
| const uint32_t image_format = image_def->Word(8); |
| // If format is 'Unknown' then need to wait until a descriptor is bound to it |
| if (image_format != spv::ImageFormatUnknown) { |
| const VkFormat compatible_format = CompatibleSpirvImageFormat(image_format); |
| if (compatible_format != VK_FORMAT_UNDEFINED) { |
| const uint32_t format_component_count = FormatComponentCount(compatible_format); |
| const uint32_t texel_component_count = module_state.GetTexelComponentCount(insn); |
| if (texel_component_count < format_component_count) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-OpImageWrite-07112", |
| "%s: OpImageWrite Texel operand only contains %" PRIu32 |
| " components, but the OpImage format mapping to %s has %" PRIu32 " components.\n%s\n%s", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), texel_component_count, |
| string_VkFormat(compatible_format), format_component_count, insn.Describe().c_str(), |
| image_def->Describe().c_str()); |
| } |
| } |
| } |
| } |
| return skip; |
| } |
| |
| static const std::string GetShaderTileImageCapabilitiesString(const SHADER_MODULE_STATE &module_state) { |
| struct SpvCapabilityWithString { |
| const spv::Capability cap; |
| const std::string cap_string; |
| }; |
| |
| // Shader tile image capabilities |
| static const std::array<SpvCapabilityWithString, 3> shader_tile_image_capabilities = { |
| {{spv::CapabilityTileImageColorReadAccessEXT, "TileImageColorReadAccessEXT"}, |
| {spv::CapabilityTileImageDepthReadAccessEXT, "TileImageDepthReadAccessEXT"}, |
| {spv::CapabilityTileImageStencilReadAccessEXT, "TileImageStencilReadAccessEXT"}}}; |
| |
| std::stringstream ss_capabilities; |
| for (auto spv_capability : shader_tile_image_capabilities) { |
| if (module_state.HasCapability(spv_capability.cap)) { |
| if (ss_capabilities.tellp()) ss_capabilities << ", "; |
| ss_capabilities << spv_capability.cap_string; |
| } |
| } |
| |
| return ss_capabilities.str(); |
| } |
| |
| bool CoreChecks::ValidateShaderTileImage(const SHADER_MODULE_STATE &module_state, const EntryPoint &entrypoint, |
| const PIPELINE_STATE &pipeline, const VkShaderStageFlagBits stage) const { |
| bool skip = false; |
| |
| if ((stage != VK_SHADER_STAGE_FRAGMENT_BIT) && !IsExtEnabled(device_extensions.vk_ext_shader_tile_image)) { |
| return skip; |
| } |
| |
| const bool using_tile_image_capability = module_state.HasCapability(spv::CapabilityTileImageColorReadAccessEXT) || |
| module_state.HasCapability(spv::CapabilityTileImageDepthReadAccessEXT) || |
| module_state.HasCapability(spv::CapabilityTileImageStencilReadAccessEXT); |
| |
| if (!using_tile_image_capability) { |
| // None of the capabilities exist. |
| return skip; |
| } |
| |
| if (pipeline.GetCreateInfo<VkGraphicsPipelineCreateInfo>().renderPass != VK_NULL_HANDLE) { |
| skip |= LogError( |
| device, "VUID-VkGraphicsPipelineCreateInfo-renderPass-08710", |
| "%s(): pCreateInfos[%" PRIu32 "] Fragment shader is using capabilities ( %s ), then renderpass must be VK_NULL_HANDLE.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, GetShaderTileImageCapabilitiesString(module_state).c_str()); |
| } |
| |
| const bool mode_early_fragment_test = entrypoint.execution_mode.Has(ExecutionModeSet::early_fragment_test_bit); |
| if (module_state.static_data_.has_shader_tile_image_depth_read) { |
| skip |= RequireFeature(module_state, enabled_features.shader_tile_image_features.shaderTileImageDepthReadAccess, |
| "shaderTileImageDepthReadAccess", "VUID-RuntimeSpirv-shaderTileImageDepthReadAccess-08729"); |
| |
| const auto *ds_state = pipeline.DepthStencilState(); |
| const bool write_enabled = |
| !pipeline.IsDynamic(VK_DYNAMIC_STATE_DEPTH_WRITE_ENABLE) && (ds_state && ds_state->depthWriteEnable); |
| if (mode_early_fragment_test && write_enabled) { |
| skip |= LogError(device, "VUID-VkGraphicsPipelineCreateInfo-pStages-08711", |
| "%s(): pCreateInfos[%" PRIu32 |
| "] Fragment shader contains OpDepthAttachmentReadEXT, and depthWriteEnable is not false.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index); |
| } |
| } |
| |
| if (module_state.static_data_.has_shader_tile_image_stencil_read) { |
| skip |= RequireFeature(module_state, enabled_features.shader_tile_image_features.shaderTileImageStencilReadAccess, |
| "shaderTileImageStencilReadAccess", "VUID-RuntimeSpirv-shaderTileImageStencilReadAccess-08730"); |
| |
| const auto *ds_state = pipeline.DepthStencilState(); |
| const bool is_write_mask_set = !pipeline.IsDynamic(VK_DYNAMIC_STATE_STENCIL_WRITE_MASK) && |
| (ds_state && (ds_state->front.writeMask != 0 || ds_state->back.writeMask != 0)); |
| if (mode_early_fragment_test && is_write_mask_set) { |
| skip |= LogError(device, "VUID-VkGraphicsPipelineCreateInfo-pStages-08712", |
| "%s(): pCreateInfos[%" PRIu32 |
| "] Fragment shader contains OpStencilAttachmentReadEXT, and stencil write mask is not equal to 0 for " |
| "both front(=%" PRIu32 ") and back (=%" PRIu32 ").", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, ds_state->front.writeMask, |
| ds_state->back.writeMask); |
| } |
| } |
| |
| if (module_state.static_data_.has_shader_tile_image_color_read) { |
| skip |= RequireFeature(module_state, enabled_features.shader_tile_image_features.shaderTileImageColorReadAccess, |
| "shaderTileImageColorReadAccess", "VUID-RuntimeSpirv-shaderTileImageColorReadAccess-08728"); |
| } |
| |
| bool using_tile_image_op = module_state.static_data_.has_shader_tile_image_depth_read || |
| module_state.static_data_.has_shader_tile_image_stencil_read || |
| module_state.static_data_.has_shader_tile_image_color_read; |
| const auto *ms_state = pipeline.MultisampleState(); |
| if (using_tile_image_op && ms_state && ms_state->sampleShadingEnable && (ms_state->minSampleShading != 1.0)) { |
| skip |= LogError(device, "VUID-RuntimeSpirv-minSampleShading-08732", |
| "%s(): pCreateInfos[%" PRIu32 "]: minSampleShading (=%f) is not equal to 1.0.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, ms_state->minSampleShading); |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidatePipelineShaderStage(const PIPELINE_STATE &pipeline, const PipelineStageState &stage_state) const { |
| bool skip = false; |
| const auto *create_info = stage_state.create_info; |
| const SHADER_MODULE_STATE &module_state = *stage_state.module_state.get(); |
| const VkShaderStageFlagBits stage = create_info->stage; |
| |
| if (pipeline.uses_shader_module_id || !module_state.has_valid_spirv) { |
| return skip; // these edge cases should be validated already |
| } |
| if (!stage_state.entrypoint) { |
| return LogError(device, "VUID-VkPipelineShaderStageCreateInfo-pName-00707", |
| "%s(): pCreateInfos[%" PRIu32 "] No entrypoint found named `%s` for stage %s.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, create_info->pName, |
| string_VkShaderStageFlagBits(stage)); |
| } |
| const EntryPoint &entrypoint = *stage_state.entrypoint; |
| |
| // to prevent const_cast on pipeline object, just store here as not needed outside function anyway |
| uint32_t local_size_x = 0; |
| uint32_t local_size_y = 0; |
| uint32_t local_size_z = 0; |
| uint32_t total_workgroup_shared_memory = 0; |
| |
| // If specialization-constant instructions are present in the shader, the specializations should be applied. |
| if (module_state.static_data_.has_specialization_constants) { |
| // both spirv-opt and spirv-val will use the same flags |
| spvtools::ValidatorOptions options; |
| AdjustValidatorOptions(device_extensions, enabled_features, options); |
| |
| // setup the call back if the optimizer fails |
| spv_target_env spirv_environment = PickSpirvEnv(api_version, IsExtEnabled(device_extensions.vk_khr_spirv_1_4)); |
| spvtools::Optimizer optimizer(spirv_environment); |
| spvtools::MessageConsumer consumer = [&skip, &module_state, &stage, &pipeline, this]( |
| spv_message_level_t level, const char *source, const spv_position_t &position, |
| const char *message) { |
| skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-module-parameter", |
| "%s(): pCreateInfos[%" PRIu32 "] %s does not contain valid spirv for stage %s. %s", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| string_VkShaderStageFlagBits(stage), message); |
| }; |
| optimizer.SetMessageConsumer(consumer); |
| |
| // The app might be using the default spec constant values, but if they pass values at runtime to the pipeline then need to |
| // use those values to apply to the spec constants |
| if (create_info->pSpecializationInfo != nullptr && create_info->pSpecializationInfo->mapEntryCount > 0 && |
| create_info->pSpecializationInfo->pMapEntries != nullptr) { |
| // Gather the specialization-constant values. |
| auto const &specialization_info = create_info->pSpecializationInfo; |
| auto const &specialization_data = reinterpret_cast<uint8_t const *>(specialization_info->pData); |
| std::unordered_map<uint32_t, std::vector<uint32_t>> id_value_map; // note: this must be std:: to work with spvtools |
| id_value_map.reserve(specialization_info->mapEntryCount); |
| for (auto i = 0u; i < specialization_info->mapEntryCount; ++i) { |
| auto const &map_entry = specialization_info->pMapEntries[i]; |
| const auto itr = module_state.static_data_.spec_const_map.find(map_entry.constantID); |
| // "If a constantID value is not a specialization constant ID used in the shader, that map entry does not affect the |
| // behavior of the pipeline." |
| if (itr != module_state.static_data_.spec_const_map.cend()) { |
| // Make sure map_entry.size matches the spec constant's size |
| uint32_t spec_const_size = DecorationSet::kInvalidValue; |
| const Instruction *def_insn = module_state.FindDef(itr->second); |
| const Instruction *type_insn = module_state.FindDef(def_insn->Word(1)); |
| // Specialization constants can only be of type bool, scalar integer, or scalar floating point |
| switch (type_insn->Opcode()) { |
| case spv::OpTypeBool: |
| // "If the specialization constant is of type boolean, size must be the byte size of VkBool32" |
| spec_const_size = sizeof(VkBool32); |
| break; |
| case spv::OpTypeInt: |
| case spv::OpTypeFloat: |
| spec_const_size = type_insn->Word(2) / 8; |
| break; |
| default: |
| // spirv-val should catch if SpecId is not used on a |
| // OpSpecConstantTrue/OpSpecConstantFalse/OpSpecConstant and OpSpecConstant is validated to be a |
| // OpTypeInt or OpTypeFloat |
| break; |
| } |
| |
| if (map_entry.size != spec_const_size) { |
| skip |= LogError( |
| device, "VUID-VkSpecializationMapEntry-constantID-00776", |
| "%s(): pCreateInfos[%" PRIu32 "] Specialization constant (ID = %" PRIu32 ", entry = %" PRIu32 |
| ") has invalid size %zu in shader module %s. Expected size is %" PRIu32 " from shader definition.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, map_entry.constantID, i, map_entry.size, |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), spec_const_size); |
| } |
| } |
| |
| if ((map_entry.offset + map_entry.size) <= specialization_info->dataSize) { |
| // Allocate enough room for ceil(map_entry.size / 4) to store entries |
| std::vector<uint32_t> entry_data((map_entry.size + 4 - 1) / 4, 0); |
| uint8_t *out_p = reinterpret_cast<uint8_t *>(entry_data.data()); |
| const uint8_t *const start_in_p = specialization_data + map_entry.offset; |
| const uint8_t *const end_in_p = start_in_p + map_entry.size; |
| |
| std::copy(start_in_p, end_in_p, out_p); |
| id_value_map.emplace(map_entry.constantID, std::move(entry_data)); |
| } |
| } |
| |
| // This pass takes the runtime spec const values and applies it into the SPIR-V |
| // will turn a spec constant like |
| // OpSpecConstant %uint 1 |
| // to a use the value passed in instead (for example if the value is 32) so now it looks like |
| // OpSpecConstant %uint 32 |
| optimizer.RegisterPass(spvtools::CreateSetSpecConstantDefaultValuePass(id_value_map)); |
| } |
| |
| // This pass will turn OpSpecConstant into a OpConstant (also OpSpecConstantTrue/OpSpecConstantFalse) |
| optimizer.RegisterPass(spvtools::CreateFreezeSpecConstantValuePass()); |
| // Using the new frozen OpConstant all OpSpecConstantComposite can be resolved turning them into OpConstantComposite |
| // This is need incase a shdaer looks like: |
| // |
| // layout(constant_id = 0) const uint x = 64; |
| // shared uint arr[x > 64 ? 64 : x]; |
| // |
| // this will generate branch/switch statements that we want to leverage spirv-opt to apply to make parsing easier |
| optimizer.RegisterPass(spvtools::CreateFoldSpecConstantOpAndCompositePass()); |
| |
| // Apply the specialization-constant values and revalidate the shader module is valid. |
| const char *pSpecializationInfo_vuid = IsExtEnabled(device_extensions.vk_ext_shader_module_identifier) |
| ? "VUID-VkPipelineShaderStageCreateInfo-pSpecializationInfo-06849" |
| : "VUID-VkPipelineShaderStageCreateInfo-pSpecializationInfo-06719"; |
| std::vector<uint32_t> specialized_spirv; |
| auto const optimized = |
| optimizer.Run(module_state.words_.data(), module_state.words_.size(), &specialized_spirv, options, true); |
| if (optimized) { |
| spv_context ctx = spvContextCreate(spirv_environment); |
| spv_const_binary_t binary{specialized_spirv.data(), specialized_spirv.size()}; |
| spv_diagnostic diag = nullptr; |
| auto const spv_valid = spvValidateWithOptions(ctx, options, &binary, &diag); |
| if (spv_valid != SPV_SUCCESS) { |
| skip |= LogError(device, pSpecializationInfo_vuid, |
| "%s(): pCreateInfos[%" PRIu32 |
| "] After specialization was applied, %s does not contain valid spirv for stage %s.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), |
| string_VkShaderStageFlagBits(stage)); |
| } |
| |
| // The new optimized SPIR-V will NOT match the original SHADER_MODULE_STATE object parsing, so a new SHADER_MODULE_STATE |
| // object is needed. This an issue due to each pipeline being able to reuse the same shader module but with different |
| // spec constant values. |
| SHADER_MODULE_STATE spec_mod(vvl::make_span<const uint32_t>(specialized_spirv.data(), specialized_spirv.size())); |
| |
| // According to https://github.com/KhronosGroup/Vulkan-Docs/issues/1671 anything labeled as "static use" (such as if an |
| // input is used or not) don't have to be checked post spec constants freezing since the device compiler is not |
| // guaranteed to run things such as dead-code elimination. The following checks are things that don't follow under |
| // "static use" rules and need to be validated still. |
| |
| const auto spec_entrypoint = spec_mod.FindEntrypoint(entrypoint.name.c_str(), entrypoint.stage); |
| assert(spec_entrypoint); // spirv-opt won't change Entrypoint Name/stage |
| |
| spec_mod.FindLocalSize(*spec_entrypoint, local_size_x, local_size_y, local_size_z); |
| |
| total_workgroup_shared_memory = spec_mod.CalculateWorkgroupSharedMemory(); |
| |
| spvDiagnosticDestroy(diag); |
| spvContextDestroy(ctx); |
| } else { |
| // Should never get here, but better then asserting |
| skip |= |
| LogError(device, pSpecializationInfo_vuid, |
| "%s(): pCreateInfos[%" PRIu32 |
| "] %s module (stage %s) attempted to apply specialization constants with spirv-opt but failed.", |
| pipeline.GetCreateFunctionName(), pipeline.create_index, |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), string_VkShaderStageFlagBits(stage)); |
| } |
| |
| if (skip) { |
| return skip; // if spec constants have errors, can produce false positives later |
| } |
| } |
| |
| // Validate descriptor set layout against what the entrypoint actually uses |
| |
| // The following tries to limit the number of passes through the shader module. The validation passes in here are "stateless" |
| // and mainly only checking the instruction in detail for a single operation |
| for (const Instruction &insn : module_state.GetInstructions()) { |
| skip |= ValidateTexelOffsetLimits(module_state, insn); |
| skip |= ValidateShaderCapabilitiesAndExtensions(insn); |
| skip |= ValidateMemoryScope(module_state, insn); |
| } |
| |
| skip |= ValidateTransformFeedback(module_state, entrypoint, pipeline); |
| skip |= ValidateShaderStageInputOutputLimits(module_state, stage, pipeline, entrypoint); |
| skip |= ValidateShaderStageMaxResources(module_state, stage, pipeline); |
| skip |= ValidateAtomicsTypes(module_state); |
| skip |= ValidateShaderStageGroupNonUniform(module_state, stage); |
| skip |= ValidateShaderClock(module_state); |
| skip |= ValidateShaderTileImage(module_state, entrypoint, pipeline, stage); |
| skip |= ValidateImageWrite(module_state); |
| skip |= ValidateExecutionModes(module_state, entrypoint, stage, pipeline); |
| skip |= ValidateSpecializations(module_state, create_info->pSpecializationInfo, pipeline); |
| skip |= ValidateVariables(module_state); |
| skip |= ValidatePointSizeShaderState(pipeline, module_state, entrypoint, stage); |
| skip |= ValidateBuiltinLimits(module_state, entrypoint, pipeline); |
| if (enabled_features.cooperative_matrix_features.cooperativeMatrix) { |
| skip |= ValidateCooperativeMatrix(module_state, create_info); |
| } |
| if (enabled_features.fragment_shading_rate_features.primitiveFragmentShadingRate) { |
| skip |= ValidatePrimitiveRateShaderState(pipeline, module_state, entrypoint, stage); |
| } |
| if (IsExtEnabled(device_extensions.vk_qcom_render_pass_shader_resolve)) { |
| skip |= ValidateShaderResolveQCOM(module_state, stage, pipeline); |
| } |
| if (IsExtEnabled(device_extensions.vk_ext_subgroup_size_control)) { |
| skip |= ValidateShaderSubgroupSizeControl(module_state, create_info->flags); |
| } |
| if (IsExtEnabled(device_extensions.vk_khr_dynamic_rendering) && IsExtEnabled(device_extensions.vk_khr_multiview)) { |
| if (stage == VK_SHADER_STAGE_FRAGMENT_BIT && |
| pipeline.GetCreateInfo<VkGraphicsPipelineCreateInfo>().renderPass == VK_NULL_HANDLE && |
| module_state.HasCapability(spv::CapabilityInputAttachment)) { |
| skip |= LogError(device, "VUID-VkGraphicsPipelineCreateInfo-renderPass-06061", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] is being created with fragment shader state and renderPass = VK_NULL_HANDLE, but fragment " |
| "shader includes InputAttachment capability.", |
| pipeline.create_index); |
| } |
| } |
| |
| if (const auto *pipeline_robustness_info = LvlFindInChain<VkPipelineRobustnessCreateInfoEXT>(create_info->pNext); |
| pipeline_robustness_info) { |
| std::stringstream parameter_name; |
| parameter_name << "pCreateInfos[" << pipeline.create_index << "]"; |
| skip |= ValidatePipelineRobustnessCreateInfo(pipeline, parameter_name.str().c_str(), *pipeline_robustness_info); |
| } |
| |
| // Validate Push Constants use |
| skip |= ValidatePushConstantUsage(pipeline, module_state, entrypoint); |
| // can dereference because entrypoint is validated by here |
| skip |= ValidateShaderDescriptorVariable(module_state, stage, pipeline, entrypoint); |
| |
| if (stage == VK_SHADER_STAGE_FRAGMENT_BIT) { |
| skip |= ValidateConservativeRasterization(module_state, entrypoint, pipeline); |
| } else if (stage == VK_SHADER_STAGE_COMPUTE_BIT) { |
| skip |= ValidateComputeWorkGroupSizes(module_state, entrypoint, stage_state, local_size_x, local_size_y, local_size_z); |
| skip |= ValidateWorkgroupSharedMemory(module_state, stage, total_workgroup_shared_memory); |
| } else if (stage == VK_SHADER_STAGE_TASK_BIT_EXT || stage == VK_SHADER_STAGE_MESH_BIT_EXT) { |
| skip |= ValidateWorkgroupSharedMemory(module_state, stage, total_workgroup_shared_memory); |
| skip |= ValidateTaskMeshWorkGroupSizes(module_state, entrypoint, stage_state, local_size_x, local_size_y, local_size_z); |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateInterfaceBetweenStages(const SHADER_MODULE_STATE &producer, const EntryPoint &producer_entrypoint, |
| const SHADER_MODULE_STATE &consumer, const EntryPoint &consumer_entrypoint, |
| uint32_t pipe_index) const { |
| bool skip = false; |
| |
| if (producer_entrypoint.has_passthrough) { |
| return skip; // PassthroughNV doesn't have to do Location matching |
| } |
| |
| const VkShaderStageFlagBits producer_stage = producer_entrypoint.stage; |
| const VkShaderStageFlagBits consumer_stage = consumer_entrypoint.stage; |
| |
| // build up a mapping of which slots are used and then go through it and look for gaps |
| struct ComponentInfo { |
| const StageInteraceVariable *output = nullptr; |
| uint32_t output_type = 0; |
| uint32_t output_width = 0; |
| const StageInteraceVariable *input = nullptr; |
| uint32_t input_type = 0; |
| uint32_t input_width = 0; |
| }; |
| // <Location, Components[4]> (only 4 componets in a Location) |
| vvl::unordered_map<uint32_t, std::array<ComponentInfo, 4>> slot_map; |
| |
| for (const auto &interface_slot : producer_entrypoint.output_interface_slots) { |
| auto &slot = slot_map[interface_slot.first.Location()][interface_slot.first.Component()]; |
| if (interface_slot.second->nested_struct) { |
| return skip; // TODO workaround |
| } |
| slot.output = interface_slot.second; |
| slot.output_type = interface_slot.first.type; |
| slot.output_width = interface_slot.first.bit_width; |
| } |
| for (const auto &interface_slot : consumer_entrypoint.input_interface_slots) { |
| auto &slot = slot_map[interface_slot.first.Location()][interface_slot.first.Component()]; |
| if (interface_slot.second->nested_struct) { |
| return skip; // TODO workaround |
| } |
| slot.input = interface_slot.second; |
| slot.input_type = interface_slot.first.type; |
| slot.input_width = interface_slot.first.bit_width; |
| } |
| |
| for (const auto &slot : slot_map) { |
| // Found that sometimes there is a big mismatch and printing out EVERY slot adds a lot of noise |
| if (skip) break; |
| |
| const uint32_t location = slot.first; |
| for (uint32_t component = 0; component < 4; component++) { |
| const auto &component_info = slot.second[component]; |
| const auto *input_var = component_info.input; |
| const auto *output_var = component_info.output; |
| |
| if ((input_var == nullptr) && (output_var == nullptr)) { |
| continue; // both empty |
| } else if ((input_var != nullptr) && (output_var != nullptr)) { |
| // if matched, need to check type |
| // Only the OpType has to match, signed vs unsigned in not important |
| if ((component_info.output_type != component_info.input_type) || |
| (component_info.output_width != component_info.input_width)) { |
| const LogObjectList objlist(producer.vk_shader_module(), consumer.vk_shader_module()); |
| skip |= |
| LogError(objlist, "VUID-RuntimeSpirv-OpEntryPoint-07754", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Type mismatch on Location %" PRIu32 |
| " Component %" PRIu32 ", between\n%s stage:\n%s\n%s stage:\n%s", |
| pipe_index, location, component, string_VkShaderStageFlagBits(producer_stage), |
| producer.DescribeType(output_var->type_id).c_str(), string_VkShaderStageFlagBits(consumer_stage), |
| consumer.DescribeType(input_var->type_id).c_str()); |
| } |
| |
| // Tessellation needs to match Patch vs Vertex |
| if ((producer_stage == VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT) && |
| (consumer_stage == VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT) && |
| (input_var->is_patch != output_var->is_patch)) { |
| const LogObjectList objlist(producer.vk_shader_module(), consumer.vk_shader_module()); |
| skip |= LogError(objlist, "VUID-RuntimeSpirv-OpVariable-08746", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] at Location %" PRIu32 |
| " Component %" PRIu32 " Tessellation Control is %s while Tessellation Evaluation is %s", |
| pipe_index, location, component, input_var->is_patch ? "patch" : "vertex", |
| output_var->is_patch ? "patch" : "vertex"); |
| } |
| |
| // If using maintenance4 need to check Vectors incase different sizes |
| if (!enabled_features.core13.maintenance4 && (output_var->base_type.Opcode() == spv::OpTypeVector) && |
| (input_var->base_type.Opcode() == spv::OpTypeVector)) { |
| // Note the "Component Count" in the VU refers to OpTypeVector's operand and NOT the "Component slot" |
| const uint32_t output_vec_size = output_var->base_type.Word(3); |
| const uint32_t input_vec_size = input_var->base_type.Word(3); |
| if (output_vec_size > input_vec_size) { |
| const LogObjectList objlist(producer.vk_shader_module(), consumer.vk_shader_module()); |
| const char *vuid = IsExtEnabled(device_extensions.vk_khr_maintenance4) |
| ? "VUID-RuntimeSpirv-maintenance4-06817" |
| : "VUID-RuntimeSpirv-OpTypeVector-06816"; |
| skip |= |
| LogError(objlist, vuid, |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] starting at Location %" PRIu32 |
| " Component %" PRIu32 " the Output (%s) has a Vec%" PRIu32 " while Input (%s) as a Vec%" PRIu32 |
| ". Enable VK_KHR_maintenance4 device extension to allow relaxed interface matching " |
| "between input and output vectors.", |
| pipe_index, location, component, string_VkShaderStageFlagBits(producer_stage), output_vec_size, |
| string_VkShaderStageFlagBits(consumer_stage), input_vec_size); |
| break; // Only need to report for the first component found |
| } |
| } |
| } else if ((input_var == nullptr) && (output_var != nullptr)) { |
| // Missing input slot |
| // It is not an error if a stage does not consume all outputs from the previous stage |
| // The values will be undefined, but still legal |
| // Don't give any warning if maintenance4 with vectors |
| if (!enabled_features.core13.maintenance4 && (output_var->base_type.Opcode() != spv::OpTypeVector)) { |
| const LogObjectList objlist(producer.vk_shader_module(), consumer.vk_shader_module()); |
| skip |= LogPerformanceWarning(objlist, kVUID_Core_Shader_OutputNotConsumed, |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 |
| "] %s declared to output location %" PRIu32 " Component %" PRIu32 |
| " but is not an Input declared by %s.", |
| pipe_index, string_VkShaderStageFlagBits(producer_stage), location, component, |
| string_VkShaderStageFlagBits(consumer_stage)); |
| } |
| } else if ((input_var != nullptr) && (output_var == nullptr)) { |
| // Missing output slot |
| if ((consumer_stage & (VK_SHADER_STAGE_GEOMETRY_BIT | VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT | |
| VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT)) && |
| (input_var->base_type.Opcode() == spv::OpTypeArray)) { |
| break; // When going inbetween Tessellation or Geometry, array size can be different |
| } |
| const LogObjectList objlist(producer.vk_shader_module(), consumer.vk_shader_module()); |
| skip |= LogError(objlist, "VUID-RuntimeSpirv-OpEntryPoint-08743", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] %s declared input at Location %" PRIu32 |
| " Component %" PRIu32 " but it is not an Output declared in %s", |
| pipe_index, string_VkShaderStageFlagBits(consumer_stage), location, component, |
| string_VkShaderStageFlagBits(producer_stage)); |
| break; // Only need to report for the first component found |
| } |
| } |
| } |
| |
| // Need to check the BuiltIn interface (if not going into Fragment) |
| if (consumer_stage == VK_SHADER_STAGE_FRAGMENT_BIT) { |
| return skip; |
| } |
| |
| std::vector<uint32_t> input_builtins_block; |
| std::vector<uint32_t> output_builtins_block; |
| for (const auto *variable : producer_entrypoint.built_in_variables) { |
| if (variable->storage_class == spv::StorageClassOutput && !variable->builtin_block.empty()) { |
| output_builtins_block = variable->builtin_block; |
| break; |
| } |
| } |
| for (const auto *variable : consumer_entrypoint.built_in_variables) { |
| if (variable->storage_class == spv::StorageClassInput && !variable->builtin_block.empty()) { |
| input_builtins_block = variable->builtin_block; |
| break; |
| } |
| } |
| |
| bool mismatch = false; |
| if (input_builtins_block.empty() || output_builtins_block.empty()) { |
| // TODO - Nothing about this in spec, need to add language to confirm this is correct |
| return skip; |
| } else if (input_builtins_block.size() != output_builtins_block.size()) { |
| mismatch = true; |
| } else { |
| for (size_t i = 0; i < input_builtins_block.size(); i++) { |
| const uint32_t input_builtin = input_builtins_block[i]; |
| const uint32_t output_builtin = output_builtins_block[i]; |
| if (input_builtin == DecorationSet::kInvalidValue || output_builtin == DecorationSet::kInvalidValue) { |
| continue; // some stages (TessControl -> TessEval) can have legal block vs non-block mistmatch |
| } else if (input_builtin != output_builtin) { |
| mismatch = true; |
| } |
| } |
| } |
| |
| if (mismatch) { |
| std::stringstream msg; |
| msg << string_VkShaderStageFlagBits(producer_stage) << " Output Block {\n"; |
| for (size_t i = 0; i < output_builtins_block.size(); i++) { |
| msg << "\t" << i << ": " << string_SpvDecoration(output_builtins_block[i]) << "\n"; |
| } |
| msg << "}\n"; |
| msg << string_VkShaderStageFlagBits(consumer_stage) << " Input Block {\n"; |
| for (size_t i = 0; i < input_builtins_block.size(); i++) { |
| msg << "\t" << i << ": " << string_SpvDecoration(input_builtins_block[i]) << "\n"; |
| } |
| msg << "}\n"; |
| const LogObjectList objlist(producer.vk_shader_module(), consumer.vk_shader_module()); |
| skip |= LogError(objlist, "VUID-RuntimeSpirv-OpVariable-08746", |
| "vkCreateGraphicsPipelines(): pCreateInfos[%" PRIu32 "] Mistmatch in BuiltIn blocks:\n %s", pipe_index, |
| msg.str().c_str()); |
| } |
| return skip; |
| } |
| |
| // Validate that the shaders used by the given pipeline and store the active_slots |
| // that are actually used by the pipeline into pPipeline->active_slots |
| bool CoreChecks::ValidateGraphicsPipelineShaderState(const PIPELINE_STATE &pipeline) const { |
| bool skip = false; |
| |
| if (!(pipeline.pre_raster_state || pipeline.fragment_shader_state)) { |
| // Only validate pipelines that contain shader stages |
| return skip; |
| } |
| |
| const PipelineStageState *vertex_stage = nullptr, *fragment_stage = nullptr; |
| for (auto &stage_state : pipeline.stage_states) { |
| const VkShaderStageFlagBits stage = stage_state.create_info->stage; |
| // Only validate the shader state once when added, not again when linked |
| if ((stage & pipeline.linking_shaders) == 0) { |
| skip |= ValidatePipelineShaderStage(pipeline, stage_state); |
| } |
| if (stage == VK_SHADER_STAGE_VERTEX_BIT) { |
| vertex_stage = &stage_state; |
| } |
| if (stage == VK_SHADER_STAGE_FRAGMENT_BIT) { |
| fragment_stage = &stage_state; |
| } |
| } |
| |
| // if the shader stages are no good individually, cross-stage validation is pointless. |
| if (skip) return true; |
| |
| if (pipeline.vertex_input_state && vertex_stage && vertex_stage->entrypoint && vertex_stage->module_state->has_valid_spirv && |
| !pipeline.IsDynamic(VK_DYNAMIC_STATE_VERTEX_INPUT_EXT)) { |
| skip |= ValidateViAgainstVsInputs(pipeline, *vertex_stage->module_state.get(), *vertex_stage->entrypoint); |
| } |
| |
| for (size_t i = 1; i < pipeline.stage_states.size(); i++) { |
| const auto &producer = pipeline.stage_states[i - 1]; |
| const auto &consumer = pipeline.stage_states[i]; |
| assert(producer.module_state); |
| if (&producer == fragment_stage) { |
| break; |
| } |
| if (consumer.module_state) { |
| if (consumer.module_state->has_valid_spirv && producer.module_state->has_valid_spirv && consumer.entrypoint && |
| producer.entrypoint) { |
| skip |= ValidateInterfaceBetweenStages(*producer.module_state.get(), *producer.entrypoint, |
| *consumer.module_state.get(), *consumer.entrypoint, pipeline.create_index); |
| } |
| } |
| } |
| return skip; |
| } |
| |
| uint32_t CoreChecks::CalcShaderStageCount(const PIPELINE_STATE &pipeline, VkShaderStageFlagBits stageBit) const { |
| uint32_t total = 0; |
| for (const auto &stage_ci : pipeline.shader_stages_ci) { |
| if (stage_ci.stage == stageBit) { |
| total++; |
| } |
| } |
| |
| if (pipeline.ray_tracing_library_ci) { |
| for (uint32_t i = 0; i < pipeline.ray_tracing_library_ci->libraryCount; ++i) { |
| auto library_pipeline = Get<PIPELINE_STATE>(pipeline.ray_tracing_library_ci->pLibraries[i]); |
| total += CalcShaderStageCount(*library_pipeline, stageBit); |
| } |
| } |
| |
| return total; |
| } |
| |
| bool CoreChecks::GroupHasValidIndex(const PIPELINE_STATE &pipeline, uint32_t group, uint32_t stage) const { |
| if (group == VK_SHADER_UNUSED_NV) { |
| return true; |
| } |
| |
| const auto num_stages = static_cast<uint32_t>(pipeline.shader_stages_ci.size()); |
| if (group < num_stages) { |
| return (pipeline.shader_stages_ci[group].stage & stage) != 0; |
| } |
| group -= num_stages; |
| |
| // Search libraries |
| if (pipeline.ray_tracing_library_ci) { |
| for (uint32_t i = 0; i < pipeline.ray_tracing_library_ci->libraryCount; ++i) { |
| auto library_pipeline = Get<PIPELINE_STATE>(pipeline.ray_tracing_library_ci->pLibraries[i]); |
| const uint32_t stage_count = static_cast<uint32_t>(library_pipeline->shader_stages_ci.size()); |
| if (group < stage_count) { |
| return (library_pipeline->shader_stages_ci[group].stage & stage) != 0; |
| } |
| group -= stage_count; |
| } |
| } |
| |
| // group index too large |
| return false; |
| } |
| |
| uint32_t ValidationCache::MakeShaderHash(VkShaderModuleCreateInfo const *smci) { return XXH32(smci->pCode, smci->codeSize, 0); } |
| |
| static ValidationCache *GetValidationCacheInfo(VkShaderModuleCreateInfo const *pCreateInfo) { |
| const auto validation_cache_ci = LvlFindInChain<VkShaderModuleValidationCacheCreateInfoEXT>(pCreateInfo->pNext); |
| if (validation_cache_ci) { |
| return CastFromHandle<ValidationCache *>(validation_cache_ci->validationCache); |
| } |
| return nullptr; |
| } |
| |
| bool CoreChecks::PreCallValidateCreateShaderModule(VkDevice device, const VkShaderModuleCreateInfo *pCreateInfo, |
| const VkAllocationCallbacks *pAllocator, VkShaderModule *pShaderModule) const { |
| bool skip = false; |
| spv_result_t spv_valid = SPV_SUCCESS; |
| |
| if (disabled[shader_validation]) { |
| return false; |
| } |
| |
| auto have_glsl_shader = IsExtEnabled(device_extensions.vk_nv_glsl_shader); |
| |
| if (!have_glsl_shader && (pCreateInfo->codeSize % 4)) { |
| skip |= LogError(device, "VUID-VkShaderModuleCreateInfo-codeSize-08735", |
| "SPIR-V module not valid: Codesize must be a multiple of 4 but is %zu", pCreateInfo->codeSize); |
| } else { |
| auto cache = GetValidationCacheInfo(pCreateInfo); |
| uint32_t hash = 0; |
| // If app isn't using a shader validation cache, use the default one from CoreChecks |
| if (!cache) cache = CastFromHandle<ValidationCache *>(core_validation_cache); |
| if (cache) { |
| hash = ValidationCache::MakeShaderHash(pCreateInfo); |
| if (cache->Contains(hash)) return false; |
| } |
| |
| // Use SPIRV-Tools validator to try and catch any issues with the module itself. If specialization constants are present, |
| // the default values will be used during validation. |
| spv_target_env spirv_environment = PickSpirvEnv(api_version, IsExtEnabled(device_extensions.vk_khr_spirv_1_4)); |
| spv_context ctx = spvContextCreate(spirv_environment); |
| spv_const_binary_t binary{pCreateInfo->pCode, pCreateInfo->codeSize / sizeof(uint32_t)}; |
| spv_diagnostic diag = nullptr; |
| spvtools::ValidatorOptions options; |
| AdjustValidatorOptions(device_extensions, enabled_features, options); |
| spv_valid = spvValidateWithOptions(ctx, options, &binary, &diag); |
| if (spv_valid != SPV_SUCCESS) { |
| if (!have_glsl_shader || (pCreateInfo->pCode[0] == spv::MagicNumber)) { |
| if (spv_valid == SPV_WARNING) { |
| skip |= LogWarning(device, "VUID-VkShaderModuleCreateInfo-pCode-01379", "SPIR-V module not valid: %s", |
| diag && diag->error ? diag->error : "(no error text)"); |
| } else { |
| skip |= LogError(device, "VUID-VkShaderModuleCreateInfo-pCode-01379", "SPIR-V module not valid: %s", |
| diag && diag->error ? diag->error : "(no error text)"); |
| } |
| } |
| } else { |
| if (cache) { |
| cache->Insert(hash); |
| } |
| } |
| |
| spvDiagnosticDestroy(diag); |
| spvContextDestroy(ctx); |
| } |
| |
| return skip; |
| } |
| |
| bool CoreChecks::PreCallValidateGetShaderModuleIdentifierEXT(VkDevice device, VkShaderModule shaderModule, |
| VkShaderModuleIdentifierEXT *pIdentifier) const { |
| bool skip = false; |
| if (!(enabled_features.shader_module_identifier_features.shaderModuleIdentifier)) { |
| skip |= LogError(shaderModule, "VUID-vkGetShaderModuleIdentifierEXT-shaderModuleIdentifier-06884", |
| "vkGetShaderModuleIdentifierEXT() was called when the shaderModuleIdentifier feature was not enabled"); |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::PreCallValidateGetShaderModuleCreateInfoIdentifierEXT(VkDevice device, const VkShaderModuleCreateInfo *pCreateInfo, |
| VkShaderModuleIdentifierEXT *pIdentifier) const { |
| bool skip = false; |
| if (!(enabled_features.shader_module_identifier_features.shaderModuleIdentifier)) { |
| skip |= LogError( |
| device, "VUID-vkGetShaderModuleCreateInfoIdentifierEXT-shaderModuleIdentifier-06885", |
| "vkGetShaderModuleCreateInfoIdentifierEXT() was called when the shaderModuleIdentifier feature was not enabled"); |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateComputeWorkGroupSizes(const SHADER_MODULE_STATE &module_state, const EntryPoint &entrypoint, |
| const PipelineStageState &stage_state, uint32_t local_size_x, uint32_t local_size_y, |
| uint32_t local_size_z) const { |
| bool skip = false; |
| // If spec constants were used then the local size are already found if possible |
| if (local_size_x == 0) { |
| if (!module_state.FindLocalSize(entrypoint, local_size_x, local_size_y, local_size_z)) { |
| return skip; // no local size found |
| } |
| } |
| |
| if (local_size_x > phys_dev_props.limits.maxComputeWorkGroupSize[0]) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-x-06429", |
| "%s local_size_x (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[0] (%" PRIu32 ").", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), local_size_x, |
| phys_dev_props.limits.maxComputeWorkGroupSize[0]); |
| } |
| if (local_size_y > phys_dev_props.limits.maxComputeWorkGroupSize[1]) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-y-06430", |
| "%s local_size_y (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[1] (%" PRIu32 ").", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), local_size_x, |
| phys_dev_props.limits.maxComputeWorkGroupSize[1]); |
| } |
| if (local_size_z > phys_dev_props.limits.maxComputeWorkGroupSize[2]) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-z-06431", |
| "%s local_size_z (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[2] (%" PRIu32 ").", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), local_size_x, |
| phys_dev_props.limits.maxComputeWorkGroupSize[2]); |
| } |
| |
| uint32_t limit = phys_dev_props.limits.maxComputeWorkGroupInvocations; |
| uint64_t invocations = local_size_x * local_size_y; |
| // Prevent overflow. |
| bool fail = false; |
| if (invocations > vvl::kU32Max || invocations > limit) { |
| fail = true; |
| } |
| if (!fail) { |
| invocations *= local_size_z; |
| if (invocations > vvl::kU32Max || invocations > limit) { |
| fail = true; |
| } |
| } |
| if (fail) { |
| skip |= LogError(module_state.vk_shader_module(), "VUID-RuntimeSpirv-x-06432", |
| "%s local_size (%" PRIu32 ", %" PRIu32 ", %" PRIu32 |
| ") exceeds device limit maxComputeWorkGroupInvocations (%" PRIu32 ").", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), local_size_x, local_size_y, |
| local_size_z, limit); |
| } |
| |
| const auto subgroup_flags = VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT | |
| VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT; |
| const auto *required_subgroup_size_features = |
| LvlFindInChain<VkPipelineShaderStageRequiredSubgroupSizeCreateInfoEXT>(stage_state.create_info->pNext); |
| if (required_subgroup_size_features) { |
| const uint32_t requiredSubgroupSize = required_subgroup_size_features->requiredSubgroupSize; |
| skip |= RequireFeature(module_state, enabled_features.core13.subgroupSizeControl, "subgroupSizeControl", |
| "VUID-VkPipelineShaderStageCreateInfo-pNext-02755"); |
| if ((phys_dev_ext_props.subgroup_size_control_props.requiredSubgroupSizeStages & stage_state.create_info->stage) == 0) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-pNext-02755", |
| "Stage %s is not in VkPhysicalDeviceSubgroupSizeControlPropertiesEXT::requiredSubgroupSizeStages (%s).", |
| string_VkShaderStageFlagBits(stage_state.create_info->stage), |
| string_VkShaderStageFlags(phys_dev_ext_props.subgroup_size_control_props.requiredSubgroupSizeStages).c_str()); |
| } |
| if ((invocations > requiredSubgroupSize * phys_dev_ext_props.subgroup_size_control_props.maxComputeWorkgroupSubgroups)) { |
| skip |= |
| LogError(module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-pNext-02756", |
| "Local workgroup size (%" PRIu32 ", %" PRIu32 ", %" PRIu32 |
| ") is greater than VkPipelineShaderStageRequiredSubgroupSizeCreateInfoEXT::requiredSubgroupSize (%" PRIu32 |
| ") * maxComputeWorkgroupSubgroups (%" PRIu32 ").", |
| local_size_x, local_size_y, local_size_z, requiredSubgroupSize, |
| phys_dev_ext_props.subgroup_size_control_props.maxComputeWorkgroupSubgroups); |
| } |
| if ((stage_state.create_info->flags & VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT) > 0) { |
| if (SafeModulo(local_size_x, requiredSubgroupSize) != 0) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-pNext-02757", |
| "Local workgroup size x (%" PRIu32 |
| ") is not a multiple of VkPipelineShaderStageRequiredSubgroupSizeCreateInfoEXT::requiredSubgroupSize (%" PRIu32 |
| ").", |
| local_size_x, requiredSubgroupSize); |
| } |
| } |
| if (!IsPowerOfTwo(requiredSubgroupSize)) { |
| skip |= LogError(module_state.vk_shader_module(), |
| "VUID-VkPipelineShaderStageRequiredSubgroupSizeCreateInfo-requiredSubgroupSize-02760", |
| "VkPhysicalDeviceSubgroupSizeControlPropertiesEXT::requiredSubgroupSizeStages (%" PRIu32 |
| ") is not a power of 2.", |
| requiredSubgroupSize); |
| } |
| if (requiredSubgroupSize < phys_dev_ext_props.subgroup_size_control_props.minSubgroupSize) { |
| skip |= LogError(module_state.vk_shader_module(), |
| "VUID-VkPipelineShaderStageRequiredSubgroupSizeCreateInfo-requiredSubgroupSize-02761", |
| "VkPhysicalDeviceSubgroupSizeControlPropertiesEXT::requiredSubgroupSizeStages (%" PRIu32 |
| ") is less than minSubgroupSize (%" PRIu32 ").", |
| requiredSubgroupSize, phys_dev_ext_props.subgroup_size_control_props.minSubgroupSize); |
| } |
| if (requiredSubgroupSize > phys_dev_ext_props.subgroup_size_control_props.maxSubgroupSize) { |
| skip |= LogError(module_state.vk_shader_module(), |
| "VUID-VkPipelineShaderStageRequiredSubgroupSizeCreateInfo-requiredSubgroupSize-02762", |
| "VkPhysicalDeviceSubgroupSizeControlPropertiesEXT::requiredSubgroupSizeStages (%" PRIu32 |
| ") is greater than maxSubgroupSize (%" PRIu32 ").", |
| requiredSubgroupSize, phys_dev_ext_props.subgroup_size_control_props.maxSubgroupSize); |
| } |
| } |
| if ((stage_state.create_info->flags & subgroup_flags) == subgroup_flags) { |
| if (SafeModulo(local_size_x, phys_dev_ext_props.subgroup_size_control_props.maxSubgroupSize) != 0) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-flags-02758", |
| "%s flags contain VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT and " |
| "VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT bits, but local workgroup size in the X " |
| "dimension (%" PRIu32 |
| ") is not a multiple of VkPhysicalDeviceSubgroupSizeControlPropertiesEXT::maxSubgroupSize (%" PRIu32 ").", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), local_size_x, |
| phys_dev_ext_props.subgroup_size_control_props.maxSubgroupSize); |
| } |
| } else if ((stage_state.create_info->flags & VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT) && |
| (stage_state.create_info->flags & VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT) == 0) { |
| if (!required_subgroup_size_features) { |
| if (SafeModulo(local_size_x, phys_dev_props_core11.subgroupSize) != 0) { |
| skip |= LogError( |
| module_state.vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-flags-02759", |
| "%s flags contain VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT bit, and not the" |
| "VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT bit, but local workgroup size in the " |
| "X dimension (%" PRIu32 ") is not a multiple of VkPhysicalDeviceVulkan11Properties::subgroupSize (%" PRIu32 |
| ").", |
| report_data->FormatHandle(module_state.vk_shader_module()).c_str(), local_size_x, |
| phys_dev_props_core11.subgroupSize); |
| } |
| } |
| } |
| return skip; |
| } |
| |
| bool CoreChecks::ValidateTaskMeshWorkGroupSizes(const SHADER_MODULE_STATE &module_state, const EntryPoint &entrypoint, |
| const PipelineStageState &stage_state, uint32_t local_size_x, uint32_t local_size_y, |
| uint32_t local_size_z) const { |
| bool skip = false; |
| // If spec constants were used then the local size are already found if possible |
| if (local_size_x == 0) { |
| if (!module_state.FindLocalSize(entrypoint, local_size_x, local_size_y, local_size_z)) { |
| return skip; // no local size found |
| } |
| } |
| |
| uint32_t max_local_size_x = 0; |
| uint32_t max_local_size_y = 0; |
| uint32_t max_local_size_z = 0; |
| uint32_t max_workgroup_size = 0; |
| const char *x_vuid; |
| const char *y_vuid; |
| const char *z_vuid; |
| const char *workgroup_size_vuid; |
| |
| switch (entrypoint.execution_model) { |
| case spv::ExecutionModelTaskEXT: { |
| x_vuid = "VUID-RuntimeSpirv-TaskEXT-07291"; |
| y_vuid = "VUID-RuntimeSpirv-TaskEXT-07292"; |
| z_vuid = "VUID-RuntimeSpirv-TaskEXT-07293"; |
| workgroup_size_vuid = "VUID-RuntimeSpirv-TaskEXT-07294"; |
| max_local_size_x = phys_dev_ext_props.mesh_shader_props_ext.maxTaskWorkGroupSize[0]; |
| max_local_size_y = phys_dev_ext_props.mesh_shader_props_ext.maxTaskWorkGroupSize[1]; |
| max_local_size_z = phys_dev_ext_props.mesh_shader_props_ext.maxTaskWorkGroupSize[2]; |
| max_workgroup_size = phys_dev_ext_props.mesh_shader_props_ext.maxTaskWorkGroupInvocations; |
| break; |
| } |
| |
| case spv::ExecutionModelMeshEXT: { |
| x_vuid = "VUID-RuntimeSpirv-MeshEXT-07295"; |
| y_vuid = "VUID-RuntimeSpirv-MeshEXT-07296"; |
| z_vuid = "VUID-RuntimeSpirv-MeshEXT-07297"; |
| workgroup_size_vuid = "VUID-RuntimeSpirv-MeshEXT-07298"; |
| max_local_size_x = phys_dev_ext_props.mesh_shader_props_ext.maxMeshWorkGroupSize[0]; |
| max_local_size_y = phys_dev_ext_props.mesh_shader_props_ext.maxMeshWorkGroupSize[1]; |
| max_local_size_z = phys_dev_ext_props.mesh_shader_props_ext.maxMeshWorkGroupSize[2]; |
| max_workgroup_size = phys_dev_ext_props.mesh_shader_props_ext.maxMeshWorkGroupInvocations; |
| break; |
| } |
| |
| // skip for spv::ExecutionModelTaskNV and spv::ExecutionModelMeshNV case |
| default: { |
| // must match one of the above case |
| return skip; |
| } |
| } |
| |
| if (local_size_x > max_local_size_x) { |
| skip |= LogError(module_state.vk_shader_module(), x_vuid, |
| "%s shader local workgroup size in X dimension (%" PRIu32 |
| ") must be less than or equal to the max workgroup size (%" PRIu32 ").", |
| string_SpvExecutionModel(entrypoint.execution_model), local_size_x, max_local_size_x); |
| } |
| |
| if (local_size_y > max_local_size_y) { |
| skip |= LogError(module_state.vk_shader_module(), y_vuid, |
| "%s shader local workgroup size in Y dimension (%" PRIu32 |
| ") must be less than or equal to the max workgroup size (%" PRIu32 ").", |
| string_SpvExecutionModel(entrypoint.execution_model), local_size_y, max_local_size_y); |
| } |
| |
| if (local_size_z > max_local_size_z) { |
| skip |= LogError(module_state.vk_shader_module(), z_vuid, |
| "%s shader local workgroup size in Z dimension (%" PRIu32 |
| ") must be less than or equal to the max workgroup size (%" PRIu32 ").", |
| string_SpvExecutionModel(entrypoint.execution_model), local_size_z, max_local_size_z); |
| } |
| |
| uint64_t invocations = local_size_x * local_size_y; |
| // Prevent overflow. |
| bool fail = false; |
| if (invocations > vvl::kU32Max || invocations > max_workgroup_size) { |
| fail = true; |
| } |
| if (!fail) { |
| invocations *= local_size_z; |
| if (invocations > vvl::kU32Max || invocations > max_workgroup_size) { |
| fail = true; |
| } |
| } |
| if (fail) { |
| skip |= LogError(module_state.vk_shader_module(), workgroup_size_vuid, |
| "%s shader total invocation size (%" PRIu32 "* %" PRIu32 "* %" PRIu32 " = %" PRIu32 |
| ") must be less than or equal to max workgroup invocations (%" PRIu32 ").", |
| string_SpvExecutionModel(entrypoint.execution_model), local_size_x, local_size_y, local_size_z, |
| local_size_x * local_size_y * local_size_z, max_workgroup_size); |
| } |
| return skip; |
| } |
| |
| spv_target_env PickSpirvEnv(APIVersion api_version, bool spirv_1_4) { |
| if (api_version >= VK_API_VERSION_1_3) { |
| return SPV_ENV_VULKAN_1_3; |
| } else if (api_version >= VK_API_VERSION_1_2) { |
| return SPV_ENV_VULKAN_1_2; |
| } else if (api_version >= VK_API_VERSION_1_1) { |
| if (spirv_1_4) { |
| return SPV_ENV_VULKAN_1_1_SPIRV_1_4; |
| } else { |
| return SPV_ENV_VULKAN_1_1; |
| } |
| } |
| return SPV_ENV_VULKAN_1_0; |
| } |
| |
| // Some Vulkan extensions/features are just all done in spirv-val behind optional settings |
| void AdjustValidatorOptions(const DeviceExtensions &device_extensions, const DeviceFeatures &enabled_features, |
| spvtools::ValidatorOptions &options) { |
| // VK_KHR_relaxed_block_layout never had a feature bit so just enabling the extension allows relaxed layout |
| // Was promotoed in Vulkan 1.1 so anyone using Vulkan 1.1 also gets this for free |
| if (IsExtEnabled(device_extensions.vk_khr_relaxed_block_layout)) { |
| // --relax-block-layout |
| options.SetRelaxBlockLayout(true); |
| } |
| |
| // The rest of the settings are controlled from a feature bit, which are set correctly in the state tracking. Regardless of |
| // Vulkan version used, the feature bit is needed (also described in the spec). |
| |
| if (enabled_features.core12.uniformBufferStandardLayout == VK_TRUE) { |
| // --uniform-buffer-standard-layout |
| options.SetUniformBufferStandardLayout(true); |
| } |
| if (enabled_features.core12.scalarBlockLayout == VK_TRUE) { |
| // --scalar-block-layout |
| options.SetScalarBlockLayout(true); |
| } |
| if (enabled_features.workgroup_memory_explicit_layout_features.workgroupMemoryExplicitLayoutScalarBlockLayout) { |
| // --workgroup-scalar-block-layout |
| options.SetWorkgroupScalarBlockLayout(true); |
| } |
| if (enabled_features.core13.maintenance4) { |
| // --allow-localsizeid |
| options.SetAllowLocalSizeId(true); |
| } |
| |
| // Faster validation without friendly names. |
| options.SetFriendlyNames(false); |
| } |