blob: 600545903b5fb4e49e99937ef5dc9d299b18d9a7 [file] [log] [blame]
sfricke-samsung691299b2021-01-01 20:48:48 -08001/* Copyright (c) 2015-2021 The Khronos Group Inc.
2 * Copyright (c) 2015-2021 Valve Corporation
3 * Copyright (c) 2015-2021 LunarG, Inc.
4 * Copyright (C) 2015-2021 Google Inc.
Tobias Hector6663c9b2020-11-05 10:18:02 +00005 * Modifications Copyright (C) 2020 Advanced Micro Devices, Inc. All rights reserved.
Chris Forbes47567b72017-06-09 12:09:45 -07006 *
7 * Licensed under the Apache License, Version 2.0 (the "License");
8 * you may not use this file except in compliance with the License.
9 * You may obtain a copy of the License at
10 *
11 * http://www.apache.org/licenses/LICENSE-2.0
12 *
13 * Unless required by applicable law or agreed to in writing, software
14 * distributed under the License is distributed on an "AS IS" BASIS,
15 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16 * See the License for the specific language governing permissions and
17 * limitations under the License.
18 *
19 * Author: Chris Forbes <chrisf@ijw.co.nz>
Dave Houlton51653902018-06-22 17:32:13 -060020 * Author: Dave Houlton <daveh@lunarg.com>
Tobias Hector6663c9b2020-11-05 10:18:02 +000021 * Author: Tobias Hector <tobias.hector@amd.com>
Chris Forbes47567b72017-06-09 12:09:45 -070022 */
23
Petr Kraus25810d02019-08-27 17:41:15 +020024#include "shader_validation.h"
25
Chris Forbes47567b72017-06-09 12:09:45 -070026#include <cassert>
Petr Kraus25810d02019-08-27 17:41:15 +020027#include <cinttypes>
Jeff Bolzf234bf82019-11-04 14:07:15 -060028#include <cmath>
Chris Forbes47567b72017-06-09 12:09:45 -070029#include <sstream>
Petr Kraus25810d02019-08-27 17:41:15 +020030#include <string>
Petr Kraus25810d02019-08-27 17:41:15 +020031#include <vector>
32
Mark Lobodzinski102687e2020-04-28 11:03:28 -060033#include <spirv/unified1/spirv.hpp>
Chris Forbes47567b72017-06-09 12:09:45 -070034#include "vk_enum_string_helper.h"
Chris Forbes47567b72017-06-09 12:09:45 -070035#include "vk_layer_data.h"
Chris Forbes47567b72017-06-09 12:09:45 -070036#include "vk_layer_utils.h"
Mark Lobodzinskib56bbb92019-02-18 11:49:59 -070037#include "chassis.h"
Chris Forbes47567b72017-06-09 12:09:45 -070038#include "core_validation.h"
sfricke-samsung3c5dee22021-10-14 09:58:14 -070039#include "spirv_grammar_helper.h"
Petr Kraus25810d02019-08-27 17:41:15 +020040
Chris Forbes9a61e082017-07-24 15:35:29 -070041#include "xxhash.h"
Chris Forbes47567b72017-06-09 12:09:45 -070042
Chris Forbes47567b72017-06-09 12:09:45 -070043static shader_stage_attributes shader_stage_attribs[] = {
Ari Suonpaa696b3432019-03-11 14:02:57 +020044 {"vertex shader", false, false, VK_SHADER_STAGE_VERTEX_BIT},
45 {"tessellation control shader", true, true, VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT},
46 {"tessellation evaluation shader", true, false, VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT},
47 {"geometry shader", true, false, VK_SHADER_STAGE_GEOMETRY_BIT},
48 {"fragment shader", false, false, VK_SHADER_STAGE_FRAGMENT_BIT},
Chris Forbes47567b72017-06-09 12:09:45 -070049};
50
ziga-lunarg19fc6ae2021-09-09 00:05:19 +020051static const spirv_inst_iter GetBaseTypeIter(SHADER_MODULE_STATE const *src, uint32_t type) {
52 const auto &insn = src->get_def(type);
53 const uint32_t base_insn_id = src->GetBaseType(insn);
54 return src->get_def(base_insn_id);
55}
56
ziga-lunarg8346fe82021-08-22 17:30:50 +020057static bool BaseTypesMatch(SHADER_MODULE_STATE const *a, SHADER_MODULE_STATE const *b, const spirv_inst_iter &a_base_insn,
58 const spirv_inst_iter &b_base_insn) {
59 const uint32_t a_opcode = a_base_insn.opcode();
60 const uint32_t b_opcode = b_base_insn.opcode();
61 if (a_opcode == b_opcode) {
62 if (a_opcode == spv::OpTypeInt) {
63 // Match width and signedness
64 return a_base_insn.word(2) == b_base_insn.word(2) && a_base_insn.word(3) == b_base_insn.word(3);
65 } else if (a_opcode == spv::OpTypeFloat) {
66 // Match width
67 return a_base_insn.word(2) == b_base_insn.word(2);
68 } else if (a_opcode == spv::OpTypeStruct) {
69 // Match on all element types
70 if (a_base_insn.len() != b_base_insn.len()) {
71 return false; // Structs cannot match if member counts differ
72 }
73
ziga-lunarg19fc6ae2021-09-09 00:05:19 +020074 for (uint32_t i = 2; i < a_base_insn.len(); i++) {
75 const auto &c_base_insn = GetBaseTypeIter(a, a_base_insn.word(i));
76 const auto &d_base_insn = GetBaseTypeIter(b, b_base_insn.word(i));
77 if (!BaseTypesMatch(a, b, c_base_insn, d_base_insn)) {
ziga-lunarg8346fe82021-08-22 17:30:50 +020078 return false;
79 }
80 }
81
82 return true;
83 }
84 }
85 return false;
Chris Forbes47567b72017-06-09 12:09:45 -070086}
87
ziga-lunarg19fc6ae2021-09-09 00:05:19 +020088static bool TypesMatch(SHADER_MODULE_STATE const *a, SHADER_MODULE_STATE const *b, uint32_t a_type, uint32_t b_type) {
89 const auto &a_base_insn = GetBaseTypeIter(a, a_type);
90 const auto &b_base_insn = GetBaseTypeIter(b, b_type);
Chris Forbes47567b72017-06-09 12:09:45 -070091
ziga-lunarg8346fe82021-08-22 17:30:50 +020092 return BaseTypesMatch(a, b, a_base_insn, b_base_insn);
Chris Forbes47567b72017-06-09 12:09:45 -070093}
94
Shannon McPhersonc06c33d2018-06-28 17:21:12 -060095static unsigned GetLocationsConsumedByFormat(VkFormat format) {
Chris Forbes47567b72017-06-09 12:09:45 -070096 switch (format) {
97 case VK_FORMAT_R64G64B64A64_SFLOAT:
98 case VK_FORMAT_R64G64B64A64_SINT:
99 case VK_FORMAT_R64G64B64A64_UINT:
100 case VK_FORMAT_R64G64B64_SFLOAT:
101 case VK_FORMAT_R64G64B64_SINT:
102 case VK_FORMAT_R64G64B64_UINT:
103 return 2;
104 default:
105 return 1;
106 }
107}
108
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600109static unsigned GetFormatType(VkFormat fmt) {
sfricke-samsunge3086292021-11-18 23:02:35 -0800110 if (FormatIsSINT(fmt)) return FORMAT_TYPE_SINT;
111 if (FormatIsUINT(fmt)) return FORMAT_TYPE_UINT;
sfricke-samsunged028b02021-09-06 23:14:51 -0700112 // Formats such as VK_FORMAT_D16_UNORM_S8_UINT are both
Dave Houltona9df0ce2018-02-07 10:51:23 -0700113 if (FormatIsDepthAndStencil(fmt)) return FORMAT_TYPE_FLOAT | FORMAT_TYPE_UINT;
114 if (fmt == VK_FORMAT_UNDEFINED) return 0;
Chris Forbes47567b72017-06-09 12:09:45 -0700115 // everything else -- UNORM/SNORM/FLOAT/USCALED/SSCALED is all float in the shader.
116 return FORMAT_TYPE_FLOAT;
117}
118
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600119static uint32_t GetShaderStageId(VkShaderStageFlagBits stage) {
Chris Forbes47567b72017-06-09 12:09:45 -0700120 uint32_t bit_pos = uint32_t(u_ffs(stage));
121 return bit_pos - 1;
122}
123
Mark Lobodzinskid8d658e2020-01-30 15:05:51 -0700124bool CoreChecks::ValidateViConsistency(VkPipelineVertexInputStateCreateInfo const *vi) const {
Chris Forbes47567b72017-06-09 12:09:45 -0700125 // Walk the binding descriptions, which describe the step rate and stride of each vertex buffer. Each binding should
126 // be specified only once.
Jeremy Gebbencbf22862021-03-03 12:01:22 -0700127 layer_data::unordered_map<uint32_t, VkVertexInputBindingDescription const *> bindings;
Chris Forbes47567b72017-06-09 12:09:45 -0700128 bool skip = false;
129
130 for (unsigned i = 0; i < vi->vertexBindingDescriptionCount; i++) {
131 auto desc = &vi->pVertexBindingDescriptions[i];
132 auto &binding = bindings[desc->binding];
133 if (binding) {
Dave Houlton78d09922018-05-17 15:48:45 -0600134 // TODO: "VUID-VkGraphicsPipelineCreateInfo-pStages-00742" perhaps?
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700135 skip |= LogError(device, kVUID_Core_Shader_InconsistentVi, "Duplicate vertex input binding descriptions for binding %d",
136 desc->binding);
Chris Forbes47567b72017-06-09 12:09:45 -0700137 } else {
138 binding = desc;
139 }
140 }
141
142 return skip;
143}
144
Mark Lobodzinskid8d658e2020-01-30 15:05:51 -0700145bool CoreChecks::ValidateViAgainstVsInputs(VkPipelineVertexInputStateCreateInfo const *vi, SHADER_MODULE_STATE const *vs,
146 spirv_inst_iter entrypoint) const {
Chris Forbes47567b72017-06-09 12:09:45 -0700147 bool skip = false;
148
sfricke-samsung962cad92021-04-13 00:46:29 -0700149 const auto inputs = vs->CollectInterfaceByLocation(entrypoint, spv::StorageClassInput, false);
Chris Forbes47567b72017-06-09 12:09:45 -0700150
151 // Build index by location
Petr Kraus25810d02019-08-27 17:41:15 +0200152 std::map<uint32_t, const VkVertexInputAttributeDescription *> attribs;
Chris Forbes47567b72017-06-09 12:09:45 -0700153 if (vi) {
Petr Kraus25810d02019-08-27 17:41:15 +0200154 for (uint32_t i = 0; i < vi->vertexAttributeDescriptionCount; ++i) {
155 const auto num_locations = GetLocationsConsumedByFormat(vi->pVertexAttributeDescriptions[i].format);
156 for (uint32_t j = 0; j < num_locations; ++j) {
Chris Forbes47567b72017-06-09 12:09:45 -0700157 attribs[vi->pVertexAttributeDescriptions[i].location + j] = &vi->pVertexAttributeDescriptions[i];
158 }
159 }
160 }
161
Petr Kraus25810d02019-08-27 17:41:15 +0200162 struct AttribInputPair {
163 const VkVertexInputAttributeDescription *attrib = nullptr;
164 const interface_var *input = nullptr;
165 };
166 std::map<uint32_t, AttribInputPair> location_map;
167 for (const auto &attrib_it : attribs) location_map[attrib_it.first].attrib = attrib_it.second;
168 for (const auto &input_it : inputs) location_map[input_it.first.first].input = &input_it.second;
Chris Forbes47567b72017-06-09 12:09:45 -0700169
Jamie Madillc1f7ca82020-03-16 17:08:26 -0400170 for (const auto &location_it : location_map) {
Petr Kraus25810d02019-08-27 17:41:15 +0200171 const auto location = location_it.first;
172 const auto attrib = location_it.second.attrib;
173 const auto input = location_it.second.input;
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -0600174
Petr Kraus25810d02019-08-27 17:41:15 +0200175 if (attrib && !input) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600176 skip |= LogPerformanceWarning(vs->vk_shader_module(), kVUID_Core_Shader_OutputNotConsumed,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700177 "Vertex attribute at location %" PRIu32 " not consumed by vertex shader", location);
Petr Kraus25810d02019-08-27 17:41:15 +0200178 } else if (!attrib && input) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600179 skip |= LogError(vs->vk_shader_module(), kVUID_Core_Shader_InputNotProduced,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700180 "Vertex shader consumes input at location %" PRIu32 " but not provided", location);
Petr Kraus25810d02019-08-27 17:41:15 +0200181 } else if (attrib && input) {
182 const auto attrib_type = GetFormatType(attrib->format);
sfricke-samsung962cad92021-04-13 00:46:29 -0700183 const auto input_type = vs->GetFundamentalType(input->type_id);
Chris Forbes47567b72017-06-09 12:09:45 -0700184
185 // Type checking
186 if (!(attrib_type & input_type)) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600187 skip |= LogError(vs->vk_shader_module(), kVUID_Core_Shader_InterfaceTypeMismatch,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700188 "Attribute type of `%s` at location %" PRIu32 " does not match vertex shader input type of `%s`",
sfricke-samsung962cad92021-04-13 00:46:29 -0700189 string_VkFormat(attrib->format), location, vs->DescribeType(input->type_id).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -0700190 }
Petr Kraus25810d02019-08-27 17:41:15 +0200191 } else { // !attrib && !input
192 assert(false); // at least one exists in the map
Chris Forbes47567b72017-06-09 12:09:45 -0700193 }
194 }
195
196 return skip;
197}
198
Mark Lobodzinskid8d658e2020-01-30 15:05:51 -0700199bool CoreChecks::ValidateFsOutputsAgainstRenderPass(SHADER_MODULE_STATE const *fs, spirv_inst_iter entrypoint,
200 PIPELINE_STATE const *pipeline, uint32_t subpass_index) const {
Petr Kraus25810d02019-08-27 17:41:15 +0200201 bool skip = false;
Chris Forbes8bca1652017-07-20 11:10:09 -0700202
Jeremy Hayes3699c7c2019-10-09 12:24:55 -0600203 struct Attachment {
Mike Schuchardt2df08912020-12-15 16:28:09 -0800204 const VkAttachmentReference2 *reference = nullptr;
205 const VkAttachmentDescription2 *attachment = nullptr;
Jeremy Hayes3699c7c2019-10-09 12:24:55 -0600206 const interface_var *output = nullptr;
207 };
208 std::map<uint32_t, Attachment> location_map;
209
amhagana448ea52021-11-02 14:09:14 -0400210 if (pipeline->rp_state && !pipeline->rp_state->use_dynamic_rendering) {
211 const auto rpci = pipeline->rp_state->createInfo.ptr();
212 const auto subpass = rpci->pSubpasses[subpass_index];
213 for (uint32_t i = 0; i < subpass.colorAttachmentCount; ++i) {
214 auto const &reference = subpass.pColorAttachments[i];
215 location_map[i].reference = &reference;
216 if (reference.attachment != VK_ATTACHMENT_UNUSED &&
217 rpci->pAttachments[reference.attachment].format != VK_FORMAT_UNDEFINED) {
218 location_map[i].attachment = &rpci->pAttachments[reference.attachment];
219 }
Chris Forbes47567b72017-06-09 12:09:45 -0700220 }
221 }
222
Chris Forbes47567b72017-06-09 12:09:45 -0700223 // TODO: dual source blend index (spv::DecIndex, zero if not provided)
224
sfricke-samsung962cad92021-04-13 00:46:29 -0700225 const auto outputs = fs->CollectInterfaceByLocation(entrypoint, spv::StorageClassOutput, false);
Jeremy Hayes3699c7c2019-10-09 12:24:55 -0600226 for (const auto &output_it : outputs) {
227 auto const location = output_it.first.first;
228 location_map[location].output = &output_it.second;
229 }
Chris Forbes47567b72017-06-09 12:09:45 -0700230
Jeremy Gebben11af9792021-08-20 10:20:09 -0600231 const bool alpha_to_coverage_enabled = pipeline->create_info.graphics.pMultisampleState != NULL &&
232 pipeline->create_info.graphics.pMultisampleState->alphaToCoverageEnable == VK_TRUE;
Chris Forbes47567b72017-06-09 12:09:45 -0700233
Jamie Madillc1f7ca82020-03-16 17:08:26 -0400234 for (const auto &location_it : location_map) {
Jeremy Hayes3699c7c2019-10-09 12:24:55 -0600235 const auto reference = location_it.second.reference;
236 if (reference != nullptr && reference->attachment == VK_ATTACHMENT_UNUSED) {
237 continue;
238 }
239
Petr Kraus25810d02019-08-27 17:41:15 +0200240 const auto location = location_it.first;
241 const auto attachment = location_it.second.attachment;
242 const auto output = location_it.second.output;
Petr Kraus25810d02019-08-27 17:41:15 +0200243 if (attachment && !output) {
244 if (pipeline->attachments[location].colorWriteMask != 0) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600245 skip |= LogWarning(fs->vk_shader_module(), kVUID_Core_Shader_InputNotProduced,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700246 "Attachment %" PRIu32
247 " not written by fragment shader; undefined values will be written to attachment",
248 location);
Petr Kraus25810d02019-08-27 17:41:15 +0200249 }
250 } else if (!attachment && output) {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700251 if (!(alpha_to_coverage_enabled && location == 0)) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600252 skip |= LogWarning(fs->vk_shader_module(), kVUID_Core_Shader_OutputNotConsumed,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700253 "fragment shader writes to output location %" PRIu32 " with no matching attachment", location);
Ari Suonpaa412b23b2019-02-26 07:56:58 +0200254 }
Petr Kraus25810d02019-08-27 17:41:15 +0200255 } else if (attachment && output) {
256 const auto attachment_type = GetFormatType(attachment->format);
sfricke-samsung962cad92021-04-13 00:46:29 -0700257 const auto output_type = fs->GetFundamentalType(output->type_id);
Chris Forbes47567b72017-06-09 12:09:45 -0700258
259 // Type checking
Petr Kraus25810d02019-08-27 17:41:15 +0200260 if (!(output_type & attachment_type)) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700261 skip |=
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600262 LogWarning(fs->vk_shader_module(), kVUID_Core_Shader_InterfaceTypeMismatch,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700263 "Attachment %" PRIu32
264 " of type `%s` does not match fragment shader output type of `%s`; resulting values are undefined",
sfricke-samsung962cad92021-04-13 00:46:29 -0700265 location, string_VkFormat(attachment->format), fs->DescribeType(output->type_id).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -0700266 }
Petr Kraus25810d02019-08-27 17:41:15 +0200267 } else { // !attachment && !output
268 assert(false); // at least one exists in the map
Chris Forbes47567b72017-06-09 12:09:45 -0700269 }
270 }
271
Petr Kraus25810d02019-08-27 17:41:15 +0200272 const auto output_zero = location_map.count(0) ? location_map[0].output : nullptr;
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700273 bool location_zero_has_alpha = output_zero && fs->get_def(output_zero->type_id) != fs->end() &&
sfricke-samsung962cad92021-04-13 00:46:29 -0700274 fs->GetComponentsConsumedByType(output_zero->type_id, false) == 4;
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700275 if (alpha_to_coverage_enabled && !location_zero_has_alpha) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600276 skip |= LogError(fs->vk_shader_module(), kVUID_Core_Shader_NoAlphaAtLocation0WithAlphaToCoverage,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700277 "fragment shader doesn't declare alpha output at location 0 even though alpha to coverage is enabled.");
Ari Suonpaa412b23b2019-02-26 07:56:58 +0200278 }
279
Chris Forbes47567b72017-06-09 12:09:45 -0700280 return skip;
281}
282
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600283PushConstantByteState CoreChecks::ValidatePushConstantSetUpdate(const std::vector<uint8_t> &push_constant_data_update,
284 const shader_struct_member &push_constant_used_in_shader,
285 uint32_t &out_issue_index) const {
locke-lunargde3f0fa2020-09-10 11:55:31 -0600286 const auto *used_bytes = push_constant_used_in_shader.GetUsedbytes();
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600287 const auto used_bytes_size = used_bytes->size();
288 if (used_bytes_size == 0) return PC_Byte_Updated;
289
290 const auto push_constant_data_update_size = push_constant_data_update.size();
291 const auto *data = push_constant_data_update.data();
292 if ((*data == PC_Byte_Updated) && std::memcmp(data, data + 1, push_constant_data_update_size - 1) == 0) {
293 if (used_bytes_size <= push_constant_data_update_size) {
294 return PC_Byte_Updated;
295 }
296 const auto used_bytes_size1 = used_bytes_size - push_constant_data_update_size;
297
298 const auto *used_bytes_data1 = used_bytes->data() + push_constant_data_update_size;
299 if ((*used_bytes_data1 == 0) && std::memcmp(used_bytes_data1, used_bytes_data1 + 1, used_bytes_size1 - 1) == 0) {
300 return PC_Byte_Updated;
301 }
locke-lunargde3f0fa2020-09-10 11:55:31 -0600302 }
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600303
locke-lunargde3f0fa2020-09-10 11:55:31 -0600304 uint32_t i = 0;
305 for (const auto used : *used_bytes) {
306 if (used) {
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600307 if (i >= push_constant_data_update.size() || push_constant_data_update[i] == PC_Byte_Not_Set) {
locke-lunargde3f0fa2020-09-10 11:55:31 -0600308 out_issue_index = i;
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600309 return PC_Byte_Not_Set;
310 } else if (push_constant_data_update[i] == PC_Byte_Not_Updated) {
locke-lunargde3f0fa2020-09-10 11:55:31 -0600311 out_issue_index = i;
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600312 return PC_Byte_Not_Updated;
locke-lunargde3f0fa2020-09-10 11:55:31 -0600313 }
314 }
315 ++i;
316 }
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600317 return PC_Byte_Updated;
locke-lunargde3f0fa2020-09-10 11:55:31 -0600318}
319
320bool CoreChecks::ValidatePushConstantUsage(const PIPELINE_STATE &pipeline, SHADER_MODULE_STATE const *src,
sfricke-samsung7699b912021-04-12 23:01:51 -0700321 VkPipelineShaderStageCreateInfo const *pStage, const std::string &vuid) const {
Chris Forbes47567b72017-06-09 12:09:45 -0700322 bool skip = false;
sfricke-samsung5c65b372021-03-25 05:39:57 -0700323 // Temp workaround to prevent false positive errors
324 // https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/2450
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -0600325 if (src->HasMultipleEntryPoints()) {
sfricke-samsung5c65b372021-03-25 05:39:57 -0700326 return skip;
327 }
328
Chris Forbes47567b72017-06-09 12:09:45 -0700329 // Validate directly off the offsets. this isn't quite correct for arrays and matrices, but is a good first step.
sfricke-samsung962cad92021-04-13 00:46:29 -0700330 const auto *entrypoint = src->FindEntrypointStruct(pStage->pName, pStage->stage);
locke-lunargde3f0fa2020-09-10 11:55:31 -0600331 if (!entrypoint || !entrypoint->push_constant_used_in_shader.IsUsed()) {
332 return skip;
333 }
334 std::vector<VkPushConstantRange> const *push_constant_ranges = pipeline.pipeline_layout->push_constant_ranges.get();
Chris Forbes47567b72017-06-09 12:09:45 -0700335
locke-lunargde3f0fa2020-09-10 11:55:31 -0600336 bool found_stage = false;
337 for (auto const &range : *push_constant_ranges) {
338 if (range.stageFlags & pStage->stage) {
339 found_stage = true;
340 std::string location_desc;
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600341 std::vector<uint8_t> push_constant_bytes_set;
locke-lunargde3f0fa2020-09-10 11:55:31 -0600342 if (range.offset > 0) {
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600343 push_constant_bytes_set.resize(range.offset, PC_Byte_Not_Set);
locke-lunargde3f0fa2020-09-10 11:55:31 -0600344 }
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600345 push_constant_bytes_set.resize(range.offset + range.size, PC_Byte_Updated);
locke-lunargde3f0fa2020-09-10 11:55:31 -0600346 uint32_t issue_index = 0;
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600347 const auto ret =
348 ValidatePushConstantSetUpdate(push_constant_bytes_set, entrypoint->push_constant_used_in_shader, issue_index);
Chris Forbes47567b72017-06-09 12:09:45 -0700349
locke-lunarg3d8b8f32020-10-26 17:04:16 -0600350 if (ret == PC_Byte_Not_Set) {
locke-lunargde3f0fa2020-09-10 11:55:31 -0600351 const auto loc_descr = entrypoint->push_constant_used_in_shader.GetLocationDesc(issue_index);
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600352 LogObjectList objlist(src->vk_shader_module());
353 objlist.add(pipeline.pipeline_layout->layout());
sfricke-samsung7699b912021-04-12 23:01:51 -0700354 skip |= LogError(objlist, vuid, "Push constant buffer:%s in %s is out of range in %s.", loc_descr.c_str(),
locke-lunargde3f0fa2020-09-10 11:55:31 -0600355 string_VkShaderStageFlags(pStage->stage).c_str(),
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600356 report_data->FormatHandle(pipeline.pipeline_layout->layout()).c_str());
locke-lunargde3f0fa2020-09-10 11:55:31 -0600357 break;
Chris Forbes47567b72017-06-09 12:09:45 -0700358 }
359 }
360 }
361
locke-lunargde3f0fa2020-09-10 11:55:31 -0600362 if (!found_stage) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600363 LogObjectList objlist(src->vk_shader_module());
364 objlist.add(pipeline.pipeline_layout->layout());
sfricke-samsung7699b912021-04-12 23:01:51 -0700365 skip |= LogError(objlist, vuid, "Push constant is used in %s of %s. But %s doesn't set %s.",
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -0600366 string_VkShaderStageFlags(pStage->stage).c_str(), report_data->FormatHandle(src->vk_shader_module()).c_str(),
367 report_data->FormatHandle(pipeline.pipeline_layout->layout()).c_str(),
sfricke-samsung7699b912021-04-12 23:01:51 -0700368 string_VkShaderStageFlags(pStage->stage).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -0700369 }
Chris Forbes47567b72017-06-09 12:09:45 -0700370 return skip;
371}
372
sfricke-samsungcfb44592021-07-25 00:36:28 -0700373bool CoreChecks::ValidateBuiltinLimits(SHADER_MODULE_STATE const *src, spirv_inst_iter entrypoint) const {
sfricke-samsungef2a68c2020-10-26 04:22:46 -0700374 bool skip = false;
375
376 // Currently all builtin tested are only found in fragment shaders
sfricke-samsungcfb44592021-07-25 00:36:28 -0700377 if (entrypoint.word(1) != spv::ExecutionModelFragment) {
sfricke-samsungef2a68c2020-10-26 04:22:46 -0700378 return skip;
379 }
380
sfricke-samsungcfb44592021-07-25 00:36:28 -0700381 // Find all builtin from just the interface variables
382 for (uint32_t id : FindEntrypointInterfaces(entrypoint)) {
sfricke-samsungef2a68c2020-10-26 04:22:46 -0700383 auto insn = src->get_def(id);
sfricke-samsungcfb44592021-07-25 00:36:28 -0700384 assert(insn.opcode() == spv::OpVariable);
sfricke-samsungef2a68c2020-10-26 04:22:46 -0700385 const decoration_set decorations = src->get_decorations(insn.word(2));
386
sfricke-samsungcfb44592021-07-25 00:36:28 -0700387 // Currently don't need to search in structs
388 if (((decorations.flags & decoration_set::builtin_bit) != 0) && (decorations.builtin == spv::BuiltInSampleMask)) {
sfricke-samsungef2a68c2020-10-26 04:22:46 -0700389 auto type_pointer = src->get_def(insn.word(1));
390 assert(type_pointer.opcode() == spv::OpTypePointer);
391
392 auto type = src->get_def(type_pointer.word(3));
393 if (type.opcode() == spv::OpTypeArray) {
sfricke-samsung962cad92021-04-13 00:46:29 -0700394 uint32_t length = static_cast<uint32_t>(src->GetConstantValueById(type.word(3)));
sfricke-samsungcfb44592021-07-25 00:36:28 -0700395 // Handles both the input and output sampleMask
396 if (length > phys_dev_props.limits.maxSampleMaskWords) {
397 skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-maxSampleMaskWords-00711",
398 "vkCreateGraphicsPipelines(): The BuiltIns SampleMask array sizes is %u which exceeds "
399 "maxSampleMaskWords of %u in %s.",
400 length, phys_dev_props.limits.maxSampleMaskWords,
401 report_data->FormatHandle(src->vk_shader_module()).c_str());
sfricke-samsungef2a68c2020-10-26 04:22:46 -0700402 }
sfricke-samsungcfb44592021-07-25 00:36:28 -0700403 break;
sfricke-samsungef2a68c2020-10-26 04:22:46 -0700404 }
405 }
406 }
407
408 return skip;
409}
410
Chris Forbes47567b72017-06-09 12:09:45 -0700411// Validate that data for each specialization entry is fully contained within the buffer.
ziga-lunargae2a5c42021-07-23 16:18:09 +0200412bool CoreChecks::ValidateSpecializations(VkPipelineShaderStageCreateInfo const *info) const {
Chris Forbes47567b72017-06-09 12:09:45 -0700413 bool skip = false;
414
415 VkSpecializationInfo const *spec = info->pSpecializationInfo;
416
417 if (spec) {
418 for (auto i = 0u; i < spec->mapEntryCount; i++) {
Jeremy Hayes6c555c32019-09-09 17:14:09 -0600419 if (spec->pMapEntries[i].offset >= spec->dataSize) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700420 skip |= LogError(device, "VUID-VkSpecializationInfo-offset-00773",
421 "Specialization entry %u (for constant id %u) references memory outside provided specialization "
Petr Krausb0d5e592021-05-21 23:37:11 +0200422 "data (bytes %u.." PRINTF_SIZE_T_SPECIFIER "; " PRINTF_SIZE_T_SPECIFIER " bytes provided).",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700423 i, spec->pMapEntries[i].constantID, spec->pMapEntries[i].offset,
424 spec->pMapEntries[i].offset + spec->dataSize - 1, spec->dataSize);
Jeremy Hayes6c555c32019-09-09 17:14:09 -0600425
426 continue;
427 }
Chris Forbes47567b72017-06-09 12:09:45 -0700428 if (spec->pMapEntries[i].offset + spec->pMapEntries[i].size > spec->dataSize) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700429 skip |= LogError(device, "VUID-VkSpecializationInfo-pMapEntries-00774",
430 "Specialization entry %u (for constant id %u) references memory outside provided specialization "
Petr Krausb0d5e592021-05-21 23:37:11 +0200431 "data (bytes %u.." PRINTF_SIZE_T_SPECIFIER "; " PRINTF_SIZE_T_SPECIFIER " bytes provided).",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700432 i, spec->pMapEntries[i].constantID, spec->pMapEntries[i].offset,
433 spec->pMapEntries[i].offset + spec->pMapEntries[i].size - 1, spec->dataSize);
Chris Forbes47567b72017-06-09 12:09:45 -0700434 }
ziga-lunargae2a5c42021-07-23 16:18:09 +0200435 for (uint32_t j = i + 1; j < spec->mapEntryCount; ++j) {
436 if (spec->pMapEntries[i].constantID == spec->pMapEntries[j].constantID) {
437 skip |= LogError(device, "VUID-VkSpecializationInfo-constantID-04911",
438 "Specialization entry %" PRIu32 " and %" PRIu32 " have the same constantID (%" PRIu32 ").", i,
439 j, spec->pMapEntries[i].constantID);
440 }
441 }
Chris Forbes47567b72017-06-09 12:09:45 -0700442 }
443 }
444
445 return skip;
446}
447
Jeff Bolz38b3ce72018-09-19 12:53:38 -0500448// TODO (jbolz): Can this return a const reference?
sourav parmarcd5fb182020-07-17 12:58:44 -0700449static std::set<uint32_t> TypeToDescriptorTypeSet(SHADER_MODULE_STATE const *module, uint32_t type_id, unsigned &descriptor_count,
450 bool is_khr) {
Chris Forbes47567b72017-06-09 12:09:45 -0700451 auto type = module->get_def(type_id);
Chris Forbes9f89d752018-03-07 12:57:48 -0800452 bool is_storage_buffer = false;
Chris Forbes47567b72017-06-09 12:09:45 -0700453 descriptor_count = 1;
Jeff Bolze54ae892018-09-08 12:16:29 -0500454 std::set<uint32_t> ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700455
456 // Strip off any array or ptrs. Where we remove array levels, adjust the descriptor count for each dimension.
Jeff Bolzfdf96072018-04-10 14:32:18 -0500457 while (type.opcode() == spv::OpTypeArray || type.opcode() == spv::OpTypePointer || type.opcode() == spv::OpTypeRuntimeArray) {
458 if (type.opcode() == spv::OpTypeRuntimeArray) {
459 descriptor_count = 0;
460 type = module->get_def(type.word(2));
461 } else if (type.opcode() == spv::OpTypeArray) {
sfricke-samsung962cad92021-04-13 00:46:29 -0700462 descriptor_count *= module->GetConstantValueById(type.word(3));
Chris Forbes47567b72017-06-09 12:09:45 -0700463 type = module->get_def(type.word(2));
464 } else {
Chris Forbes9f89d752018-03-07 12:57:48 -0800465 if (type.word(2) == spv::StorageClassStorageBuffer) {
466 is_storage_buffer = true;
467 }
Chris Forbes47567b72017-06-09 12:09:45 -0700468 type = module->get_def(type.word(3));
469 }
470 }
471
472 switch (type.opcode()) {
473 case spv::OpTypeStruct: {
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -0600474 for (const auto insn : module->GetDecorationInstructions()) {
sfricke-samsung94d71a52021-02-26 05:25:43 -0800475 if (insn.word(1) == type.word(1)) {
Chris Forbes47567b72017-06-09 12:09:45 -0700476 if (insn.word(2) == spv::DecorationBlock) {
Chris Forbes9f89d752018-03-07 12:57:48 -0800477 if (is_storage_buffer) {
Jeff Bolze54ae892018-09-08 12:16:29 -0500478 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER);
479 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC);
480 return ret;
Chris Forbes9f89d752018-03-07 12:57:48 -0800481 } else {
Jeff Bolze54ae892018-09-08 12:16:29 -0500482 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER);
483 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC);
484 ret.insert(VK_DESCRIPTOR_TYPE_INLINE_UNIFORM_BLOCK_EXT);
485 return ret;
Chris Forbes9f89d752018-03-07 12:57:48 -0800486 }
Chris Forbes47567b72017-06-09 12:09:45 -0700487 } else if (insn.word(2) == spv::DecorationBufferBlock) {
Jeff Bolze54ae892018-09-08 12:16:29 -0500488 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER);
489 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC);
490 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700491 }
492 }
493 }
494
495 // Invalid
Jeff Bolze54ae892018-09-08 12:16:29 -0500496 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700497 }
498
499 case spv::OpTypeSampler:
Jeff Bolze54ae892018-09-08 12:16:29 -0500500 ret.insert(VK_DESCRIPTOR_TYPE_SAMPLER);
501 ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER);
502 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700503
Chris Forbes73c00bf2018-06-22 16:28:06 -0700504 case spv::OpTypeSampledImage: {
505 // Slight relaxation for some GLSL historical madness: samplerBuffer doesn't really have a sampler, and a texel
506 // buffer descriptor doesn't really provide one. Allow this slight mismatch.
507 auto image_type = module->get_def(type.word(2));
508 auto dim = image_type.word(3);
509 auto sampled = image_type.word(7);
510 if (dim == spv::DimBuffer && sampled == 1) {
Jeff Bolze54ae892018-09-08 12:16:29 -0500511 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER);
512 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700513 }
Chris Forbes73c00bf2018-06-22 16:28:06 -0700514 }
Jeff Bolze54ae892018-09-08 12:16:29 -0500515 ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER);
516 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700517
518 case spv::OpTypeImage: {
519 // Many descriptor types backing image types-- depends on dimension and whether the image will be used with a sampler.
520 // SPIRV for Vulkan requires that sampled be 1 or 2 -- leaving the decision to runtime is unacceptable.
521 auto dim = type.word(3);
522 auto sampled = type.word(7);
523
524 if (dim == spv::DimSubpassData) {
Jeff Bolze54ae892018-09-08 12:16:29 -0500525 ret.insert(VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
526 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700527 } else if (dim == spv::DimBuffer) {
528 if (sampled == 1) {
Jeff Bolze54ae892018-09-08 12:16:29 -0500529 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER);
530 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700531 } else {
Jeff Bolze54ae892018-09-08 12:16:29 -0500532 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER);
533 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700534 }
535 } else if (sampled == 1) {
Jeff Bolze54ae892018-09-08 12:16:29 -0500536 ret.insert(VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE);
537 ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER);
538 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700539 } else {
Jeff Bolze54ae892018-09-08 12:16:29 -0500540 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_IMAGE);
541 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700542 }
543 }
Shannon McPherson0fa28232018-11-01 11:59:02 -0600544 case spv::OpTypeAccelerationStructureNV:
sourav parmarcd5fb182020-07-17 12:58:44 -0700545 is_khr ? ret.insert(VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_KHR)
546 : ret.insert(VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_NV);
Jeff Bolz105d6492018-09-29 15:46:44 -0500547 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -0700548
549 // We shouldn't really see any other junk types -- but if we do, they're a mismatch.
550 default:
Jeff Bolze54ae892018-09-08 12:16:29 -0500551 return ret; // Matches nothing
Chris Forbes47567b72017-06-09 12:09:45 -0700552 }
553}
554
Jeff Bolze54ae892018-09-08 12:16:29 -0500555static std::string string_descriptorTypes(const std::set<uint32_t> &descriptor_types) {
Chris Forbes73c00bf2018-06-22 16:28:06 -0700556 std::stringstream ss;
Jeff Bolze54ae892018-09-08 12:16:29 -0500557 for (auto it = descriptor_types.begin(); it != descriptor_types.end(); ++it) {
558 if (ss.tellp()) ss << ", ";
559 ss << string_VkDescriptorType(VkDescriptorType(*it));
Chris Forbes73c00bf2018-06-22 16:28:06 -0700560 }
561 return ss.str();
562}
563
sfricke-samsung0065ce02020-12-03 22:46:37 -0800564bool CoreChecks::RequirePropertyFlag(VkBool32 check, char const *flag, char const *structure, const char *vuid) const {
Jeff Bolzee743412019-06-20 22:24:32 -0500565 if (!check) {
sfricke-samsung0065ce02020-12-03 22:46:37 -0800566 if (LogError(device, vuid, "Shader requires flag %s set in %s but it is not set on the device", flag, structure)) {
Jeff Bolzee743412019-06-20 22:24:32 -0500567 return true;
568 }
569 }
570
571 return false;
572}
573
sfricke-samsung0065ce02020-12-03 22:46:37 -0800574bool CoreChecks::RequireFeature(VkBool32 feature, char const *feature_name, const char *vuid) const {
Chris Forbes47567b72017-06-09 12:09:45 -0700575 if (!feature) {
sfricke-samsung0065ce02020-12-03 22:46:37 -0800576 if (LogError(device, vuid, "Shader requires %s but is not enabled on the device", feature_name)) {
Chris Forbes47567b72017-06-09 12:09:45 -0700577 return true;
578 }
579 }
580
581 return false;
582}
583
locke-lunarg63e4daf2020-08-17 17:53:25 -0600584bool CoreChecks::ValidateShaderStageWritableOrAtomicDescriptor(VkShaderStageFlagBits stage, bool has_writable_descriptor,
585 bool has_atomic_descriptor) const {
Jeff Bolzee743412019-06-20 22:24:32 -0500586 bool skip = false;
587
locke-lunarg63e4daf2020-08-17 17:53:25 -0600588 if (has_writable_descriptor || has_atomic_descriptor) {
Chris Forbes349b3132018-03-07 11:38:08 -0800589 switch (stage) {
590 case VK_SHADER_STAGE_COMPUTE_BIT:
Jeff Bolz148d94e2018-12-13 21:25:56 -0600591 case VK_SHADER_STAGE_RAYGEN_BIT_NV:
592 case VK_SHADER_STAGE_ANY_HIT_BIT_NV:
593 case VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV:
594 case VK_SHADER_STAGE_MISS_BIT_NV:
595 case VK_SHADER_STAGE_INTERSECTION_BIT_NV:
596 case VK_SHADER_STAGE_CALLABLE_BIT_NV:
597 case VK_SHADER_STAGE_TASK_BIT_NV:
598 case VK_SHADER_STAGE_MESH_BIT_NV:
Chris Forbes349b3132018-03-07 11:38:08 -0800599 /* No feature requirements for writes and atomics from compute
Jeff Bolz148d94e2018-12-13 21:25:56 -0600600 * raytracing, or mesh stages */
Chris Forbes349b3132018-03-07 11:38:08 -0800601 break;
602 case VK_SHADER_STAGE_FRAGMENT_BIT:
sfricke-samsung0065ce02020-12-03 22:46:37 -0800603 skip |= RequireFeature(enabled_features.core.fragmentStoresAndAtomics, "fragmentStoresAndAtomics",
sfricke-samsung1ff329f2021-09-16 10:06:47 -0700604 "VUID-RuntimeSpirv-NonWritable-06340");
Chris Forbes349b3132018-03-07 11:38:08 -0800605 break;
606 default:
sfricke-samsung0065ce02020-12-03 22:46:37 -0800607 skip |= RequireFeature(enabled_features.core.vertexPipelineStoresAndAtomics, "vertexPipelineStoresAndAtomics",
sfricke-samsung1ff329f2021-09-16 10:06:47 -0700608 "VUID-RuntimeSpirv-NonWritable-06341");
Chris Forbes349b3132018-03-07 11:38:08 -0800609 break;
610 }
611 }
612
Chris Forbes47567b72017-06-09 12:09:45 -0700613 return skip;
614}
615
sfricke-samsung94167ca2021-02-26 04:14:59 -0800616bool CoreChecks::ValidateShaderStageGroupNonUniform(SHADER_MODULE_STATE const *module, VkShaderStageFlagBits stage,
617 spirv_inst_iter &insn) const {
Jeff Bolzee743412019-06-20 22:24:32 -0500618 bool skip = false;
619
sfricke-samsung94167ca2021-02-26 04:14:59 -0800620 // Check anything using a group operation (which currently is only OpGroupNonUnifrom* operations)
621 if (GroupOperation(insn.opcode()) == true) {
622 // Check the quad operations.
623 if ((insn.opcode() == spv::OpGroupNonUniformQuadBroadcast) || (insn.opcode() == spv::OpGroupNonUniformQuadSwap)) {
624 if ((stage != VK_SHADER_STAGE_FRAGMENT_BIT) && (stage != VK_SHADER_STAGE_COMPUTE_BIT)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -0700625 skip |=
626 RequireFeature(phys_dev_props_core11.subgroupQuadOperationsInAllStages,
627 "VkPhysicalDeviceSubgroupProperties::quadOperationsInAllStages", "VUID-RuntimeSpirv-None-06342");
sfricke-samsung0065ce02020-12-03 22:46:37 -0800628 }
sfricke-samsung94167ca2021-02-26 04:14:59 -0800629 }
Jeff Bolz526f2d52019-09-18 13:18:08 -0500630
sfricke-samsung94167ca2021-02-26 04:14:59 -0800631 uint32_t scope_type = spv::ScopeMax;
632 if (insn.opcode() == spv::OpGroupNonUniformPartitionNV) {
633 // OpGroupNonUniformPartitionNV always assumed subgroup as missing operand
634 scope_type = spv::ScopeSubgroup;
635 } else {
636 // "All <id> used for Scope <id> must be of an OpConstant"
637 auto scope_id = module->get_def(insn.word(3));
638 scope_type = scope_id.word(3);
639 }
sfricke-samsung0065ce02020-12-03 22:46:37 -0800640
sfricke-samsung94167ca2021-02-26 04:14:59 -0800641 if (scope_type == spv::ScopeSubgroup) {
642 // "Group operations with subgroup scope" must have stage support
643 const VkSubgroupFeatureFlags supported_stages = phys_dev_props_core11.subgroupSupportedStages;
644 skip |= RequirePropertyFlag(supported_stages & stage, string_VkShaderStageFlagBits(stage),
sfricke-samsung1ff329f2021-09-16 10:06:47 -0700645 "VkPhysicalDeviceSubgroupProperties::supportedStages", "VUID-RuntimeSpirv-None-06343");
sfricke-samsung94167ca2021-02-26 04:14:59 -0800646 }
647
648 if (!enabled_features.core12.shaderSubgroupExtendedTypes) {
649 auto type = module->get_def(insn.word(1));
650
651 if (type.opcode() == spv::OpTypeVector) {
652 // Get the element type
653 type = module->get_def(type.word(2));
sfricke-samsung0065ce02020-12-03 22:46:37 -0800654 }
655
sfricke-samsung94167ca2021-02-26 04:14:59 -0800656 if (type.opcode() != spv::OpTypeBool) {
sfricke-samsung0065ce02020-12-03 22:46:37 -0800657 // Both OpTypeInt and OpTypeFloat the width is in the 2nd word.
658 const uint32_t width = type.word(2);
Jeff Bolz526f2d52019-09-18 13:18:08 -0500659
sfricke-samsung0065ce02020-12-03 22:46:37 -0800660 if ((type.opcode() == spv::OpTypeFloat && width == 16) ||
661 (type.opcode() == spv::OpTypeInt && (width == 8 || width == 16 || width == 64))) {
662 skip |= RequireFeature(enabled_features.core12.shaderSubgroupExtendedTypes,
663 "VkPhysicalDeviceShaderSubgroupExtendedTypesFeatures::shaderSubgroupExtendedTypes",
sfricke-samsung1ff329f2021-09-16 10:06:47 -0700664 "VUID-RuntimeSpirv-None-06275");
Jeff Bolz526f2d52019-09-18 13:18:08 -0500665 }
666 }
667 }
Jeff Bolzee743412019-06-20 22:24:32 -0500668 }
669
670 return skip;
671}
672
ziga-lunarg70651522021-10-11 17:23:30 +0200673bool CoreChecks::ValidateMemoryScope(SHADER_MODULE_STATE const *src, const spirv_inst_iter &insn) const {
674 bool skip = false;
675
sfricke-samsung3511e312021-11-04 21:14:31 -0700676 const auto &entry = MemoryScopeParamPosition(insn.opcode());
ziga-lunarg70651522021-10-11 17:23:30 +0200677 if (entry > 0) {
678 const uint32_t scope_id = insn.word(entry);
679 if (enabled_features.core12.vulkanMemoryModel && !enabled_features.core12.vulkanMemoryModelDeviceScope) {
680 const auto &iter = src->GetConstantDef(scope_id);
681 if (iter != src->end()) {
682 if (GetConstantValue(iter) == spv::Scope::ScopeDevice) {
683 skip |= LogError(device, "VUID-RuntimeSpirv-vulkanMemoryModel-06265",
684 "VkPhysicalDeviceVulkan12Features::vulkanMemoryModel is enabled and "
685 "VkPhysicalDeviceVulkan12Features::vulkanMemoryModelDeviceScope is disabled, but Device "
686 "memory scope is used.");
687 }
688 }
689 } else if (!enabled_features.core12.vulkanMemoryModel) {
690 const auto &iter = src->GetConstantDef(scope_id);
691 if (iter != src->end()) {
692 if (GetConstantValue(iter) == spv::Scope::ScopeQueueFamily) {
693 skip |= LogError(device, "VUID-RuntimeSpirv-vulkanMemoryModel-06266",
694 "VkPhysicalDeviceVulkan12Features::vulkanMemoryModel is not enabled, but QueueFamily "
695 "memory scope is used.");
696 }
697 }
698 }
699 }
700
701 return skip;
702}
703
ziga-lunarg2818f492021-08-12 14:30:51 +0200704bool CoreChecks::ValidateWorkgroupSize(SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
705 const std::unordered_map<uint32_t, std::vector<uint32_t>>& id_value_map) const {
706 bool skip = false;
707
708 std::array<uint32_t, 3> work_group_size = src->GetWorkgroupSize(pStage, id_value_map);
709
710 for (uint32_t i = 0; i < 3; ++i) {
711 if (work_group_size[i] > phys_dev_props.limits.maxComputeWorkGroupSize[i]) {
712 const char member = 'x' + static_cast<int8_t>(i);
713 skip |= LogError(device, kVUID_Core_Shader_MaxComputeWorkGroupSize,
714 "Specialization constant is being used to specialize WorkGroupSize.%c, but value (%" PRIu32
715 ") is greater than VkPhysicalDeviceLimits::maxComputeWorkGroupSize[%" PRIu32 "] = %" PRIu32 ".",
716 member, work_group_size[i], i, phys_dev_props.limits.maxComputeWorkGroupSize[i]);
717 }
718 }
719 return skip;
720}
721
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600722bool CoreChecks::ValidateShaderStageInputOutputLimits(SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
John Zulaufac4c6e12019-07-01 16:05:58 -0600723 const PIPELINE_STATE *pipeline, spirv_inst_iter entrypoint) const {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200724 if (pStage->stage == VK_SHADER_STAGE_COMPUTE_BIT || pStage->stage == VK_SHADER_STAGE_ALL_GRAPHICS ||
725 pStage->stage == VK_SHADER_STAGE_ALL) {
726 return false;
727 }
728
729 bool skip = false;
Mark Lobodzinski518eadc2019-03-09 12:07:30 -0700730 auto const &limits = phys_dev_props.limits;
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200731
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700732 std::set<uint32_t> patch_i_ds;
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200733 struct Variable {
734 uint32_t baseTypePtrID;
735 uint32_t ID;
736 uint32_t storageClass;
737 };
738 std::vector<Variable> variables;
739
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700740 uint32_t num_vertices = 0;
Nathaniel Cesario75fb7222020-12-07 10:54:53 -0700741 bool is_iso_lines = false;
742 bool is_point_mode = false;
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500743
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700744 auto entrypoint_variables = FindEntrypointInterfaces(entrypoint);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600745
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200746 for (auto insn : *src) {
747 switch (insn.opcode()) {
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500748 // Find all Patch decorations
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200749 case spv::OpDecorate:
750 switch (insn.word(2)) {
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500751 case spv::DecorationPatch: {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700752 patch_i_ds.insert(insn.word(1));
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200753 break;
754 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200755 default:
756 break;
757 }
758 break;
759 // Find all input and output variables
760 case spv::OpVariable: {
761 Variable var = {};
762 var.storageClass = insn.word(3);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600763 if ((var.storageClass == spv::StorageClassInput || var.storageClass == spv::StorageClassOutput) &&
764 // Only include variables in the entrypoint's interface
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700765 find(entrypoint_variables.begin(), entrypoint_variables.end(), insn.word(2)) != entrypoint_variables.end()) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200766 var.baseTypePtrID = insn.word(1);
767 var.ID = insn.word(2);
768 variables.push_back(var);
769 }
770 break;
771 }
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500772 case spv::OpExecutionMode:
773 if (insn.word(1) == entrypoint.word(2)) {
774 switch (insn.word(2)) {
775 default:
776 break;
777 case spv::ExecutionModeOutputVertices:
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700778 num_vertices = insn.word(3);
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500779 break;
Nathaniel Cesario75fb7222020-12-07 10:54:53 -0700780 case spv::ExecutionModeIsolines:
781 is_iso_lines = true;
782 break;
783 case spv::ExecutionModePointMode:
784 is_point_mode = true;
785 break;
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500786 }
787 }
788 break;
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200789 default:
790 break;
791 }
792 }
793
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500794 bool strip_output_array_level =
795 (pStage->stage == VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT || pStage->stage == VK_SHADER_STAGE_MESH_BIT_NV);
796 bool strip_input_array_level =
797 (pStage->stage == VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT ||
798 pStage->stage == VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT || pStage->stage == VK_SHADER_STAGE_GEOMETRY_BIT);
799
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700800 uint32_t num_comp_in = 0, num_comp_out = 0;
801 int max_comp_in = 0, max_comp_out = 0;
Jeff Bolzf234bf82019-11-04 14:07:15 -0600802
sfricke-samsung962cad92021-04-13 00:46:29 -0700803 auto inputs = src->CollectInterfaceByLocation(entrypoint, spv::StorageClassInput, strip_input_array_level);
804 auto outputs = src->CollectInterfaceByLocation(entrypoint, spv::StorageClassOutput, strip_output_array_level);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600805
806 // Find max component location used for input variables.
807 for (auto &var : inputs) {
808 int location = var.first.first;
809 int component = var.first.second;
810 interface_var &iv = var.second;
811
812 // Only need to look at the first location, since we use the type's whole size
813 if (iv.offset != 0) {
814 continue;
815 }
816
817 if (iv.is_patch) {
818 continue;
819 }
820
sfricke-samsung962cad92021-04-13 00:46:29 -0700821 int num_components = src->GetComponentsConsumedByType(iv.type_id, strip_input_array_level);
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700822 max_comp_in = std::max(max_comp_in, location * 4 + component + num_components);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600823 }
824
825 // Find max component location used for output variables.
826 for (auto &var : outputs) {
827 int location = var.first.first;
828 int component = var.first.second;
829 interface_var &iv = var.second;
830
831 // Only need to look at the first location, since we use the type's whole size
832 if (iv.offset != 0) {
833 continue;
834 }
835
836 if (iv.is_patch) {
837 continue;
838 }
839
sfricke-samsung962cad92021-04-13 00:46:29 -0700840 int num_components = src->GetComponentsConsumedByType(iv.type_id, strip_output_array_level);
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700841 max_comp_out = std::max(max_comp_out, location * 4 + component + num_components);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600842 }
843
844 // XXX TODO: Would be nice to rewrite this to use CollectInterfaceByLocation (or something similar),
845 // but that doesn't include builtins.
sfricke-samsung406766a2021-07-02 12:04:09 -0700846 // When rewritten, using the CreatePipelineExceedVertexMaxComponentsWithBuiltins test it would be nice to also let the user know
847 // how many components were from builtins as it might not be obvious
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200848 for (auto &var : variables) {
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500849 // Check if the variable is a patch. Patches can also be members of blocks,
850 // but if they are then the top-level arrayness has already been stripped
851 // by the time GetComponentsConsumedByType gets to it.
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700852 bool is_patch = patch_i_ds.find(var.ID) != patch_i_ds.end();
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200853
854 if (var.storageClass == spv::StorageClassInput) {
sfricke-samsung962cad92021-04-13 00:46:29 -0700855 num_comp_in += src->GetComponentsConsumedByType(var.baseTypePtrID, strip_input_array_level && !is_patch);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200856 } else { // var.storageClass == spv::StorageClassOutput
sfricke-samsung962cad92021-04-13 00:46:29 -0700857 num_comp_out += src->GetComponentsConsumedByType(var.baseTypePtrID, strip_output_array_level && !is_patch);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200858 }
859 }
860
861 switch (pStage->stage) {
862 case VK_SHADER_STAGE_VERTEX_BIT:
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700863 if (num_comp_out > limits.maxVertexOutputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700864 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700865 "Invalid Pipeline CreateInfo State: Vertex shader exceeds "
866 "VkPhysicalDeviceLimits::maxVertexOutputComponents of %u "
867 "components by %u components",
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700868 limits.maxVertexOutputComponents, num_comp_out - limits.maxVertexOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200869 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700870 if (max_comp_out > static_cast<int>(limits.maxVertexOutputComponents)) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700871 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700872 "Invalid Pipeline CreateInfo State: Vertex shader output variable uses location that "
873 "exceeds component limit VkPhysicalDeviceLimits::maxVertexOutputComponents (%u)",
874 limits.maxVertexOutputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600875 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200876 break;
877
878 case VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT:
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700879 if (num_comp_in > limits.maxTessellationControlPerVertexInputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700880 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700881 "Invalid Pipeline CreateInfo State: Tessellation control shader exceeds "
882 "VkPhysicalDeviceLimits::maxTessellationControlPerVertexInputComponents of %u "
883 "components by %u components",
884 limits.maxTessellationControlPerVertexInputComponents,
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700885 num_comp_in - limits.maxTessellationControlPerVertexInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200886 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700887 if (max_comp_in > static_cast<int>(limits.maxTessellationControlPerVertexInputComponents)) {
Jeff Bolzf234bf82019-11-04 14:07:15 -0600888 skip |=
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700889 LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700890 "Invalid Pipeline CreateInfo State: Tessellation control shader input variable uses location that "
891 "exceeds component limit VkPhysicalDeviceLimits::maxTessellationControlPerVertexInputComponents (%u)",
892 limits.maxTessellationControlPerVertexInputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600893 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700894 if (num_comp_out > limits.maxTessellationControlPerVertexOutputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700895 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700896 "Invalid Pipeline CreateInfo State: Tessellation control shader exceeds "
897 "VkPhysicalDeviceLimits::maxTessellationControlPerVertexOutputComponents of %u "
898 "components by %u components",
899 limits.maxTessellationControlPerVertexOutputComponents,
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700900 num_comp_out - limits.maxTessellationControlPerVertexOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200901 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700902 if (max_comp_out > static_cast<int>(limits.maxTessellationControlPerVertexOutputComponents)) {
Jeff Bolzf234bf82019-11-04 14:07:15 -0600903 skip |=
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700904 LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700905 "Invalid Pipeline CreateInfo State: Tessellation control shader output variable uses location that "
906 "exceeds component limit VkPhysicalDeviceLimits::maxTessellationControlPerVertexOutputComponents (%u)",
907 limits.maxTessellationControlPerVertexOutputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600908 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200909 break;
910
911 case VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT:
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700912 if (num_comp_in > limits.maxTessellationEvaluationInputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700913 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700914 "Invalid Pipeline CreateInfo State: Tessellation evaluation shader exceeds "
915 "VkPhysicalDeviceLimits::maxTessellationEvaluationInputComponents of %u "
916 "components by %u components",
917 limits.maxTessellationEvaluationInputComponents,
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700918 num_comp_in - limits.maxTessellationEvaluationInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200919 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700920 if (max_comp_in > static_cast<int>(limits.maxTessellationEvaluationInputComponents)) {
Jeff Bolzf234bf82019-11-04 14:07:15 -0600921 skip |=
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700922 LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700923 "Invalid Pipeline CreateInfo State: Tessellation evaluation shader input variable uses location that "
924 "exceeds component limit VkPhysicalDeviceLimits::maxTessellationEvaluationInputComponents (%u)",
925 limits.maxTessellationEvaluationInputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600926 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700927 if (num_comp_out > limits.maxTessellationEvaluationOutputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700928 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700929 "Invalid Pipeline CreateInfo State: Tessellation evaluation shader exceeds "
930 "VkPhysicalDeviceLimits::maxTessellationEvaluationOutputComponents of %u "
931 "components by %u components",
932 limits.maxTessellationEvaluationOutputComponents,
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700933 num_comp_out - limits.maxTessellationEvaluationOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200934 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700935 if (max_comp_out > static_cast<int>(limits.maxTessellationEvaluationOutputComponents)) {
Jeff Bolzf234bf82019-11-04 14:07:15 -0600936 skip |=
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700937 LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700938 "Invalid Pipeline CreateInfo State: Tessellation evaluation shader output variable uses location that "
939 "exceeds component limit VkPhysicalDeviceLimits::maxTessellationEvaluationOutputComponents (%u)",
940 limits.maxTessellationEvaluationOutputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600941 }
Nathaniel Cesario75fb7222020-12-07 10:54:53 -0700942 // Portability validation
943 if (IsExtEnabled(device_extensions.vk_khr_portability_subset)) {
944 if (is_iso_lines && (VK_FALSE == enabled_features.portability_subset_features.tessellationIsolines)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -0700945 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-tessellationShader-06326",
Nathaniel Cesario75fb7222020-12-07 10:54:53 -0700946 "Invalid Pipeline CreateInfo state (portability error): Tessellation evaluation shader"
947 " is using abstract patch type IsoLines, but this is not supported on this platform");
948 }
949 if (is_point_mode && (VK_FALSE == enabled_features.portability_subset_features.tessellationPointMode)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -0700950 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-tessellationShader-06327",
Nathaniel Cesario75fb7222020-12-07 10:54:53 -0700951 "Invalid Pipeline CreateInfo state (portability error): Tessellation evaluation shader"
952 " is using abstract patch type PointMode, but this is not supported on this platform");
953 }
954 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200955 break;
956
957 case VK_SHADER_STAGE_GEOMETRY_BIT:
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700958 if (num_comp_in > limits.maxGeometryInputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700959 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700960 "Invalid Pipeline CreateInfo State: Geometry shader exceeds "
961 "VkPhysicalDeviceLimits::maxGeometryInputComponents of %u "
962 "components by %u components",
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700963 limits.maxGeometryInputComponents, num_comp_in - limits.maxGeometryInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200964 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700965 if (max_comp_in > static_cast<int>(limits.maxGeometryInputComponents)) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700966 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700967 "Invalid Pipeline CreateInfo State: Geometry shader input variable uses location that "
968 "exceeds component limit VkPhysicalDeviceLimits::maxGeometryInputComponents (%u)",
969 limits.maxGeometryInputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600970 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700971 if (num_comp_out > limits.maxGeometryOutputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700972 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700973 "Invalid Pipeline CreateInfo State: Geometry shader exceeds "
974 "VkPhysicalDeviceLimits::maxGeometryOutputComponents of %u "
975 "components by %u components",
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700976 limits.maxGeometryOutputComponents, num_comp_out - limits.maxGeometryOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200977 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700978 if (max_comp_out > static_cast<int>(limits.maxGeometryOutputComponents)) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700979 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700980 "Invalid Pipeline CreateInfo State: Geometry shader output variable uses location that "
981 "exceeds component limit VkPhysicalDeviceLimits::maxGeometryOutputComponents (%u)",
982 limits.maxGeometryOutputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -0600983 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700984 if (num_comp_out * num_vertices > limits.maxGeometryTotalOutputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700985 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700986 "Invalid Pipeline CreateInfo State: Geometry shader exceeds "
987 "VkPhysicalDeviceLimits::maxGeometryTotalOutputComponents of %u "
988 "components by %u components",
989 limits.maxGeometryTotalOutputComponents,
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700990 num_comp_out * num_vertices - limits.maxGeometryTotalOutputComponents);
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500991 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200992 break;
993
994 case VK_SHADER_STAGE_FRAGMENT_BIT:
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -0700995 if (num_comp_in > limits.maxFragmentInputComponents) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -0700996 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -0700997 "Invalid Pipeline CreateInfo State: Fragment shader exceeds "
998 "VkPhysicalDeviceLimits::maxFragmentInputComponents of %u "
999 "components by %u components",
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001000 limits.maxFragmentInputComponents, num_comp_in - limits.maxFragmentInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001001 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001002 if (max_comp_in > static_cast<int>(limits.maxFragmentInputComponents)) {
sfricke-samsung782b9cb2021-09-16 23:53:32 -07001003 skip |= LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-Location-06272",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001004 "Invalid Pipeline CreateInfo State: Fragment shader input variable uses location that "
1005 "exceeds component limit VkPhysicalDeviceLimits::maxFragmentInputComponents (%u)",
1006 limits.maxFragmentInputComponents);
Jeff Bolzf234bf82019-11-04 14:07:15 -06001007 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001008 break;
1009
Jeff Bolz148d94e2018-12-13 21:25:56 -06001010 case VK_SHADER_STAGE_RAYGEN_BIT_NV:
1011 case VK_SHADER_STAGE_ANY_HIT_BIT_NV:
1012 case VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV:
1013 case VK_SHADER_STAGE_MISS_BIT_NV:
1014 case VK_SHADER_STAGE_INTERSECTION_BIT_NV:
1015 case VK_SHADER_STAGE_CALLABLE_BIT_NV:
1016 case VK_SHADER_STAGE_TASK_BIT_NV:
1017 case VK_SHADER_STAGE_MESH_BIT_NV:
1018 break;
1019
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001020 default:
1021 assert(false); // This should never happen
1022 }
1023 return skip;
1024}
1025
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001026bool CoreChecks::ValidateShaderStorageImageFormats(SHADER_MODULE_STATE const *src) const {
1027 bool skip = false;
1028
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001029 // Got through all ImageRead/Write instructions
1030 for (auto insn : *src) {
1031 switch (insn.opcode()) {
1032 case spv::OpImageSparseRead:
1033 case spv::OpImageRead: {
Lionel Landwerlin5f2065a2021-07-23 11:51:28 +03001034 spirv_inst_iter type_def = src->GetImageFormatInst(insn.word(3));
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001035 if (type_def != src->end()) {
Tim Van Pattenffe91322021-07-26 10:20:50 -06001036 const auto dim = type_def.word(3);
1037 // If the Image Dim operand is not SubpassData, the Image Format must not be Unknown, unless the
1038 // StorageImageReadWithoutFormat Capability was declared.
1039 if (dim != spv::DimSubpassData && type_def.word(8) == spv::ImageFormatUnknown) {
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001040 skip |= RequireFeature(enabled_features.core.shaderStorageImageReadWithoutFormat,
1041 "shaderStorageImageReadWithoutFormat",
1042 kVUID_Features_shaderStorageImageReadWithoutFormat);
1043 }
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001044 }
1045 break;
1046 }
1047 case spv::OpImageWrite: {
Lionel Landwerlin5f2065a2021-07-23 11:51:28 +03001048 spirv_inst_iter type_def = src->GetImageFormatInst(insn.word(1));
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001049 if (type_def != src->end()) {
1050 if (type_def.word(8) == spv::ImageFormatUnknown) {
1051 skip |= RequireFeature(enabled_features.core.shaderStorageImageWriteWithoutFormat,
1052 "shaderStorageImageWriteWithoutFormat",
1053 kVUID_Features_shaderStorageImageWriteWithoutFormat);
1054 }
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001055 }
1056 break;
1057 }
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001058 }
1059 }
1060
Lionel Landwerlin38d2e122021-07-21 14:21:47 +03001061 // Go through all variables for images and check decorations
1062 for (auto insn : *src) {
1063 if (insn.opcode() != spv::OpVariable)
1064 continue;
1065
1066 uint32_t var = insn.word(2);
Lionel Landwerlin5f2065a2021-07-23 11:51:28 +03001067 spirv_inst_iter type_def = src->GetImageFormatInst(insn.word(1));
Lionel Landwerlin38d2e122021-07-21 14:21:47 +03001068 if (type_def == src->end())
1069 continue;
Tim Van Pattenffe91322021-07-26 10:20:50 -06001070 // Only check if the Image Dim operand is not SubpassData
1071 const auto dim = type_def.word(3);
1072 if (dim == spv::DimSubpassData) continue;
Corentin Wallez91f8b6d2021-07-23 10:11:31 +02001073 // Only check storage images
1074 if (type_def.word(7) != 2) continue;
Tim Van Pattenffe91322021-07-26 10:20:50 -06001075 if (type_def.word(8) != spv::ImageFormatUnknown) continue;
Lionel Landwerlin38d2e122021-07-21 14:21:47 +03001076
1077 decoration_set img_decorations = src->get_decorations(var);
1078
1079 if (!enabled_features.core.shaderStorageImageReadWithoutFormat &&
1080 !(img_decorations.flags & decoration_set::nonreadable_bit)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001081 skip |= LogError(device, "VUID-RuntimeSpirv-OpTypeImage-06270",
1082 "shaderStorageImageReadWithoutFormat not supported but variable %" PRIu32
1083 " "
1084 " without format not marked a NonReadable",
1085 var);
Lionel Landwerlin38d2e122021-07-21 14:21:47 +03001086 }
1087
1088 if (!enabled_features.core.shaderStorageImageWriteWithoutFormat &&
1089 !(img_decorations.flags & decoration_set::nonwritable_bit)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001090 skip |= LogError(device, "VUID-RuntimeSpirv-OpTypeImage-06269",
1091 "shaderStorageImageWriteWithoutFormat not supported but variable %" PRIu32
1092 " "
1093 "without format not marked a NonWritable",
1094 var);
Lionel Landwerlin38d2e122021-07-21 14:21:47 +03001095 }
1096 }
1097
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03001098 return skip;
1099}
1100
sfricke-samsungdc96f302020-03-18 20:42:10 -07001101bool CoreChecks::ValidateShaderStageMaxResources(VkShaderStageFlagBits stage, const PIPELINE_STATE *pipeline) const {
1102 bool skip = false;
1103 uint32_t total_resources = 0;
1104
1105 // Only currently testing for graphics and compute pipelines
1106 // TODO: Add check and support for Ray Tracing pipeline VUID 03428
1107 if ((stage & (VK_SHADER_STAGE_ALL_GRAPHICS | VK_SHADER_STAGE_COMPUTE_BIT)) == 0) {
1108 return false;
1109 }
1110
1111 if (stage == VK_SHADER_STAGE_FRAGMENT_BIT) {
amhagana448ea52021-11-02 14:09:14 -04001112 if (pipeline->rp_state->use_dynamic_rendering) {
Aaron Hagan92a44f82021-11-19 09:34:56 -05001113 total_resources += pipeline->rp_state->dynamic_rendering_pipeline_create_info.colorAttachmentCount;
amhagana448ea52021-11-02 14:09:14 -04001114 } else {
1115 // "For the fragment shader stage the framebuffer color attachments also count against this limit"
1116 total_resources +=
1117 pipeline->rp_state->createInfo.pSubpasses[pipeline->create_info.graphics.subpass].colorAttachmentCount;
1118 }
sfricke-samsungdc96f302020-03-18 20:42:10 -07001119 }
1120
1121 // TODO: This reuses a lot of GetDescriptorCountMaxPerStage but currently would need to make it agnostic in a way to handle
1122 // input from CreatePipeline and CreatePipelineLayout level
1123 for (auto set_layout : pipeline->pipeline_layout->set_layouts) {
1124 if ((set_layout->GetCreateFlags() & VK_DESCRIPTOR_SET_LAYOUT_CREATE_UPDATE_AFTER_BIND_POOL_BIT) != 0) {
1125 continue;
1126 }
1127
1128 for (uint32_t binding_idx = 0; binding_idx < set_layout->GetBindingCount(); binding_idx++) {
1129 const VkDescriptorSetLayoutBinding *binding = set_layout->GetDescriptorSetLayoutBindingPtrFromIndex(binding_idx);
1130 // Bindings with a descriptorCount of 0 are "reserved" and should be skipped
1131 if (((stage & binding->stageFlags) != 0) && (binding->descriptorCount > 0)) {
1132 // Check only descriptor types listed in maxPerStageResources description in spec
1133 switch (binding->descriptorType) {
1134 case VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER:
1135 case VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE:
1136 case VK_DESCRIPTOR_TYPE_STORAGE_IMAGE:
1137 case VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER:
1138 case VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER:
1139 case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER:
1140 case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER:
1141 case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC:
1142 case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC:
1143 case VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT:
1144 total_resources += binding->descriptorCount;
1145 break;
1146 default:
1147 break;
1148 }
1149 }
1150 }
1151 }
1152
1153 if (total_resources > phys_dev_props.limits.maxPerStageResources) {
1154 const char *vuid = (stage == VK_SHADER_STAGE_COMPUTE_BIT) ? "VUID-VkComputePipelineCreateInfo-layout-01687"
1155 : "VUID-VkGraphicsPipelineCreateInfo-layout-01688";
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06001156 skip |= LogError(pipeline->pipeline(), vuid,
sfricke-samsungdc96f302020-03-18 20:42:10 -07001157 "Invalid Pipeline CreateInfo State: Shader Stage %s exceeds component limit "
1158 "VkPhysicalDeviceLimits::maxPerStageResources (%u)",
1159 string_VkShaderStageFlagBits(stage), phys_dev_props.limits.maxPerStageResources);
1160 }
1161
1162 return skip;
1163}
1164
Jeff Bolze4356752019-03-07 11:23:46 -06001165// copy the specialization constant value into buf, if it is present
1166void GetSpecConstantValue(VkPipelineShaderStageCreateInfo const *pStage, uint32_t spec_id, void *buf) {
1167 VkSpecializationInfo const *spec = pStage->pSpecializationInfo;
1168
1169 if (spec && spec_id < spec->mapEntryCount) {
1170 memcpy(buf, (uint8_t *)spec->pData + spec->pMapEntries[spec_id].offset, spec->pMapEntries[spec_id].size);
1171 }
1172}
1173
1174// Fill in value with the constant or specialization constant value, if available.
1175// Returns true if the value has been accurately filled out.
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001176static bool GetIntConstantValue(spirv_inst_iter insn, SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
Jeremy Gebbencbf22862021-03-03 12:01:22 -07001177 const layer_data::unordered_map<uint32_t, uint32_t> &id_to_spec_id, uint32_t *value) {
Jeff Bolze4356752019-03-07 11:23:46 -06001178 auto type_id = src->get_def(insn.word(1));
1179 if (type_id.opcode() != spv::OpTypeInt || type_id.word(2) != 32) {
1180 return false;
1181 }
1182 switch (insn.opcode()) {
1183 case spv::OpSpecConstant:
1184 *value = insn.word(3);
1185 GetSpecConstantValue(pStage, id_to_spec_id.at(insn.word(2)), value);
1186 return true;
1187 case spv::OpConstant:
1188 *value = insn.word(3);
1189 return true;
1190 default:
1191 return false;
1192 }
1193}
1194
1195// Map SPIR-V type to VK_COMPONENT_TYPE enum
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001196VkComponentTypeNV GetComponentType(spirv_inst_iter insn, SHADER_MODULE_STATE const *src) {
Jeff Bolze4356752019-03-07 11:23:46 -06001197 switch (insn.opcode()) {
1198 case spv::OpTypeInt:
1199 switch (insn.word(2)) {
1200 case 8:
1201 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT8_NV : VK_COMPONENT_TYPE_UINT8_NV;
1202 case 16:
1203 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT16_NV : VK_COMPONENT_TYPE_UINT16_NV;
1204 case 32:
1205 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT32_NV : VK_COMPONENT_TYPE_UINT32_NV;
1206 case 64:
1207 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT64_NV : VK_COMPONENT_TYPE_UINT64_NV;
1208 default:
1209 return VK_COMPONENT_TYPE_MAX_ENUM_NV;
1210 }
1211 case spv::OpTypeFloat:
1212 switch (insn.word(2)) {
1213 case 16:
1214 return VK_COMPONENT_TYPE_FLOAT16_NV;
1215 case 32:
1216 return VK_COMPONENT_TYPE_FLOAT32_NV;
1217 case 64:
1218 return VK_COMPONENT_TYPE_FLOAT64_NV;
1219 default:
1220 return VK_COMPONENT_TYPE_MAX_ENUM_NV;
1221 }
1222 default:
1223 return VK_COMPONENT_TYPE_MAX_ENUM_NV;
1224 }
1225}
1226
1227// Validate SPV_NV_cooperative_matrix behavior that can't be statically validated
1228// in SPIRV-Tools (e.g. due to specialization constant usage).
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001229bool CoreChecks::ValidateCooperativeMatrix(SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
John Zulaufac4c6e12019-07-01 16:05:58 -06001230 const PIPELINE_STATE *pipeline) const {
Jeff Bolze4356752019-03-07 11:23:46 -06001231 bool skip = false;
1232
1233 // Map SPIR-V result ID to specialization constant id (SpecId decoration value)
Jeremy Gebbencbf22862021-03-03 12:01:22 -07001234 layer_data::unordered_map<uint32_t, uint32_t> id_to_spec_id;
Jeff Bolze4356752019-03-07 11:23:46 -06001235 // Map SPIR-V result ID to the ID of its type.
Jeremy Gebbencbf22862021-03-03 12:01:22 -07001236 layer_data::unordered_map<uint32_t, uint32_t> id_to_type_id;
Jeff Bolze4356752019-03-07 11:23:46 -06001237
1238 struct CoopMatType {
1239 uint32_t scope, rows, cols;
1240 VkComponentTypeNV component_type;
1241 bool all_constant;
1242
1243 CoopMatType() : scope(0), rows(0), cols(0), component_type(VK_COMPONENT_TYPE_MAX_ENUM_NV), all_constant(false) {}
1244
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001245 void Init(uint32_t id, SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
Jeremy Gebbencbf22862021-03-03 12:01:22 -07001246 const layer_data::unordered_map<uint32_t, uint32_t> &id_to_spec_id) {
Jeff Bolze4356752019-03-07 11:23:46 -06001247 spirv_inst_iter insn = src->get_def(id);
1248 uint32_t component_type_id = insn.word(2);
1249 uint32_t scope_id = insn.word(3);
1250 uint32_t rows_id = insn.word(4);
1251 uint32_t cols_id = insn.word(5);
1252 auto component_type_iter = src->get_def(component_type_id);
1253 auto scope_iter = src->get_def(scope_id);
1254 auto rows_iter = src->get_def(rows_id);
1255 auto cols_iter = src->get_def(cols_id);
1256
1257 all_constant = true;
1258 if (!GetIntConstantValue(scope_iter, src, pStage, id_to_spec_id, &scope)) {
1259 all_constant = false;
1260 }
1261 if (!GetIntConstantValue(rows_iter, src, pStage, id_to_spec_id, &rows)) {
1262 all_constant = false;
1263 }
1264 if (!GetIntConstantValue(cols_iter, src, pStage, id_to_spec_id, &cols)) {
1265 all_constant = false;
1266 }
1267 component_type = GetComponentType(component_type_iter, src);
1268 }
1269 };
1270
1271 bool seen_coopmat_capability = false;
1272
1273 for (auto insn : *src) {
1274 // Whitelist instructions whose result can be a cooperative matrix type, and
1275 // keep track of their types. It would be nice if SPIRV-Headers generated code
1276 // to identify which instructions have a result type and result id. Lacking that,
1277 // this whitelist is based on the set of instructions that
1278 // SPV_NV_cooperative_matrix says can be used with cooperative matrix types.
1279 switch (insn.opcode()) {
1280 case spv::OpLoad:
1281 case spv::OpCooperativeMatrixLoadNV:
1282 case spv::OpCooperativeMatrixMulAddNV:
1283 case spv::OpSNegate:
1284 case spv::OpFNegate:
1285 case spv::OpIAdd:
1286 case spv::OpFAdd:
1287 case spv::OpISub:
1288 case spv::OpFSub:
1289 case spv::OpFDiv:
1290 case spv::OpSDiv:
1291 case spv::OpUDiv:
1292 case spv::OpMatrixTimesScalar:
1293 case spv::OpConstantComposite:
1294 case spv::OpCompositeConstruct:
1295 case spv::OpConvertFToU:
1296 case spv::OpConvertFToS:
1297 case spv::OpConvertSToF:
1298 case spv::OpConvertUToF:
1299 case spv::OpUConvert:
1300 case spv::OpSConvert:
1301 case spv::OpFConvert:
1302 id_to_type_id[insn.word(2)] = insn.word(1);
1303 break;
1304 default:
1305 break;
1306 }
1307
1308 switch (insn.opcode()) {
1309 case spv::OpDecorate:
1310 if (insn.word(2) == spv::DecorationSpecId) {
1311 id_to_spec_id[insn.word(1)] = insn.word(3);
1312 }
1313 break;
1314 case spv::OpCapability:
1315 if (insn.word(1) == spv::CapabilityCooperativeMatrixNV) {
1316 seen_coopmat_capability = true;
1317
1318 if (!(pStage->stage & phys_dev_ext_props.cooperative_matrix_props.cooperativeMatrixSupportedStages)) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001319 skip |= LogError(
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001320 pipeline->pipeline(), "VUID-RuntimeSpirv-OpTypeCooperativeMatrixNV-06322",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001321 "OpTypeCooperativeMatrixNV used in shader stage not in cooperativeMatrixSupportedStages (= %u)",
1322 phys_dev_ext_props.cooperative_matrix_props.cooperativeMatrixSupportedStages);
Jeff Bolze4356752019-03-07 11:23:46 -06001323 }
1324 }
1325 break;
1326 case spv::OpMemoryModel:
1327 // If the capability isn't enabled, don't bother with the rest of this function.
1328 // OpMemoryModel is the first required instruction after all OpCapability instructions.
1329 if (!seen_coopmat_capability) {
1330 return skip;
1331 }
1332 break;
1333 case spv::OpTypeCooperativeMatrixNV: {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001334 CoopMatType m;
1335 m.Init(insn.word(1), src, pStage, id_to_spec_id);
Jeff Bolze4356752019-03-07 11:23:46 -06001336
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001337 if (m.all_constant) {
Jeff Bolze4356752019-03-07 11:23:46 -06001338 // Validate that the type parameters are all supported for one of the
1339 // operands of a cooperative matrix property.
1340 bool valid = false;
1341 for (unsigned i = 0; i < cooperative_matrix_properties.size(); ++i) {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001342 if (cooperative_matrix_properties[i].AType == m.component_type &&
1343 cooperative_matrix_properties[i].MSize == m.rows && cooperative_matrix_properties[i].KSize == m.cols &&
1344 cooperative_matrix_properties[i].scope == m.scope) {
Jeff Bolze4356752019-03-07 11:23:46 -06001345 valid = true;
1346 break;
1347 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001348 if (cooperative_matrix_properties[i].BType == m.component_type &&
1349 cooperative_matrix_properties[i].KSize == m.rows && cooperative_matrix_properties[i].NSize == m.cols &&
1350 cooperative_matrix_properties[i].scope == m.scope) {
Jeff Bolze4356752019-03-07 11:23:46 -06001351 valid = true;
1352 break;
1353 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001354 if (cooperative_matrix_properties[i].CType == m.component_type &&
1355 cooperative_matrix_properties[i].MSize == m.rows && cooperative_matrix_properties[i].NSize == m.cols &&
1356 cooperative_matrix_properties[i].scope == m.scope) {
Jeff Bolze4356752019-03-07 11:23:46 -06001357 valid = true;
1358 break;
1359 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001360 if (cooperative_matrix_properties[i].DType == m.component_type &&
1361 cooperative_matrix_properties[i].MSize == m.rows && cooperative_matrix_properties[i].NSize == m.cols &&
1362 cooperative_matrix_properties[i].scope == m.scope) {
Jeff Bolze4356752019-03-07 11:23:46 -06001363 valid = true;
1364 break;
1365 }
1366 }
1367 if (!valid) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06001368 skip |= LogError(pipeline->pipeline(), kVUID_Core_Shader_CooperativeMatrixType,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001369 "OpTypeCooperativeMatrixNV (result id = %u) operands don't match a supported matrix type",
1370 insn.word(1));
Jeff Bolze4356752019-03-07 11:23:46 -06001371 }
1372 }
1373 break;
1374 }
1375 case spv::OpCooperativeMatrixMulAddNV: {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001376 CoopMatType a, b, c, d;
Jeff Bolze4356752019-03-07 11:23:46 -06001377 if (id_to_type_id.find(insn.word(2)) == id_to_type_id.end() ||
1378 id_to_type_id.find(insn.word(3)) == id_to_type_id.end() ||
1379 id_to_type_id.find(insn.word(4)) == id_to_type_id.end() ||
1380 id_to_type_id.find(insn.word(5)) == id_to_type_id.end()) {
Mike Schuchardte48dc142019-04-18 09:12:03 -07001381 // Couldn't find type of matrix
1382 assert(false);
Jeff Bolze4356752019-03-07 11:23:46 -06001383 break;
1384 }
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001385 d.Init(id_to_type_id[insn.word(2)], src, pStage, id_to_spec_id);
1386 a.Init(id_to_type_id[insn.word(3)], src, pStage, id_to_spec_id);
1387 b.Init(id_to_type_id[insn.word(4)], src, pStage, id_to_spec_id);
1388 c.Init(id_to_type_id[insn.word(5)], src, pStage, id_to_spec_id);
Jeff Bolze4356752019-03-07 11:23:46 -06001389
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001390 if (a.all_constant && b.all_constant && c.all_constant && d.all_constant) {
Jeff Bolze4356752019-03-07 11:23:46 -06001391 // Validate that the type parameters are all supported for the same
1392 // cooperative matrix property.
1393 bool valid = false;
1394 for (unsigned i = 0; i < cooperative_matrix_properties.size(); ++i) {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001395 if (cooperative_matrix_properties[i].AType == a.component_type &&
1396 cooperative_matrix_properties[i].MSize == a.rows && cooperative_matrix_properties[i].KSize == a.cols &&
1397 cooperative_matrix_properties[i].scope == a.scope &&
Jeff Bolze4356752019-03-07 11:23:46 -06001398
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001399 cooperative_matrix_properties[i].BType == b.component_type &&
1400 cooperative_matrix_properties[i].KSize == b.rows && cooperative_matrix_properties[i].NSize == b.cols &&
1401 cooperative_matrix_properties[i].scope == b.scope &&
Jeff Bolze4356752019-03-07 11:23:46 -06001402
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001403 cooperative_matrix_properties[i].CType == c.component_type &&
1404 cooperative_matrix_properties[i].MSize == c.rows && cooperative_matrix_properties[i].NSize == c.cols &&
1405 cooperative_matrix_properties[i].scope == c.scope &&
Jeff Bolze4356752019-03-07 11:23:46 -06001406
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001407 cooperative_matrix_properties[i].DType == d.component_type &&
1408 cooperative_matrix_properties[i].MSize == d.rows && cooperative_matrix_properties[i].NSize == d.cols &&
1409 cooperative_matrix_properties[i].scope == d.scope) {
Jeff Bolze4356752019-03-07 11:23:46 -06001410 valid = true;
1411 break;
1412 }
1413 }
1414 if (!valid) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06001415 skip |= LogError(pipeline->pipeline(), kVUID_Core_Shader_CooperativeMatrixMulAdd,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001416 "OpCooperativeMatrixMulAddNV (result id = %u) operands don't match a supported matrix "
1417 "VkCooperativeMatrixPropertiesNV",
1418 insn.word(2));
Jeff Bolze4356752019-03-07 11:23:46 -06001419 }
1420 }
1421 break;
1422 }
1423 default:
1424 break;
1425 }
1426 }
1427
1428 return skip;
1429}
1430
Jeff Leger9b3dcff2021-05-27 15:40:20 -04001431bool CoreChecks::ValidateShaderResolveQCOM(SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
1432 const PIPELINE_STATE *pipeline) const {
1433 bool skip = false;
1434
1435 // If the pipeline's subpass description contains flag VK_SUBPASS_DESCRIPTION_FRAGMENT_REGION_BIT_QCOM,
1436 // then the fragment shader must not enable the SPIRV SampleRateShading capability.
1437 if (pStage->stage == VK_SHADER_STAGE_FRAGMENT_BIT) {
1438 for (auto insn : *src) {
1439 switch (insn.opcode()) {
1440 case spv::OpCapability:
1441 if (insn.word(1) == spv::CapabilitySampleRateShading) {
1442 auto subpass_flags =
1443 (pipeline->rp_state == nullptr)
1444 ? 0
Jeremy Gebben11af9792021-08-20 10:20:09 -06001445 : pipeline->rp_state->createInfo.pSubpasses[pipeline->create_info.graphics.subpass].flags;
Jeff Leger9b3dcff2021-05-27 15:40:20 -04001446 if ((subpass_flags & VK_SUBPASS_DESCRIPTION_FRAGMENT_REGION_BIT_QCOM) != 0) {
1447 skip |=
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001448 LogError(pipeline->pipeline(), "VUID-RuntimeSpirv-SampleRateShading-06378",
Jeff Leger9b3dcff2021-05-27 15:40:20 -04001449 "Invalid Pipeline CreateInfo State: fragment shader enables SampleRateShading capability "
1450 "and the subpass flags includes VK_SUBPASS_DESCRIPTION_FRAGMENT_REGION_BIT_QCOM.");
1451 }
1452 }
1453 break;
1454 default:
1455 break;
1456 }
1457 }
1458 }
1459
1460 return skip;
1461}
1462
ziga-lunarg73163742021-08-25 13:15:29 +02001463bool CoreChecks::ValidateShaderSubgroupSizeControl(VkPipelineShaderStageCreateInfo const *pStage) const {
1464 bool skip = false;
1465
1466 if ((pStage->flags & VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT) != 0 &&
1467 !enabled_features.subgroup_size_control_features.subgroupSizeControl) {
1468 skip |= LogError(
1469 device, "VUID-VkPipelineShaderStageCreateInfo-flags-02784",
1470 "VkPipelineShaderStageCreateInfo flags contain VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT, "
1471 "but the VkPhysicalDeviceSubgroupSizeControlFeaturesEXT::subgroupSizeControl feature is not enabled.");
1472 }
1473
1474 if ((pStage->flags & VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT) != 0 &&
1475 !enabled_features.subgroup_size_control_features.computeFullSubgroups) {
1476 skip |= LogError(
1477 device, "VUID-VkPipelineShaderStageCreateInfo-flags-02785",
1478 "VkPipelineShaderStageCreateInfo flags contain VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT, but the "
1479 "VkPhysicalDeviceSubgroupSizeControlFeaturesEXT::computeFullSubgroups feature is not enabled");
1480 }
1481
1482 return skip;
1483}
1484
sfricke-samsung58b84352021-07-31 21:41:04 -07001485bool CoreChecks::ValidateAtomicsTypes(SHADER_MODULE_STATE const *src) const {
1486 bool skip = false;
1487
sfricke-samsung6c9eb712021-08-04 09:38:54 -07001488 // "If sparseImageInt64Atomics is enabled, shaderImageInt64Atomics must be enabled"
sfricke-samsung828e59d2021-08-22 23:20:49 -07001489 const bool valid_image_64_int = enabled_features.shader_image_atomic_int64_features.shaderImageInt64Atomics == VK_TRUE;
sfricke-samsung6c9eb712021-08-04 09:38:54 -07001490
sfricke-samsungf5042b12021-08-05 01:09:40 -07001491 const VkPhysicalDeviceShaderAtomicFloatFeaturesEXT &float_features = enabled_features.shader_atomic_float_features;
1492 const VkPhysicalDeviceShaderAtomicFloat2FeaturesEXT &float2_features = enabled_features.shader_atomic_float2_features;
1493
1494 const bool valid_storage_buffer_float = (
1495 (float_features.shaderBufferFloat32Atomics == VK_TRUE) ||
1496 (float_features.shaderBufferFloat32AtomicAdd == VK_TRUE) ||
1497 (float_features.shaderBufferFloat64Atomics == VK_TRUE) ||
1498 (float_features.shaderBufferFloat64AtomicAdd == VK_TRUE) ||
1499 (float2_features.shaderBufferFloat16Atomics == VK_TRUE) ||
1500 (float2_features.shaderBufferFloat16AtomicAdd == VK_TRUE) ||
1501 (float2_features.shaderBufferFloat16AtomicMinMax == VK_TRUE) ||
1502 (float2_features.shaderBufferFloat32AtomicMinMax == VK_TRUE) ||
1503 (float2_features.shaderBufferFloat64AtomicMinMax == VK_TRUE));
1504
1505 const bool valid_workgroup_float = (
1506 (float_features.shaderSharedFloat32Atomics == VK_TRUE) ||
1507 (float_features.shaderSharedFloat32AtomicAdd == VK_TRUE) ||
1508 (float_features.shaderSharedFloat64Atomics == VK_TRUE) ||
1509 (float_features.shaderSharedFloat64AtomicAdd == VK_TRUE) ||
1510 (float2_features.shaderSharedFloat16Atomics == VK_TRUE) ||
1511 (float2_features.shaderSharedFloat16AtomicAdd == VK_TRUE) ||
1512 (float2_features.shaderSharedFloat16AtomicMinMax == VK_TRUE) ||
1513 (float2_features.shaderSharedFloat32AtomicMinMax == VK_TRUE) ||
1514 (float2_features.shaderSharedFloat64AtomicMinMax == VK_TRUE));
1515
1516 const bool valid_image_float = (
1517 (float_features.shaderImageFloat32Atomics == VK_TRUE) ||
1518 (float_features.shaderImageFloat32AtomicAdd == VK_TRUE) ||
1519 (float2_features.shaderImageFloat32AtomicMinMax == VK_TRUE));
1520
1521 const bool valid_16_float = (
1522 (float2_features.shaderBufferFloat16Atomics == VK_TRUE) ||
1523 (float2_features.shaderBufferFloat16AtomicAdd == VK_TRUE) ||
1524 (float2_features.shaderBufferFloat16AtomicMinMax == VK_TRUE) ||
1525 (float2_features.shaderSharedFloat16Atomics == VK_TRUE) ||
1526 (float2_features.shaderSharedFloat16AtomicAdd == VK_TRUE) ||
1527 (float2_features.shaderSharedFloat16AtomicMinMax == VK_TRUE));
1528
1529 const bool valid_32_float = (
1530 (float_features.shaderBufferFloat32Atomics == VK_TRUE) ||
1531 (float_features.shaderBufferFloat32AtomicAdd == VK_TRUE) ||
1532 (float_features.shaderSharedFloat32Atomics == VK_TRUE) ||
1533 (float_features.shaderSharedFloat32AtomicAdd == VK_TRUE) ||
1534 (float_features.shaderImageFloat32Atomics == VK_TRUE) ||
1535 (float_features.shaderImageFloat32AtomicAdd == VK_TRUE) ||
1536 (float2_features.shaderBufferFloat32AtomicMinMax == VK_TRUE) ||
1537 (float2_features.shaderSharedFloat32AtomicMinMax == VK_TRUE) ||
1538 (float2_features.shaderImageFloat32AtomicMinMax == VK_TRUE));
1539
1540 const bool valid_64_float = (
1541 (float_features.shaderBufferFloat64Atomics == VK_TRUE) ||
1542 (float_features.shaderBufferFloat64AtomicAdd == VK_TRUE) ||
1543 (float_features.shaderSharedFloat64Atomics == VK_TRUE) ||
1544 (float_features.shaderSharedFloat64AtomicAdd == VK_TRUE) ||
1545 (float2_features.shaderBufferFloat64AtomicMinMax == VK_TRUE) ||
1546 (float2_features.shaderSharedFloat64AtomicMinMax == VK_TRUE));
1547 // clang-format on
1548
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06001549 for (const auto &atomic_inst : src->GetAtomicInstructions()) {
sfricke-samsung58b84352021-07-31 21:41:04 -07001550 const atomic_instruction &atomic = atomic_inst.second;
sfricke-samsungf5042b12021-08-05 01:09:40 -07001551 const uint32_t opcode = src->at(atomic_inst.first).opcode();
sfricke-samsung58b84352021-07-31 21:41:04 -07001552
1553 if ((atomic.bit_width == 64) && (atomic.type == spv::OpTypeInt)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001554 // Validate 64-bit image atomics
sfricke-samsung58b84352021-07-31 21:41:04 -07001555 if (((atomic.storage_class == spv::StorageClassStorageBuffer) || (atomic.storage_class == spv::StorageClassUniform)) &&
1556 (enabled_features.core12.shaderBufferInt64Atomics == VK_FALSE)) {
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001557 skip |= LogError(device, "VUID-RuntimeSpirv-None-06278",
1558 "%s: Can't use 64-bit int atomics operations (%s) with %s storage class without "
1559 "shaderBufferInt64Atomics enabled.",
1560 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode),
1561 StorageClassName(atomic.storage_class));
sfricke-samsung58b84352021-07-31 21:41:04 -07001562 } else if ((atomic.storage_class == spv::StorageClassWorkgroup) &&
1563 (enabled_features.core12.shaderSharedInt64Atomics == VK_FALSE)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001564 skip |= LogError(device, "VUID-RuntimeSpirv-None-06279",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001565 "%s: Can't use 64-bit int atomics operations (%s) with Workgroup storage class without "
sfricke-samsung58b84352021-07-31 21:41:04 -07001566 "shaderSharedInt64Atomics enabled.",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001567 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsung6c9eb712021-08-04 09:38:54 -07001568 } else if ((atomic.storage_class == spv::StorageClassImage) && (valid_image_64_int == false)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001569 skip |= LogError(device, "VUID-RuntimeSpirv-None-06288",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001570 "%s: Can't use 64-bit int atomics operations (%s) with Image storage class without "
sfricke-samsung6c9eb712021-08-04 09:38:54 -07001571 "shaderImageInt64Atomics enabled.",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001572 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsung58b84352021-07-31 21:41:04 -07001573 }
sfricke-samsungf5042b12021-08-05 01:09:40 -07001574 } else if (atomic.type == spv::OpTypeFloat) {
1575 // Validate Floats
1576 if (atomic.storage_class == spv::StorageClassStorageBuffer) {
1577 if (valid_storage_buffer_float == false) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001578 const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06284"
1579 : "VUID-RuntimeSpirv-None-06280";
1580 skip |= LogError(device, vuid,
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001581 "%s: Can't use float atomics operations (%s) with StorageBuffer storage class without "
sfricke-samsungf5042b12021-08-05 01:09:40 -07001582 "shaderBufferFloat32Atomics or shaderBufferFloat32AtomicAdd or shaderBufferFloat64Atomics or "
1583 "shaderBufferFloat64AtomicAdd or shaderBufferFloat16Atomics or shaderBufferFloat16AtomicAdd "
1584 "or shaderBufferFloat16AtomicMinMax or shaderBufferFloat32AtomicMinMax or "
1585 "shaderBufferFloat64AtomicMinMax enabled.",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001586 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsungf5042b12021-08-05 01:09:40 -07001587 } else if (opcode == spv::OpAtomicFAddEXT) {
1588 if ((atomic.bit_width == 16) && (float2_features.shaderBufferFloat16AtomicAdd == VK_FALSE)) {
1589 skip |= LogError(device, kVUID_Core_Shader_AtomicFeature,
1590 "%s: Can't use 16-bit float atomics for add operations (OpAtomicFAddEXT) with "
1591 "StorageBuffer storage class without shaderBufferFloat16AtomicAdd enabled.",
1592 report_data->FormatHandle(src->vk_shader_module()).c_str());
1593 } else if ((atomic.bit_width == 32) && (float_features.shaderBufferFloat32AtomicAdd == VK_FALSE)) {
1594 skip |= LogError(device, kVUID_Core_Shader_AtomicFeature,
1595 "%s: Can't use 32-bit float atomics for add operations (OpAtomicFAddEXT) with "
1596 "StorageBuffer storage class without shaderBufferFloat32AtomicAdd enabled.",
1597 report_data->FormatHandle(src->vk_shader_module()).c_str());
1598 } else if ((atomic.bit_width == 64) && (float_features.shaderBufferFloat64AtomicAdd == VK_FALSE)) {
1599 skip |= LogError(device, kVUID_Core_Shader_AtomicFeature,
1600 "%s: Can't use 64-bit float atomics for add operations (OpAtomicFAddEXT) with "
1601 "StorageBuffer storage class without shaderBufferFloat64AtomicAdd enabled.",
1602 report_data->FormatHandle(src->vk_shader_module()).c_str());
1603 }
1604 } else if (opcode == spv::OpAtomicFMinEXT || opcode == spv::OpAtomicFMaxEXT) {
1605 if ((atomic.bit_width == 16) && (float2_features.shaderBufferFloat16AtomicMinMax == VK_FALSE)) {
1606 skip |= LogError(
1607 device, kVUID_Core_Shader_AtomicFeature,
1608 "%s: Can't use 16-bit float atomics for min/max operations (OpAtomicFMinEXT or OpAtomicFMaxEXT) with "
1609 "StorageBuffer storage class without shaderBufferFloat16AtomicMinMax enabled.",
1610 report_data->FormatHandle(src->vk_shader_module()).c_str());
1611 } else if ((atomic.bit_width == 32) && (float2_features.shaderBufferFloat32AtomicMinMax == VK_FALSE)) {
1612 skip |= LogError(
1613 device, kVUID_Core_Shader_AtomicFeature,
1614 "%s: Can't use 32-bit float atomics for min/max operations (OpAtomicFMinEXT or OpAtomicFMaxEXT) with "
1615 "StorageBuffer storage class without shaderBufferFloat32AtomicMinMax enabled.",
1616 report_data->FormatHandle(src->vk_shader_module()).c_str());
1617 } else if ((atomic.bit_width == 64) && (float2_features.shaderBufferFloat64AtomicMinMax == VK_FALSE)) {
1618 skip |= LogError(
1619 device, kVUID_Core_Shader_AtomicFeature,
1620 "%s: Can't use 64-bit float atomics for min/max operations (OpAtomicFMinEXT or OpAtomicFMaxEXT) with "
1621 "StorageBuffer storage class without shaderBufferFloat64AtomicMinMax enabled.",
1622 report_data->FormatHandle(src->vk_shader_module()).c_str());
1623 }
1624 } else {
1625 // Assume is valid load/store/exchange (rest of supported atomic operations) or else spirv-val will catch
1626 if ((atomic.bit_width == 16) && (float2_features.shaderBufferFloat16Atomics == VK_FALSE)) {
1627 skip |= LogError(
1628 device, kVUID_Core_Shader_AtomicFeature,
1629 "%s: Can't use 16-bit float atomics for load/store/exhange operations (OpAtomicLoad, OpAtomicStore, "
1630 "OpAtomicExchange) with StorageBuffer storage class without shaderBufferFloat16Atomics enabled.",
1631 report_data->FormatHandle(src->vk_shader_module()).c_str());
1632 } else if ((atomic.bit_width == 32) && (float_features.shaderBufferFloat32Atomics == VK_FALSE)) {
1633 skip |= LogError(
1634 device, kVUID_Core_Shader_AtomicFeature,
1635 "%s: Can't use 32-bit float atomics for load/store/exhange operations (OpAtomicLoad, OpAtomicStore, "
1636 "OpAtomicExchange) with StorageBuffer storage class without shaderBufferFloat32Atomics enabled.",
1637 report_data->FormatHandle(src->vk_shader_module()).c_str());
1638 } else if ((atomic.bit_width == 64) && (float_features.shaderBufferFloat64Atomics == VK_FALSE)) {
1639 skip |= LogError(
1640 device, kVUID_Core_Shader_AtomicFeature,
1641 "%s: Can't use 64-bit float atomics for load/store/exhange operations (OpAtomicLoad, OpAtomicStore, "
1642 "OpAtomicExchange) with StorageBuffer storage class without shaderBufferFloat64Atomics enabled.",
1643 report_data->FormatHandle(src->vk_shader_module()).c_str());
1644 }
1645 }
1646 } else if (atomic.storage_class == spv::StorageClassWorkgroup) {
1647 if (valid_workgroup_float == false) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001648 const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06285"
1649 : "VUID-RuntimeSpirv-None-06281";
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001650 skip |=
1651 LogError(device, vuid,
1652 "%s: Can't use float atomics operations (%s) with Workgroup storage class without "
1653 "shaderSharedFloat32Atomics or "
1654 "shaderSharedFloat32AtomicAdd or shaderSharedFloat64Atomics or shaderSharedFloat64AtomicAdd or "
1655 "shaderSharedFloat16Atomics or shaderSharedFloat16AtomicAdd or shaderSharedFloat16AtomicMinMax or "
1656 "shaderSharedFloat32AtomicMinMax or shaderSharedFloat64AtomicMinMax enabled.",
1657 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsungf5042b12021-08-05 01:09:40 -07001658 } else if (opcode == spv::OpAtomicFAddEXT) {
1659 if ((atomic.bit_width == 16) && (float2_features.shaderSharedFloat16AtomicAdd == VK_FALSE)) {
1660 skip |= LogError(device, kVUID_Core_Shader_AtomicFeature,
1661 "%s: Can't use 16-bit float atomics for add operations (OpAtomicFAddEXT) with Workgroup "
1662 "storage class without shaderSharedFloat16AtomicAdd enabled.",
1663 report_data->FormatHandle(src->vk_shader_module()).c_str());
1664 } else if ((atomic.bit_width == 32) && (float_features.shaderSharedFloat32AtomicAdd == VK_FALSE)) {
1665 skip |= LogError(device, kVUID_Core_Shader_AtomicFeature,
1666 "%s: Can't use 32-bit float atomics for add operations (OpAtomicFAddEXT) with Workgroup "
1667 "storage class without shaderSharedFloat32AtomicAdd enabled.",
1668 report_data->FormatHandle(src->vk_shader_module()).c_str());
1669 } else if ((atomic.bit_width == 64) && (float_features.shaderSharedFloat64AtomicAdd == VK_FALSE)) {
1670 skip |= LogError(device, kVUID_Core_Shader_AtomicFeature,
1671 "%s: Can't use 64-bit float atomics for add operations (OpAtomicFAddEXT) with Workgroup "
1672 "storage class without shaderSharedFloat64AtomicAdd enabled.",
1673 report_data->FormatHandle(src->vk_shader_module()).c_str());
1674 }
1675 } else if (opcode == spv::OpAtomicFMinEXT || opcode == spv::OpAtomicFMaxEXT) {
1676 if ((atomic.bit_width == 16) && (float2_features.shaderSharedFloat16AtomicMinMax == VK_FALSE)) {
1677 skip |= LogError(
1678 device, kVUID_Core_Shader_AtomicFeature,
1679 "%s: Can't use 16-bit float atomics for min/max operations (OpAtomicFMinEXT or OpAtomicFMaxEXT) with "
1680 "Workgroup storage class without shaderSharedFloat16AtomicMinMax enabled.",
1681 report_data->FormatHandle(src->vk_shader_module()).c_str());
1682 } else if ((atomic.bit_width == 32) && (float2_features.shaderSharedFloat32AtomicMinMax == VK_FALSE)) {
1683 skip |= LogError(
1684 device, kVUID_Core_Shader_AtomicFeature,
1685 "%s: Can't use 32-bit float atomics for min/max operations (OpAtomicFMinEXT or OpAtomicFMaxEXT) with "
1686 "Workgroup storage class without shaderSharedFloat32AtomicMinMax enabled.",
1687 report_data->FormatHandle(src->vk_shader_module()).c_str());
1688 } else if ((atomic.bit_width == 64) && (float2_features.shaderSharedFloat64AtomicMinMax == VK_FALSE)) {
1689 skip |= LogError(
1690 device, kVUID_Core_Shader_AtomicFeature,
1691 "%s: Can't use 64-bit float atomics for min/max operations (OpAtomicFMinEXT or OpAtomicFMaxEXT) with "
1692 "Workgroup storage class without shaderSharedFloat64AtomicMinMax enabled.",
1693 report_data->FormatHandle(src->vk_shader_module()).c_str());
1694 }
1695 } else {
1696 // Assume is valid load/store/exchange (rest of supported atomic operations) or else spirv-val will catch
1697 if ((atomic.bit_width == 16) && (float2_features.shaderSharedFloat16Atomics == VK_FALSE)) {
1698 skip |= LogError(
1699 device, kVUID_Core_Shader_AtomicFeature,
1700 "%s: Can't use 16-bit float atomics for load/store/exhange operations (OpAtomicLoad, OpAtomicStore, "
1701 "OpAtomicExchange) with Workgroup storage class without shaderSharedFloat16Atomics enabled.",
1702 report_data->FormatHandle(src->vk_shader_module()).c_str());
1703 } else if ((atomic.bit_width == 32) && (float_features.shaderSharedFloat32Atomics == VK_FALSE)) {
1704 skip |= LogError(
1705 device, kVUID_Core_Shader_AtomicFeature,
1706 "%s: Can't use 32-bit float atomics for load/store/exhange operations (OpAtomicLoad, OpAtomicStore, "
1707 "OpAtomicExchange) with Workgroup storage class without shaderSharedFloat32Atomics enabled.",
1708 report_data->FormatHandle(src->vk_shader_module()).c_str());
1709 } else if ((atomic.bit_width == 64) && (float_features.shaderSharedFloat64Atomics == VK_FALSE)) {
1710 skip |= LogError(
1711 device, kVUID_Core_Shader_AtomicFeature,
1712 "%s: Can't use 64-bit float atomics for load/store/exhange operations (OpAtomicLoad, OpAtomicStore, "
1713 "OpAtomicExchange) with Workgroup storage class without shaderSharedFloat64Atomics enabled.",
1714 report_data->FormatHandle(src->vk_shader_module()).c_str());
1715 }
1716 }
1717 } else if ((atomic.storage_class == spv::StorageClassImage) && (valid_image_float == false)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001718 const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06286"
1719 : "VUID-RuntimeSpirv-None-06282";
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001720 skip |= LogError(
1721 device, vuid,
1722 "%s: Can't use float atomics operations (%s) with Image storage class without shaderImageFloat32Atomics or "
1723 "shaderImageFloat32AtomicAdd or shaderImageFloat32AtomicMinMax enabled.",
1724 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsungf5042b12021-08-05 01:09:40 -07001725 } else if ((atomic.bit_width == 16) && (valid_16_float == false)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001726 skip |= LogError(device, "VUID-RuntimeSpirv-None-06337",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001727 "%s: Can't use 16-bit float atomics operations (%s) without shaderBufferFloat16Atomics, "
sfricke-samsungf5042b12021-08-05 01:09:40 -07001728 "shaderBufferFloat16AtomicAdd, shaderBufferFloat16AtomicMinMax, shaderSharedFloat16Atomics, "
1729 "shaderSharedFloat16AtomicAdd or shaderSharedFloat16AtomicMinMax enabled.",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001730 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsungf5042b12021-08-05 01:09:40 -07001731 } else if ((atomic.bit_width == 32) && (valid_32_float == false)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001732 const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06338"
1733 : "VUID-RuntimeSpirv-None-06335";
1734 skip |= LogError(device, vuid,
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001735 "%s: Can't use 32-bit float atomics operations (%s) without shaderBufferFloat32AtomicMinMax, "
sfricke-samsungf5042b12021-08-05 01:09:40 -07001736 "shaderSharedFloat32AtomicMinMax, shaderImageFloat32AtomicMinMax, sparseImageFloat32AtomicMinMax, "
1737 "shaderBufferFloat32Atomics, shaderBufferFloat32AtomicAdd, shaderSharedFloat32Atomics, "
1738 "shaderSharedFloat32AtomicAdd, shaderImageFloat32Atomics, shaderImageFloat32AtomicAdd, "
1739 "sparseImageFloat32Atomics or sparseImageFloat32AtomicAdd enabled.",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001740 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsungf5042b12021-08-05 01:09:40 -07001741 } else if ((atomic.bit_width == 64) && (valid_64_float == false)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001742 const char *vuid = IsExtEnabled(device_extensions.vk_ext_shader_atomic_float2) ? "VUID-RuntimeSpirv-None-06339"
1743 : "VUID-RuntimeSpirv-None-06336";
1744 skip |= LogError(device, vuid,
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001745 "%s: Can't use 64-bit float atomics operations (%s) without shaderBufferFloat64AtomicMinMax, "
sfricke-samsungf5042b12021-08-05 01:09:40 -07001746 "shaderSharedFloat64AtomicMinMax, shaderBufferFloat64Atomics, shaderBufferFloat64AtomicAdd, "
1747 "shaderSharedFloat64Atomics or shaderSharedFloat64AtomicAdd enabled.",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07001748 report_data->FormatHandle(src->vk_shader_module()).c_str(), string_SpvOpcode(opcode));
sfricke-samsungf5042b12021-08-05 01:09:40 -07001749 }
sfricke-samsung58b84352021-07-31 21:41:04 -07001750 }
1751 }
sfricke-samsung58b84352021-07-31 21:41:04 -07001752 return skip;
1753}
1754
John Zulaufac4c6e12019-07-01 16:05:58 -06001755bool CoreChecks::ValidateExecutionModes(SHADER_MODULE_STATE const *src, spirv_inst_iter entrypoint) const {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001756 auto entrypoint_id = entrypoint.word(2);
1757
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01001758 // The first denorm execution mode encountered, along with its bit width.
1759 // Used to check if SeparateDenormSettings is respected.
1760 std::pair<spv::ExecutionMode, uint32_t> first_denorm_execution_mode = std::make_pair(spv::ExecutionModeMax, 0);
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001761
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01001762 // The first rounding mode encountered, along with its bit width.
1763 // Used to check if SeparateRoundingModeSettings is respected.
1764 std::pair<spv::ExecutionMode, uint32_t> first_rounding_mode = std::make_pair(spv::ExecutionModeMax, 0);
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001765
1766 bool skip = false;
1767
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001768 uint32_t vertices_out = 0;
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001769 uint32_t invocations = 0;
1770
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06001771 const auto &execution_mode_inst = src->GetExecutionModeInstructions();
1772 auto it = execution_mode_inst.find(entrypoint_id);
1773 if (it != execution_mode_inst.end()) {
sfricke-samsung8a7341a2021-02-28 07:30:21 -08001774 for (auto insn : it->second) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001775 auto mode = insn.word(2);
1776 switch (mode) {
1777 case spv::ExecutionModeSignedZeroInfNanPreserve: {
1778 auto bit_width = insn.word(3);
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001779 if (bit_width == 16 && !phys_dev_props_core12.shaderSignedZeroInfNanPreserveFloat16) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001780 skip |= LogError(
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001781 device, "VUID-RuntimeSpirv-shaderSignedZeroInfNanPreserveFloat16-06293",
1782 "Shader requires SignedZeroInfNanPreserve for bit width 16 but it is not enabled on the device");
1783 } else if (bit_width == 32 && !phys_dev_props_core12.shaderSignedZeroInfNanPreserveFloat32) {
1784 skip |= LogError(
1785 device, "VUID-RuntimeSpirv-shaderSignedZeroInfNanPreserveFloat32-06294",
1786 "Shader requires SignedZeroInfNanPreserve for bit width 32 but it is not enabled on the device");
1787 } else if (bit_width == 64 && !phys_dev_props_core12.shaderSignedZeroInfNanPreserveFloat64) {
1788 skip |= LogError(
1789 device, "VUID-RuntimeSpirv-shaderSignedZeroInfNanPreserveFloat64-06295",
1790 "Shader requires SignedZeroInfNanPreserve for bit width 64 but it is not enabled on the device");
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001791 }
1792 break;
1793 }
1794
1795 case spv::ExecutionModeDenormPreserve: {
1796 auto bit_width = insn.word(3);
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001797 if (bit_width == 16 && !phys_dev_props_core12.shaderDenormPreserveFloat16) {
1798 skip |= LogError(device, "VUID-RuntimeSpirv-shaderDenormPreserveFloat16-06296",
1799 "Shader requires DenormPreserve for bit width 16 but it is not enabled on the device");
1800 } else if (bit_width == 32 && !phys_dev_props_core12.shaderDenormPreserveFloat32) {
1801 skip |= LogError(device, "VUID-RuntimeSpirv-shaderDenormPreserveFloat32-06297",
1802 "Shader requires DenormPreserve for bit width 32 but it is not enabled on the device");
1803 } else if (bit_width == 64 && !phys_dev_props_core12.shaderDenormPreserveFloat64) {
1804 skip |= LogError(device, "VUID-RuntimeSpirv-shaderDenormPreserveFloat64-06298",
1805 "Shader requires DenormPreserve for bit width 64 but it is not enabled on the device");
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001806 }
1807
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01001808 if (first_denorm_execution_mode.first == spv::ExecutionModeMax) {
1809 // Register the first denorm execution mode found
1810 first_denorm_execution_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001811 } else if (first_denorm_execution_mode.first != mode && first_denorm_execution_mode.second != bit_width) {
Piers Daniell41b8c5d2020-01-10 15:42:00 -07001812 switch (phys_dev_props_core12.denormBehaviorIndependence) {
Mike Schuchardt2df08912020-12-15 16:28:09 -08001813 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001814 if (first_rounding_mode.second != 32 && bit_width != 32) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001815 skip |= LogError(device, "VUID-RuntimeSpirv-denormBehaviorIndependence-06289",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001816 "Shader uses different denorm execution modes for 16 and 64-bit but "
1817 "denormBehaviorIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001818 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001819 }
1820 break;
1821
Mike Schuchardt2df08912020-12-15 16:28:09 -08001822 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_ALL:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001823 break;
1824
Mike Schuchardt2df08912020-12-15 16:28:09 -08001825 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE:
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001826 skip |= LogError(device, "VUID-RuntimeSpirv-denormBehaviorIndependence-06290",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001827 "Shader uses different denorm execution modes for different bit widths but "
1828 "denormBehaviorIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001829 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001830 break;
1831
1832 default:
1833 break;
1834 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001835 }
1836 break;
1837 }
1838
1839 case spv::ExecutionModeDenormFlushToZero: {
1840 auto bit_width = insn.word(3);
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001841 if (bit_width == 16 && !phys_dev_props_core12.shaderDenormFlushToZeroFloat16) {
1842 skip |= LogError(device, "VUID-RuntimeSpirv-shaderDenormFlushToZeroFloat16-06299",
1843 "Shader requires DenormFlushToZero for bit width 16 but it is not enabled on the device");
1844 } else if (bit_width == 32 && !phys_dev_props_core12.shaderDenormFlushToZeroFloat32) {
1845 skip |= LogError(device, "VUID-RuntimeSpirv-shaderDenormFlushToZeroFloat32-06300",
1846 "Shader requires DenormFlushToZero for bit width 32 but it is not enabled on the device");
1847 } else if (bit_width == 64 && !phys_dev_props_core12.shaderDenormFlushToZeroFloat64) {
1848 skip |= LogError(device, "VUID-RuntimeSpirv-shaderDenormFlushToZeroFloat64-06301",
1849 "Shader requires DenormFlushToZero for bit width 64 but it is not enabled on the device");
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001850 }
1851
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01001852 if (first_denorm_execution_mode.first == spv::ExecutionModeMax) {
1853 // Register the first denorm execution mode found
1854 first_denorm_execution_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001855 } else if (first_denorm_execution_mode.first != mode && first_denorm_execution_mode.second != bit_width) {
Piers Daniell41b8c5d2020-01-10 15:42:00 -07001856 switch (phys_dev_props_core12.denormBehaviorIndependence) {
Mike Schuchardt2df08912020-12-15 16:28:09 -08001857 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001858 if (first_rounding_mode.second != 32 && bit_width != 32) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001859 skip |= LogError(device, "VUID-RuntimeSpirv-denormBehaviorIndependence-06289",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001860 "Shader uses different denorm execution modes for 16 and 64-bit but "
1861 "denormBehaviorIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001862 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001863 }
1864 break;
1865
Mike Schuchardt2df08912020-12-15 16:28:09 -08001866 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_ALL:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001867 break;
1868
Mike Schuchardt2df08912020-12-15 16:28:09 -08001869 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE:
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001870 skip |= LogError(device, "VUID-RuntimeSpirv-denormBehaviorIndependence-06290",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001871 "Shader uses different denorm execution modes for different bit widths but "
1872 "denormBehaviorIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001873 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001874 break;
1875
1876 default:
1877 break;
1878 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001879 }
1880 break;
1881 }
1882
1883 case spv::ExecutionModeRoundingModeRTE: {
1884 auto bit_width = insn.word(3);
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001885 if (bit_width == 16 && !phys_dev_props_core12.shaderRoundingModeRTEFloat16) {
1886 skip |= LogError(device, "VUID-RuntimeSpirv-shaderRoundingModeRTEFloat16-06302",
1887 "Shader requires RoundingModeRTE for bit width 16 but it is not enabled on the device");
1888 } else if (bit_width == 32 && !phys_dev_props_core12.shaderRoundingModeRTEFloat32) {
1889 skip |= LogError(device, "VUID-RuntimeSpirv-shaderRoundingModeRTEFloat32-06303",
1890 "Shader requires RoundingModeRTE for bit width 32 but it is not enabled on the device");
1891 } else if (bit_width == 64 && !phys_dev_props_core12.shaderRoundingModeRTEFloat64) {
1892 skip |= LogError(device, "VUID-RuntimeSpirv-shaderRoundingModeRTEFloat64-06304",
1893 "Shader requires RoundingModeRTE for bit width 64 but it is not enabled on the device");
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001894 }
1895
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01001896 if (first_rounding_mode.first == spv::ExecutionModeMax) {
1897 // Register the first rounding mode found
1898 first_rounding_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001899 } else if (first_rounding_mode.first != mode && first_rounding_mode.second != bit_width) {
Piers Daniell41b8c5d2020-01-10 15:42:00 -07001900 switch (phys_dev_props_core12.roundingModeIndependence) {
Mike Schuchardt2df08912020-12-15 16:28:09 -08001901 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001902 if (first_rounding_mode.second != 32 && bit_width != 32) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001903 skip |= LogError(device, "VUID-RuntimeSpirv-roundingModeIndependence-06291",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001904 "Shader uses different rounding modes for 16 and 64-bit but "
1905 "roundingModeIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001906 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001907 }
1908 break;
1909
Mike Schuchardt2df08912020-12-15 16:28:09 -08001910 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_ALL:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001911 break;
1912
Mike Schuchardt2df08912020-12-15 16:28:09 -08001913 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE:
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001914 skip |= LogError(device, "VUID-RuntimeSpirv-roundingModeIndependence-06292",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001915 "Shader uses different rounding modes for different bit widths but "
1916 "roundingModeIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001917 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001918 break;
1919
1920 default:
1921 break;
1922 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001923 }
1924 break;
1925 }
1926
1927 case spv::ExecutionModeRoundingModeRTZ: {
1928 auto bit_width = insn.word(3);
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001929 if (bit_width == 16 && !phys_dev_props_core12.shaderRoundingModeRTZFloat16) {
1930 skip |= LogError(device, "VUID-RuntimeSpirv-shaderRoundingModeRTZFloat16-06305",
1931 "Shader requires RoundingModeRTZ for bit width 16 but it is not enabled on the device");
1932 } else if (bit_width == 32 && !phys_dev_props_core12.shaderRoundingModeRTZFloat32) {
1933 skip |= LogError(device, "VUID-RuntimeSpirv-shaderRoundingModeRTZFloat32-06306",
1934 "Shader requires RoundingModeRTZ for bit width 32 but it is not enabled on the device");
1935 } else if (bit_width == 64 && !phys_dev_props_core12.shaderRoundingModeRTZFloat64) {
1936 skip |= LogError(device, "VUID-RuntimeSpirv-shaderRoundingModeRTZFloat64-06307",
1937 "Shader requires RoundingModeRTZ for bit width 64 but it is not enabled on the device");
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001938 }
1939
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01001940 if (first_rounding_mode.first == spv::ExecutionModeMax) {
1941 // Register the first rounding mode found
1942 first_rounding_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001943 } else if (first_rounding_mode.first != mode && first_rounding_mode.second != bit_width) {
Piers Daniell41b8c5d2020-01-10 15:42:00 -07001944 switch (phys_dev_props_core12.roundingModeIndependence) {
Mike Schuchardt2df08912020-12-15 16:28:09 -08001945 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001946 if (first_rounding_mode.second != 32 && bit_width != 32) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001947 skip |= LogError(device, "VUID-RuntimeSpirv-roundingModeIndependence-06291",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001948 "Shader uses different rounding modes for 16 and 64-bit but "
1949 "roundingModeIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001950 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_32_BIT_ONLY on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001951 }
1952 break;
1953
Mike Schuchardt2df08912020-12-15 16:28:09 -08001954 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_ALL:
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001955 break;
1956
Mike Schuchardt2df08912020-12-15 16:28:09 -08001957 case VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE:
sfricke-samsung1ff329f2021-09-16 10:06:47 -07001958 skip |= LogError(device, "VUID-RuntimeSpirv-roundingModeIndependence-06292",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001959 "Shader uses different rounding modes for different bit widths but "
1960 "roundingModeIndependence is "
Mike Schuchardt2df08912020-12-15 16:28:09 -08001961 "VK_SHADER_FLOAT_CONTROLS_INDEPENDENCE_NONE on the device");
Jason Ekstrande1e06de2019-08-05 11:43:43 -05001962 break;
1963
1964 default:
1965 break;
1966 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001967 }
1968 break;
1969 }
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001970
1971 case spv::ExecutionModeOutputVertices: {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001972 vertices_out = insn.word(3);
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001973 break;
1974 }
1975
1976 case spv::ExecutionModeInvocations: {
1977 invocations = insn.word(3);
1978 break;
1979 }
Piers Daniella7f93b62021-11-20 12:32:04 -07001980
1981 case spv::ExecutionModeLocalSizeId: {
1982 if (!enabled_features.maintenance4_features.maintenance4) {
1983 skip |= LogError(device, "VUID-RuntimeSpirv-LocalSizeId-06434",
1984 "LocalSizeId execution mode used but maintenance4 feature not enabled");
1985 }
1986 break;
1987 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001988 }
1989 }
1990 }
1991
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001992 if (entrypoint.word(1) == spv::ExecutionModelGeometry) {
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001993 if (vertices_out == 0 || vertices_out > phys_dev_props.limits.maxGeometryOutputVertices) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07001994 skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-stage-00714",
1995 "Geometry shader entry point must have an OpExecutionMode instruction that "
1996 "specifies a maximum output vertex count that is greater than 0 and less "
1997 "than or equal to maxGeometryOutputVertices. "
1998 "OutputVertices=%d, maxGeometryOutputVertices=%d",
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07001999 vertices_out, phys_dev_props.limits.maxGeometryOutputVertices);
Jeff Bolze9ee3d82019-05-29 13:45:13 -05002000 }
2001
2002 if (invocations == 0 || invocations > phys_dev_props.limits.maxGeometryShaderInvocations) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002003 skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-stage-00715",
2004 "Geometry shader entry point must have an OpExecutionMode instruction that "
2005 "specifies an invocation count that is greater than 0 and less "
2006 "than or equal to maxGeometryShaderInvocations. "
2007 "Invocations=%d, maxGeometryShaderInvocations=%d",
2008 invocations, phys_dev_props.limits.maxGeometryShaderInvocations);
Jeff Bolze9ee3d82019-05-29 13:45:13 -05002009 }
2010 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002011 return skip;
2012}
2013
Chris Forbes47567b72017-06-09 12:09:45 -07002014// For given pipelineLayout verify that the set_layout_node at slot.first
2015// has the requested binding at slot.second and return ptr to that binding
Mark Lobodzinskica6ebe32019-04-25 11:43:37 -06002016static VkDescriptorSetLayoutBinding const *GetDescriptorBinding(PIPELINE_LAYOUT_STATE const *pipelineLayout,
Jeremy Gebben7fc88a22021-08-25 13:30:45 -06002017 DescriptorSlot slot) {
Chris Forbes47567b72017-06-09 12:09:45 -07002018 if (!pipelineLayout) return nullptr;
2019
Jeremy Gebben7fc88a22021-08-25 13:30:45 -06002020 if (slot.set >= pipelineLayout->set_layouts.size()) return nullptr;
Chris Forbes47567b72017-06-09 12:09:45 -07002021
Jeremy Gebben7fc88a22021-08-25 13:30:45 -06002022 return pipelineLayout->set_layouts[slot.set]->GetDescriptorSetLayoutBindingPtrFromBinding(slot.binding);
Chris Forbes47567b72017-06-09 12:09:45 -07002023}
2024
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002025// If PointList topology is specified in the pipeline, verify that a shader geometry stage writes PointSize
2026// o If there is only a vertex shader : gl_PointSize must be written when using points
2027// o If there is a geometry or tessellation shader:
2028// - If shaderTessellationAndGeometryPointSize feature is enabled:
2029// * gl_PointSize must be written in the final geometry stage
2030// - If shaderTessellationAndGeometryPointSize feature is disabled:
2031// * gl_PointSize must NOT be written and a default of 1.0 is assumed
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002032bool CoreChecks::ValidatePointListShaderState(const PIPELINE_STATE *pipeline, SHADER_MODULE_STATE const *src,
John Zulaufac4c6e12019-07-01 16:05:58 -06002033 spirv_inst_iter entrypoint, VkShaderStageFlagBits stage) const {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002034 if (pipeline->topology_at_rasterizer != VK_PRIMITIVE_TOPOLOGY_POINT_LIST) {
2035 return false;
2036 }
2037
2038 bool pointsize_written = false;
2039 bool skip = false;
2040
2041 // Search for PointSize built-in decorations
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06002042 for (const auto &set : src->GetBuiltinDecorationList()) {
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002043 auto insn = src->at(set.offset);
2044 if (set.builtin == spv::BuiltInPointSize) {
sfricke-samsung962cad92021-04-13 00:46:29 -07002045 pointsize_written = src->IsBuiltInWritten(insn, entrypoint);
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002046 if (pointsize_written) {
2047 break;
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002048 }
2049 }
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002050 }
2051
2052 if ((stage == VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT || stage == VK_SHADER_STAGE_GEOMETRY_BIT) &&
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002053 !enabled_features.core.shaderTessellationAndGeometryPointSize) {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002054 if (pointsize_written) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002055 skip |= LogError(pipeline->pipeline(), kVUID_Core_Shader_PointSizeBuiltInOverSpecified,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002056 "Pipeline topology is set to POINT_LIST and geometry or tessellation shaders write PointSize which "
2057 "is prohibited when the shaderTessellationAndGeometryPointSize feature is not enabled.");
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002058 }
2059 } else if (!pointsize_written) {
2060 skip |=
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002061 LogError(pipeline->pipeline(), kVUID_Core_Shader_MissingPointSizeBuiltIn,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002062 "Pipeline topology is set to POINT_LIST, but PointSize is not written to in the shader corresponding to %s.",
2063 string_VkShaderStageFlagBits(stage));
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002064 }
2065 return skip;
2066}
John Zulauf14c355b2019-06-27 16:09:37 -06002067
Tobias Hector6663c9b2020-11-05 10:18:02 +00002068bool CoreChecks::ValidatePrimitiveRateShaderState(const PIPELINE_STATE *pipeline, SHADER_MODULE_STATE const *src,
2069 spirv_inst_iter entrypoint, VkShaderStageFlagBits stage) const {
2070 bool primitiverate_written = false;
2071 bool viewportindex_written = false;
2072 bool viewportmask_written = false;
2073 bool skip = false;
2074
2075 // Check if the primitive shading rate is written
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06002076 for (const auto &set : src->GetBuiltinDecorationList()) {
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002077 auto insn = src->at(set.offset);
2078 if (set.builtin == spv::BuiltInPrimitiveShadingRateKHR) {
sfricke-samsung962cad92021-04-13 00:46:29 -07002079 primitiverate_written = src->IsBuiltInWritten(insn, entrypoint);
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002080 } else if (set.builtin == spv::BuiltInViewportIndex) {
sfricke-samsung962cad92021-04-13 00:46:29 -07002081 viewportindex_written = src->IsBuiltInWritten(insn, entrypoint);
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002082 } else if (set.builtin == spv::BuiltInViewportMaskNV) {
sfricke-samsung962cad92021-04-13 00:46:29 -07002083 viewportmask_written = src->IsBuiltInWritten(insn, entrypoint);
Tobias Hector6663c9b2020-11-05 10:18:02 +00002084 }
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002085 if (primitiverate_written && viewportindex_written && viewportmask_written) {
2086 break;
2087 }
Tobias Hector6663c9b2020-11-05 10:18:02 +00002088 }
2089
Tony-LunarGd44844c2021-01-22 13:24:37 -07002090 if (!phys_dev_ext_props.fragment_shading_rate_props.primitiveFragmentShadingRateWithMultipleViewports &&
Jeremy Gebben11af9792021-08-20 10:20:09 -06002091 (pipeline->GetPipelineType() == VK_PIPELINE_BIND_POINT_GRAPHICS) && pipeline->create_info.graphics.pViewportState) {
Tobias Hector6663c9b2020-11-05 10:18:02 +00002092 if (!IsDynamic(pipeline, VK_DYNAMIC_STATE_VIEWPORT_WITH_COUNT_EXT) &&
Jeremy Gebben11af9792021-08-20 10:20:09 -06002093 pipeline->create_info.graphics.pViewportState->viewportCount > 1 && primitiverate_written) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002094 skip |= LogError(pipeline->pipeline(),
Mark Lobodzinski1f887d32020-12-30 15:31:33 -07002095 "VUID-VkGraphicsPipelineCreateInfo-primitiveFragmentShadingRateWithMultipleViewports-04503",
2096 "vkCreateGraphicsPipelines: %s shader statically writes to PrimitiveShadingRateKHR built-in, but "
2097 "multiple viewports "
2098 "are used and the primitiveFragmentShadingRateWithMultipleViewports limit is not supported.",
2099 string_VkShaderStageFlagBits(stage));
Tobias Hector6663c9b2020-11-05 10:18:02 +00002100 }
2101
2102 if (primitiverate_written && viewportindex_written) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002103 skip |= LogError(pipeline->pipeline(),
Tobias Hector6663c9b2020-11-05 10:18:02 +00002104 "VUID-VkGraphicsPipelineCreateInfo-primitiveFragmentShadingRateWithMultipleViewports-04504",
2105 "vkCreateGraphicsPipelines: %s shader statically writes to both PrimitiveShadingRateKHR and "
2106 "ViewportIndex built-ins,"
2107 "but the primitiveFragmentShadingRateWithMultipleViewports limit is not supported.",
2108 string_VkShaderStageFlagBits(stage));
2109 }
2110
2111 if (primitiverate_written && viewportmask_written) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002112 skip |= LogError(pipeline->pipeline(),
Tobias Hector6663c9b2020-11-05 10:18:02 +00002113 "VUID-VkGraphicsPipelineCreateInfo-primitiveFragmentShadingRateWithMultipleViewports-04505",
2114 "vkCreateGraphicsPipelines: %s shader statically writes to both PrimitiveShadingRateKHR and "
2115 "ViewportMaskNV built-ins,"
2116 "but the primitiveFragmentShadingRateWithMultipleViewports limit is not supported.",
2117 string_VkShaderStageFlagBits(stage));
2118 }
2119 }
2120 return skip;
2121}
2122
ziga-lunargce66e542021-09-19 00:11:14 +02002123bool CoreChecks::ValidateDecorations(SHADER_MODULE_STATE const* module) const {
2124 bool skip = false;
2125
ziga-lunargef2c3172021-11-07 10:35:29 +01002126 std::vector<uint32_t> xfb_buffers;
2127 std::vector<spirv_inst_iter> xfb_offsets;
2128
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06002129 for (const auto &op_decorate : module->GetDecorationInstructions()) {
ziga-lunargce66e542021-09-19 00:11:14 +02002130 uint32_t decoration = op_decorate.word(2);
2131 if (decoration == spv::DecorationXfbStride) {
2132 uint32_t stride = op_decorate.word(3);
2133 if (stride > phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataStride) {
2134 skip |= LogError(
2135 device, "VUID-RuntimeSpirv-XfbStride-06313",
2136 "vkCreateGraphicsPipelines(): shader uses transform feedback with xfb_stride (%" PRIu32
2137 ") greater than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackBufferDataStride (%" PRIu32
2138 ").",
2139 stride, phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataStride);
2140 }
2141 }
ziga-lunarg423cf212021-11-07 00:00:27 +01002142 if (decoration == spv::DecorationStream) {
2143 uint32_t stream = op_decorate.word(3);
2144 if (stream >= phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams) {
2145 skip |= LogError(
2146 device, "VUID-RuntimeSpirv-Stream-06312",
2147 "vkCreateGraphicsPipelines(): shader uses transform feedback with stream (%" PRIu32
2148 ") not less than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackStreams (%" PRIu32 ").",
2149 stream, phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams);
2150 }
2151 }
ziga-lunargef2c3172021-11-07 10:35:29 +01002152 if (decoration == spv::DecorationXfbBuffer) {
2153 xfb_buffers.push_back(op_decorate.word(1));
2154 }
2155 if (decoration == spv::DecorationOffset) {
2156 xfb_offsets.push_back(op_decorate);
2157 }
2158 }
2159
2160 for (const auto &op_decorate : xfb_offsets) {
2161 for (const auto xfb_buffer : xfb_buffers) {
2162 if (xfb_buffer == op_decorate.word(1)) {
2163 const auto offset = op_decorate.word(3);
2164 const auto def = module->get_def(xfb_buffer);
2165 const auto size = module->GetTypeBytesSize(def);
2166 if (offset + size > phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataSize) {
2167 skip |= LogError(
2168 device, "VUID-RuntimeSpirv-Offset-06308",
2169 "vkCreateGraphicsPipelines(): shader uses transform feedback with xfb_offset (%" PRIu32
2170 ") + size of variable (%" PRIu32 ") greater than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackBufferDataSize "
2171 "(%" PRIu32 ").",
2172 offset, size, phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackBufferDataSize);
2173 }
2174 break;
2175 }
2176 }
ziga-lunargce66e542021-09-19 00:11:14 +02002177 }
2178
2179 return skip;
2180}
2181
ziga-lunarg28d08792021-10-13 15:42:59 +02002182bool CoreChecks::ValidateTransformFeedback(SHADER_MODULE_STATE const *src) const {
ziga-lunargce66e542021-09-19 00:11:14 +02002183 bool skip = false;
2184
ziga-lunarg28d08792021-10-13 15:42:59 +02002185 // Temp workaround to prevent false positive errors
2186 // https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/2450
2187 if (src->HasMultipleEntryPoints()) {
2188 return skip;
2189 }
2190
2191 layer_data::unordered_set<uint32_t> emitted_streams;
2192 bool output_points = false;
2193 for (const auto& insn : *src) {
2194 const uint32_t opcode = insn.opcode();
2195 if (opcode == spv::OpEmitStreamVertex) {
2196 emitted_streams.emplace(static_cast<uint32_t>(src->GetConstantValueById(insn.word(1))));
ziga-lunargce66e542021-09-19 00:11:14 +02002197 }
ziga-lunarg28d08792021-10-13 15:42:59 +02002198 if (opcode == spv::OpEmitStreamVertex || opcode == spv::OpEndStreamPrimitive) {
2199 uint32_t stream = static_cast<uint32_t>(src->GetConstantValueById(insn.word(1)));
2200 if (stream >= phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams) {
2201 skip |= LogError(
2202 device, "VUID-RuntimeSpirv-OpEmitStreamVertex-06310",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07002203 "vkCreateGraphicsPipelines(): shader uses transform feedback stream (%s) with index %" PRIu32
ziga-lunarg28d08792021-10-13 15:42:59 +02002204 ", which is not less than VkPhysicalDeviceTransformFeedbackPropertiesEXT::maxTransformFeedbackStreams (%" PRIu32
2205 ").",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07002206 string_SpvOpcode(opcode), stream, phys_dev_ext_props.transform_feedback_props.maxTransformFeedbackStreams);
ziga-lunarg28d08792021-10-13 15:42:59 +02002207 }
2208 }
2209 if (opcode == spv::OpExecutionMode && insn.word(2) == spv::ExecutionModeOutputPoints) {
2210 output_points = true;
2211 }
2212 }
2213
2214 const uint32_t emitted_streams_size = static_cast<uint32_t>(emitted_streams.size());
2215 if (emitted_streams_size > 1 && !output_points &&
2216 phys_dev_ext_props.transform_feedback_props.transformFeedbackStreamsLinesTriangles == VK_FALSE) {
2217 skip |= LogError(
2218 device, "VUID-RuntimeSpirv-transformFeedbackStreamsLinesTriangles-06311",
2219 "vkCreateGraphicsPipelines(): shader emits to %" PRIu32 " vertex streams and VkPhysicalDeviceTransformFeedbackPropertiesEXT::transformFeedbackStreamsLinesTriangles is VK_FALSE, but execution mode is not OutputPoints.",
2220 emitted_streams_size);
ziga-lunargce66e542021-09-19 00:11:14 +02002221 }
2222
2223 return skip;
2224}
2225
sfricke-samsung864162a2021-11-01 21:58:01 -07002226// Checks for both TexelOffset and TexelGatherOffset limits
2227bool CoreChecks::ValidateTexelOffsetLimits(SHADER_MODULE_STATE const *src, spirv_inst_iter &insn) const {
ziga-lunarga12c75a2021-09-16 16:36:16 +02002228 bool skip = false;
2229
2230 const uint32_t opcode = insn.opcode();
sfricke-samsung864162a2021-11-01 21:58:01 -07002231 if (ImageGatherOperation(opcode) || ImageSampleOperation(opcode) || ImageFetchOperation(opcode)) {
sfricke-samsung3511e312021-11-04 21:14:31 -07002232 uint32_t image_operand_position = ImageOperandsParamPosition(opcode);
sfricke-samsung864162a2021-11-01 21:58:01 -07002233 // Image operands can be optional
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07002234 if (image_operand_position != 0 && insn.len() > image_operand_position) {
2235 auto image_operand = insn.word(image_operand_position);
sfricke-samsung864162a2021-11-01 21:58:01 -07002236 // Bits we are validating (sample/fetch only check ConstOffset)
ziga-lunarga12c75a2021-09-16 16:36:16 +02002237 uint32_t offset_bits =
sfricke-samsung864162a2021-11-01 21:58:01 -07002238 ImageGatherOperation(opcode)
2239 ? (spv::ImageOperandsOffsetMask | spv::ImageOperandsConstOffsetMask | spv::ImageOperandsConstOffsetsMask)
2240 : (spv::ImageOperandsConstOffsetMask);
ziga-lunarga12c75a2021-09-16 16:36:16 +02002241 if (image_operand & (offset_bits)) {
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07002242 // Operand values follow
2243 uint32_t index = image_operand_position + 1;
ziga-lunarga12c75a2021-09-16 16:36:16 +02002244 // Each bit has it's own operand, starts with the smallest set bit and loop to the highest bit among
2245 // ImageOperandsOffsetMask, ImageOperandsConstOffsetMask and ImageOperandsConstOffsetsMask
2246 for (uint32_t i = 1; i < spv::ImageOperandsConstOffsetsMask; i <<= 1) {
2247 if (image_operand & i) { // If the bit is set, consume operand
2248 if (insn.len() > index && (i & offset_bits)) {
2249 uint32_t constant_id = insn.word(index);
2250 const auto &constant = src->get_def(constant_id);
Shahbaz Youssefi7a6a5272021-10-06 15:07:10 -04002251 const bool is_dynamic_offset = constant == src->end();
2252 if (!is_dynamic_offset && constant.opcode() == spv::OpConstantComposite) {
ziga-lunarga12c75a2021-09-16 16:36:16 +02002253 for (uint32_t j = 3; j < constant.len(); ++j) {
2254 uint32_t comp_id = constant.word(j);
2255 const auto &comp = src->get_def(comp_id);
sfricke-samsungef3fe742021-10-06 10:51:34 -07002256 const auto &comp_type = src->get_def(comp.word(1));
ziga-lunarga12c75a2021-09-16 16:36:16 +02002257 // Get operand value
sfricke-samsungef3fe742021-10-06 10:51:34 -07002258 const uint32_t offset = comp.word(3);
sfricke-samsung864162a2021-11-01 21:58:01 -07002259 // spec requires minTexelGatherOffset/minTexelOffset to be -8 or less so never can compare if
2260 // unsigned spec requires maxTexelGatherOffset/maxTexelOffset to be 7 or greater so never can
2261 // compare if signed is less then zero
sfricke-samsungef3fe742021-10-06 10:51:34 -07002262 const int32_t signed_offset = static_cast<int32_t>(offset);
2263 const bool use_signed = (comp_type.opcode() == spv::OpTypeInt && comp_type.word(3) != 0);
2264
sfricke-samsung864162a2021-11-01 21:58:01 -07002265 // There are 2 sets of VU being covered where the only main difference is the opcode
2266 if (ImageGatherOperation(opcode)) {
2267 // min/maxTexelGatherOffset
2268 if (use_signed && (signed_offset < phys_dev_props.limits.minTexelGatherOffset)) {
2269 skip |=
2270 LogError(device, "VUID-RuntimeSpirv-OpImage-06376",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07002271 "vkCreateShaderModule(): Shader uses %s with offset (%" PRIi32
sfricke-samsungef3fe742021-10-06 10:51:34 -07002272 ") less than VkPhysicalDeviceLimits::minTexelGatherOffset (%" PRIi32 ").",
sfricke-samsung73f1a0f2021-10-28 23:27:25 -07002273 string_SpvOpcode(opcode), signed_offset,
2274 phys_dev_props.limits.minTexelGatherOffset);
sfricke-samsung864162a2021-11-01 21:58:01 -07002275 } else if ((offset > phys_dev_props.limits.maxTexelGatherOffset) &&
2276 (!use_signed || (use_signed && signed_offset > 0))) {
2277 skip |= LogError(
2278 device, "VUID-RuntimeSpirv-OpImage-06377",
2279 "vkCreateShaderModule(): Shader uses %s with offset (%" PRIu32
2280 ") greater than VkPhysicalDeviceLimits::maxTexelGatherOffset (%" PRIu32 ").",
2281 string_SpvOpcode(opcode), offset, phys_dev_props.limits.maxTexelGatherOffset);
2282 }
2283 } else {
2284 // min/maxTexelOffset
2285 if (use_signed && (signed_offset < phys_dev_props.limits.minTexelOffset)) {
2286 skip |= LogError(device, "VUID-RuntimeSpirv-OpImageSample-06435",
2287 "vkCreateShaderModule(): Shader uses %s with offset (%" PRIi32
2288 ") less than VkPhysicalDeviceLimits::minTexelOffset (%" PRIi32 ").",
2289 string_SpvOpcode(opcode), signed_offset,
2290 phys_dev_props.limits.minTexelOffset);
2291 } else if ((offset > phys_dev_props.limits.maxTexelOffset) &&
2292 (!use_signed || (use_signed && signed_offset > 0))) {
2293 skip |=
2294 LogError(device, "VUID-RuntimeSpirv-OpImageSample-06436",
2295 "vkCreateShaderModule(): Shader uses %s with offset (%" PRIu32
2296 ") greater than VkPhysicalDeviceLimits::maxTexelOffset (%" PRIu32 ").",
2297 string_SpvOpcode(opcode), offset, phys_dev_props.limits.maxTexelOffset);
2298 }
ziga-lunarga12c75a2021-09-16 16:36:16 +02002299 }
2300 }
2301 }
2302 }
sfricke-samsung3511e312021-11-04 21:14:31 -07002303 index += ImageOperandsParamCount(i);
ziga-lunarga12c75a2021-09-16 16:36:16 +02002304 }
2305 }
2306 }
2307 }
2308 }
2309
2310 return skip;
2311}
2312
sfricke-samsung306dc4c2021-09-20 15:25:18 -07002313bool CoreChecks::ValidateShaderClock(SHADER_MODULE_STATE const *module, spirv_inst_iter &insn) const {
sfricke-samsung486a51e2021-01-02 00:10:15 -08002314 bool skip = false;
2315
sfricke-samsung94167ca2021-02-26 04:14:59 -08002316 switch (insn.opcode()) {
2317 case spv::OpReadClockKHR: {
2318 auto scope_id = module->get_def(insn.word(3));
2319 auto scope_type = scope_id.word(3);
2320 // if scope isn't Subgroup or Device, spirv-val will catch
sfricke-samsung828e59d2021-08-22 23:20:49 -07002321 if ((scope_type == spv::ScopeSubgroup) && (enabled_features.shader_clock_features.shaderSubgroupClock == VK_FALSE)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07002322 skip |= LogError(device, "VUID-RuntimeSpirv-shaderSubgroupClock-06267",
sfricke-samsung94167ca2021-02-26 04:14:59 -08002323 "%s: OpReadClockKHR is used with a Subgroup scope but shaderSubgroupClock was not enabled.",
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002324 report_data->FormatHandle(module->vk_shader_module()).c_str());
sfricke-samsung828e59d2021-08-22 23:20:49 -07002325 } else if ((scope_type == spv::ScopeDevice) && (enabled_features.shader_clock_features.shaderDeviceClock == VK_FALSE)) {
sfricke-samsung1ff329f2021-09-16 10:06:47 -07002326 skip |= LogError(device, "VUID-RuntimeSpirv-shaderDeviceClock-06268",
sfricke-samsung94167ca2021-02-26 04:14:59 -08002327 "%s: OpReadClockKHR is used with a Device scope but shaderDeviceClock was not enabled.",
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002328 report_data->FormatHandle(module->vk_shader_module()).c_str());
sfricke-samsung486a51e2021-01-02 00:10:15 -08002329 }
sfricke-samsung94167ca2021-02-26 04:14:59 -08002330 break;
sfricke-samsung486a51e2021-01-02 00:10:15 -08002331 }
2332 }
2333 return skip;
2334}
2335
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002336bool CoreChecks::ValidatePipelineShaderStage(const PIPELINE_STATE *pipeline, const PipelineStageState &stage_state,
2337 bool check_point_size) const {
John Zulauf14c355b2019-06-27 16:09:37 -06002338 bool skip = false;
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002339 const auto *pStage = stage_state.create_info;
2340 const auto *module = stage_state.module.get();
2341 const auto &entrypoint = stage_state.entrypoint;
John Zulauf14c355b2019-06-27 16:09:37 -06002342 // Check the module
2343 if (!module->has_valid_spirv) {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002344 skip |= LogError(
2345 device, "VUID-VkPipelineShaderStageCreateInfo-module-parameter", "%s does not contain valid spirv for stage %s.",
2346 report_data->FormatHandle(module->vk_shader_module()).c_str(), string_VkShaderStageFlagBits(stage_state.stage_flag));
John Zulauf14c355b2019-06-27 16:09:37 -06002347 }
2348
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002349 // If specialization-constant values are given and specialization-constant instructions are present in the shader, the
2350 // specializations should be applied and validated.
2351 if (pStage->pSpecializationInfo != nullptr && pStage->pSpecializationInfo->mapEntryCount > 0 &&
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06002352 pStage->pSpecializationInfo->pMapEntries != nullptr && module->HasSpecConstants()) {
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002353 // Gather the specialization-constant values.
2354 auto const &specialization_info = pStage->pSpecializationInfo;
Jeremy Hayes521221d2020-01-15 16:48:49 -07002355 auto const &specialization_data = reinterpret_cast<uint8_t const *>(specialization_info->pData);
Jeremy Gebbencbf22862021-03-03 12:01:22 -07002356 std::unordered_map<uint32_t, std::vector<uint32_t>> id_value_map; // note: this must be std:: to work with spvtools
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002357 id_value_map.reserve(specialization_info->mapEntryCount);
2358 for (auto i = 0u; i < specialization_info->mapEntryCount; ++i) {
2359 auto const &map_entry = specialization_info->pMapEntries[i];
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06002360 const auto itr = module->GetSpecConstMap().find(map_entry.constantID);
sfricke-samsung033b0262021-07-09 00:53:06 -07002361 // "If a constantID value is not a specialization constant ID used in the shader, that map entry does not affect the
2362 // behavior of the pipeline."
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06002363 if (itr != module->GetSpecConstMap().cend()) {
sfricke-samsung033b0262021-07-09 00:53:06 -07002364 // Make sure map_entry.size matches the spec constant's size
2365 uint32_t spec_const_size = decoration_set::kInvalidValue;
2366 const auto def_ins = module->get_def(itr->second);
2367 const auto type_ins = module->get_def(def_ins.word(1));
2368 // Specialization constants can only be of type bool, scalar integer, or scalar floating point
2369 switch (type_ins.opcode()) {
2370 case spv::OpTypeBool:
2371 // "If the specialization constant is of type boolean, size must be the byte size of VkBool32"
2372 spec_const_size = sizeof(VkBool32);
2373 break;
2374 case spv::OpTypeInt:
2375 case spv::OpTypeFloat:
2376 spec_const_size = type_ins.word(2) / 8;
2377 break;
2378 default:
2379 // spirv-val should catch if SpecId is not used on a OpSpecConstantTrue/OpSpecConstantFalse/OpSpecConstant
2380 // and OpSpecConstant is validated to be a OpTypeInt or OpTypeFloat
2381 break;
2382 }
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002383
sfricke-samsung033b0262021-07-09 00:53:06 -07002384 if (map_entry.size != spec_const_size) {
2385 skip |=
2386 LogError(device, "VUID-VkSpecializationMapEntry-constantID-00776",
Nathaniel Cesariocf69bda2021-06-22 13:23:42 -06002387 "Specialization constant (ID = %" PRIu32 ", entry = %" PRIu32
2388 ") has invalid size %zu in shader module %s. Expected size is %" PRIu32 " from shader definition.",
2389 map_entry.constantID, i, map_entry.size,
2390 report_data->FormatHandle(module->vk_shader_module()).c_str(), spec_const_size);
sfricke-samsung033b0262021-07-09 00:53:06 -07002391 }
Nathaniel Cesariocf69bda2021-06-22 13:23:42 -06002392 }
2393
Jeremy Gebben12933ef2021-05-12 17:16:27 -06002394 if ((map_entry.offset + map_entry.size) <= specialization_info->dataSize) {
Nathaniel Cesario0b2a6422021-07-13 16:04:57 -06002395 // Allocate enough room for ceil(map_entry.size / 4) to store entries
2396 std::vector<uint32_t> entry_data((map_entry.size + 4 - 1) / 4, 0);
2397 uint8_t *out_p = reinterpret_cast<uint8_t *>(entry_data.data());
2398 const uint8_t *const start_in_p = specialization_data + map_entry.offset;
2399 const uint8_t *const end_in_p = start_in_p + map_entry.size;
2400
2401 std::copy(start_in_p, end_in_p, out_p);
2402 id_value_map.emplace(map_entry.constantID, std::move(entry_data));
Jeremy Gebben12933ef2021-05-12 17:16:27 -06002403 }
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002404 }
2405
sfricke-samsung5628f982021-10-19 09:21:59 -07002406 // both spirv-opt and spirv-val will use the same flags
2407 spvtools::ValidatorOptions options;
2408 AdjustValidatorOptions(device_extensions, enabled_features, options);
2409
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002410 // Apply the specialization-constant values and revalidate the shader module.
sfricke-samsung45996a42021-09-16 13:45:27 -07002411 spv_target_env spirv_environment = PickSpirvEnv(api_version, IsExtEnabled(device_extensions.vk_khr_spirv_1_4));
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002412 spvtools::Optimizer optimizer(spirv_environment);
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002413 spvtools::MessageConsumer consumer = [&skip, &module, &stage_state, this](spv_message_level_t level, const char *source,
2414 const spv_position_t &position,
2415 const char *message) {
2416 skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-module-parameter",
2417 "%s does not contain valid spirv for stage %s. %s",
2418 report_data->FormatHandle(module->vk_shader_module()).c_str(),
2419 string_VkShaderStageFlagBits(stage_state.stage_flag), message);
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002420 };
2421 optimizer.SetMessageConsumer(consumer);
2422 optimizer.RegisterPass(spvtools::CreateSetSpecConstantDefaultValuePass(id_value_map));
2423 optimizer.RegisterPass(spvtools::CreateFreezeSpecConstantValuePass());
2424 std::vector<uint32_t> specialized_spirv;
sfricke-samsung5628f982021-10-19 09:21:59 -07002425 auto const optimized = optimizer.Run(module->words.data(), module->words.size(), &specialized_spirv, options, false);
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002426 assert(optimized == true);
2427
2428 if (optimized) {
2429 spv_context ctx = spvContextCreate(spirv_environment);
2430 spv_const_binary_t binary{specialized_spirv.data(), specialized_spirv.size()};
2431 spv_diagnostic diag = nullptr;
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002432 auto const spv_valid = spvValidateWithOptions(ctx, options, &binary, &diag);
2433 if (spv_valid != SPV_SUCCESS) {
sfricke-samsungd3793802020-08-18 22:55:03 -07002434 skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-module-04145",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002435 "After specialization was applied, %s does not contain valid spirv for stage %s.",
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002436 report_data->FormatHandle(module->vk_shader_module()).c_str(),
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002437 string_VkShaderStageFlagBits(stage_state.stage_flag));
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002438 }
2439
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002440 spvDiagnosticDestroy(diag);
2441 spvContextDestroy(ctx);
2442 }
ziga-lunarg2818f492021-08-12 14:30:51 +02002443
2444 skip |= ValidateWorkgroupSize(module, pStage, id_value_map);
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06002445 }
2446
John Zulauf14c355b2019-06-27 16:09:37 -06002447 // Check the entrypoint
2448 if (entrypoint == module->end()) {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002449 skip |= LogError(device, "VUID-VkPipelineShaderStageCreateInfo-pName-00707", "No entrypoint found named `%s` for stage %s.",
2450 pStage->pName, string_VkShaderStageFlagBits(stage_state.stage_flag));
John Zulauf14c355b2019-06-27 16:09:37 -06002451 }
2452 if (skip) return true; // no point continuing beyond here, any analysis is just going to be garbage.
2453
2454 // Mark accessible ids
2455 auto &accessible_ids = stage_state.accessible_ids;
2456
Chris Forbes47567b72017-06-09 12:09:45 -07002457 // Validate descriptor set layout against what the entrypoint actually uses
Chris Forbes47567b72017-06-09 12:09:45 -07002458
sfricke-samsung94167ca2021-02-26 04:14:59 -08002459 // The following tries to limit the number of passes through the shader module. The validation passes in here are "stateless"
2460 // and mainly only checking the instruction in detail for a single operation
ziga-lunarga26b3602021-08-08 15:53:00 +02002461 uint32_t total_shared_size = 0;
sfricke-samsung94167ca2021-02-26 04:14:59 -08002462 for (auto insn : *module) {
sfricke-samsung864162a2021-11-01 21:58:01 -07002463 skip |= ValidateTexelOffsetLimits(module, insn);
sfricke-samsung94167ca2021-02-26 04:14:59 -08002464 skip |= ValidateShaderCapabilitiesAndExtensions(module, insn);
sfricke-samsung306dc4c2021-09-20 15:25:18 -07002465 skip |= ValidateShaderClock(module, insn);
sfricke-samsung94167ca2021-02-26 04:14:59 -08002466 skip |= ValidateShaderStageGroupNonUniform(module, pStage->stage, insn);
ziga-lunarg70651522021-10-11 17:23:30 +02002467 skip |= ValidateMemoryScope(module, insn);
ziga-lunarga26b3602021-08-08 15:53:00 +02002468 total_shared_size += module->CalcComputeSharedMemory(pStage->stage, insn);
2469 }
2470
2471 if (total_shared_size > phys_dev_props.limits.maxComputeSharedMemorySize) {
2472 skip |= LogError(device, kVUID_Core_Shader_MaxComputeSharedMemorySize,
ziga-lunarg76a2e6c2021-08-08 15:55:03 +02002473 "Shader uses %" PRIu32 " bytes of shared memory, more than allowed by physicalDeviceLimits::maxComputeSharedMemorySize (%" PRIu32 ")",
ziga-lunarga26b3602021-08-08 15:53:00 +02002474 total_shared_size, phys_dev_props.limits.maxComputeSharedMemorySize);
sfricke-samsung94167ca2021-02-26 04:14:59 -08002475 }
2476
ziga-lunarg28d08792021-10-13 15:42:59 +02002477 skip |= ValidateTransformFeedback(module);
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002478 skip |= ValidateShaderStageWritableOrAtomicDescriptor(pStage->stage, stage_state.has_writable_descriptor,
2479 stage_state.has_atomic_descriptor);
Jeff Bolze9ee3d82019-05-29 13:45:13 -05002480 skip |= ValidateShaderStageInputOutputLimits(module, pStage, pipeline, entrypoint);
Lionel Landwerlin892d6c32021-05-05 12:56:19 +03002481 skip |= ValidateShaderStorageImageFormats(module);
sfricke-samsungdc96f302020-03-18 20:42:10 -07002482 skip |= ValidateShaderStageMaxResources(pStage->stage, pipeline);
sfricke-samsung58b84352021-07-31 21:41:04 -07002483 skip |= ValidateAtomicsTypes(module);
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002484 skip |= ValidateExecutionModes(module, entrypoint);
ziga-lunargae2a5c42021-07-23 16:18:09 +02002485 skip |= ValidateSpecializations(pStage);
ziga-lunargce66e542021-09-19 00:11:14 +02002486 skip |= ValidateDecorations(module);
Jeremy Gebben11af9792021-08-20 10:20:09 -06002487 if (check_point_size && !pipeline->create_info.graphics.pRasterizationState->rasterizerDiscardEnable) {
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002488 skip |= ValidatePointListShaderState(pipeline, module, entrypoint, pStage->stage);
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002489 }
sfricke-samsungcfb44592021-07-25 00:36:28 -07002490 skip |= ValidateBuiltinLimits(module, entrypoint);
sfricke-samsungd093e522021-02-26 04:17:45 -08002491 if (enabled_features.cooperative_matrix_features.cooperativeMatrix) {
2492 skip |= ValidateCooperativeMatrix(module, pStage, pipeline);
2493 }
Tobias Hector6663c9b2020-11-05 10:18:02 +00002494 if (enabled_features.fragment_shading_rate_features.primitiveFragmentShadingRate) {
2495 skip |= ValidatePrimitiveRateShaderState(pipeline, module, entrypoint, pStage->stage);
2496 }
sfricke-samsung45996a42021-09-16 13:45:27 -07002497 if (IsExtEnabled(device_extensions.vk_qcom_render_pass_shader_resolve)) {
Jeff Leger9b3dcff2021-05-27 15:40:20 -04002498 skip |= ValidateShaderResolveQCOM(module, pStage, pipeline);
2499 }
ziga-lunarg73163742021-08-25 13:15:29 +02002500 if (IsExtEnabled(device_extensions.vk_ext_subgroup_size_control)) {
2501 skip |= ValidateShaderSubgroupSizeControl(pStage);
2502 }
Chris Forbes47567b72017-06-09 12:09:45 -07002503
sfricke-samsung7699b912021-04-12 23:01:51 -07002504 // "layout must be consistent with the layout of the * shader"
2505 // 'consistent' -> #descriptorsets-pipelinelayout-consistency
locke-lunarg9a16ebb2020-07-30 16:56:33 -06002506 std::string vuid_layout_mismatch;
Jeremy Gebben11af9792021-08-20 10:20:09 -06002507 switch (pipeline->create_info.graphics.sType) {
2508 case VK_STRUCTURE_TYPE_GRAPHICS_PIPELINE_CREATE_INFO:
2509 vuid_layout_mismatch = "VUID-VkGraphicsPipelineCreateInfo-layout-00756";
2510 break;
2511 case VK_STRUCTURE_TYPE_COMPUTE_PIPELINE_CREATE_INFO:
2512 vuid_layout_mismatch = "VUID-VkComputePipelineCreateInfo-layout-00703";
2513 break;
2514 case VK_STRUCTURE_TYPE_RAY_TRACING_PIPELINE_CREATE_INFO_KHR:
2515 vuid_layout_mismatch = "VUID-VkRayTracingPipelineCreateInfoKHR-layout-03427";
2516 break;
2517 case VK_STRUCTURE_TYPE_RAY_TRACING_PIPELINE_CREATE_INFO_NV:
2518 vuid_layout_mismatch = "VUID-VkRayTracingPipelineCreateInfoNV-layout-03427";
2519 break;
2520 default:
2521 assert(false);
2522 break;
locke-lunarg9a16ebb2020-07-30 16:56:33 -06002523 }
2524
sfricke-samsung7699b912021-04-12 23:01:51 -07002525 // Validate Push Constants use
2526 skip |= ValidatePushConstantUsage(*pipeline, module, pStage, vuid_layout_mismatch);
2527
Chris Forbes47567b72017-06-09 12:09:45 -07002528 // Validate descriptor use
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002529 for (auto use : stage_state.descriptor_uses) {
Chris Forbes47567b72017-06-09 12:09:45 -07002530 // Verify given pipelineLayout has requested setLayout with requested binding
Jeff Bolze7fc67b2019-10-04 12:29:31 -05002531 const auto &binding = GetDescriptorBinding(pipeline->pipeline_layout.get(), use.first);
Chris Forbes47567b72017-06-09 12:09:45 -07002532 unsigned required_descriptor_count;
sourav parmarcd5fb182020-07-17 12:58:44 -07002533 bool is_khr = binding && binding->descriptorType == VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_KHR;
2534 std::set<uint32_t> descriptor_types =
2535 TypeToDescriptorTypeSet(module, use.second.type_id, required_descriptor_count, is_khr);
Chris Forbes47567b72017-06-09 12:09:45 -07002536
2537 if (!binding) {
locke-lunarg9a16ebb2020-07-30 16:56:33 -06002538 skip |= LogError(device, vuid_layout_mismatch,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002539 "Shader uses descriptor slot %u.%u (expected `%s`) but not declared in pipeline layout",
Jeremy Gebben7fc88a22021-08-25 13:30:45 -06002540 use.first.set, use.first.binding, string_descriptorTypes(descriptor_types).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -07002541 } else if (~binding->stageFlags & pStage->stage) {
locke-lunarg9a16ebb2020-07-30 16:56:33 -06002542 skip |= LogError(device, vuid_layout_mismatch,
Jeremy Gebben7fc88a22021-08-25 13:30:45 -06002543 "Shader uses descriptor slot %u.%u but descriptor not accessible from stage %s", use.first.set,
2544 use.first.binding, string_VkShaderStageFlagBits(pStage->stage));
Tony-LunarGf563b362021-03-18 16:13:18 -06002545 } else if ((binding->descriptorType != VK_DESCRIPTOR_TYPE_MUTABLE_VALVE) &&
2546 (descriptor_types.find(binding->descriptorType) == descriptor_types.end())) {
locke-lunarg9a16ebb2020-07-30 16:56:33 -06002547 skip |= LogError(device, vuid_layout_mismatch,
Jeremy Gebben7fc88a22021-08-25 13:30:45 -06002548 "Type mismatch on descriptor slot %u.%u (expected `%s`) but descriptor of type %s", use.first.set,
2549 use.first.binding, string_descriptorTypes(descriptor_types).c_str(),
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002550 string_VkDescriptorType(binding->descriptorType));
Chris Forbes47567b72017-06-09 12:09:45 -07002551 } else if (binding->descriptorCount < required_descriptor_count) {
locke-lunarg9a16ebb2020-07-30 16:56:33 -06002552 skip |= LogError(device, vuid_layout_mismatch,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002553 "Shader expects at least %u descriptors for binding %u.%u but only %u provided",
Jeremy Gebben7fc88a22021-08-25 13:30:45 -06002554 required_descriptor_count, use.first.set, use.first.binding, binding->descriptorCount);
Chris Forbes47567b72017-06-09 12:09:45 -07002555 }
2556 }
2557
2558 // Validate use of input attachments against subpass structure
2559 if (pStage->stage == VK_SHADER_STAGE_FRAGMENT_BIT) {
sfricke-samsung962cad92021-04-13 00:46:29 -07002560 auto input_attachment_uses = module->CollectInterfaceByInputAttachmentIndex(accessible_ids);
Chris Forbes47567b72017-06-09 12:09:45 -07002561
amhagana448ea52021-11-02 14:09:14 -04002562 if (!pipeline->rp_state->use_dynamic_rendering) {
2563 auto rpci = pipeline->rp_state->createInfo.ptr();
2564 auto subpass = pipeline->create_info.graphics.subpass;
2565 for (auto use : input_attachment_uses) {
2566 auto input_attachments = rpci->pSubpasses[subpass].pInputAttachments;
2567 auto index = (input_attachments && use.first < rpci->pSubpasses[subpass].inputAttachmentCount)
2568 ? input_attachments[use.first].attachment
2569 : VK_ATTACHMENT_UNUSED;
Chris Forbes47567b72017-06-09 12:09:45 -07002570
amhagana448ea52021-11-02 14:09:14 -04002571 if (index == VK_ATTACHMENT_UNUSED) {
2572 skip |= LogError(device, kVUID_Core_Shader_MissingInputAttachment,
2573 "Shader consumes input attachment index %d but not provided in subpass", use.first);
2574 }
2575 else if (!(GetFormatType(rpci->pAttachments[index].format) & module->GetFundamentalType(use.second.type_id))) {
2576 skip |=
2577 LogError(device, kVUID_Core_Shader_InputAttachmentTypeMismatch,
2578 "Subpass input attachment %u format of %s does not match type used in shader `%s`", use.first,
2579 string_VkFormat(rpci->pAttachments[index].format), module->DescribeType(use.second.type_id).c_str());
2580 }
Chris Forbes47567b72017-06-09 12:09:45 -07002581 }
2582 }
2583 }
Lockeaa8fdc02019-04-02 11:59:20 -06002584 if (pStage->stage == VK_SHADER_STAGE_COMPUTE_BIT) {
ziga-lunarg11fecb92021-09-20 16:48:06 +02002585 skip |= ValidateComputeWorkGroupSizes(module, entrypoint, stage_state);
Lockeaa8fdc02019-04-02 11:59:20 -06002586 }
ziga-lunarg73163742021-08-25 13:15:29 +02002587
Chris Forbes47567b72017-06-09 12:09:45 -07002588 return skip;
2589}
2590
Mark Lobodzinskid8d658e2020-01-30 15:05:51 -07002591bool CoreChecks::ValidateInterfaceBetweenStages(SHADER_MODULE_STATE const *producer, spirv_inst_iter producer_entrypoint,
2592 shader_stage_attributes const *producer_stage, SHADER_MODULE_STATE const *consumer,
2593 spirv_inst_iter consumer_entrypoint,
2594 shader_stage_attributes const *consumer_stage) const {
Chris Forbes47567b72017-06-09 12:09:45 -07002595 bool skip = false;
2596
2597 auto outputs =
sfricke-samsung962cad92021-04-13 00:46:29 -07002598 producer->CollectInterfaceByLocation(producer_entrypoint, spv::StorageClassOutput, producer_stage->arrayed_output);
2599 auto inputs = consumer->CollectInterfaceByLocation(consumer_entrypoint, spv::StorageClassInput, consumer_stage->arrayed_input);
Chris Forbes47567b72017-06-09 12:09:45 -07002600
2601 auto a_it = outputs.begin();
2602 auto b_it = inputs.begin();
2603
ziga-lunarg8346fe82021-08-22 17:30:50 +02002604 uint32_t a_component = 0;
2605 uint32_t b_component = 0;
2606
Chris Forbes47567b72017-06-09 12:09:45 -07002607 // Maps sorted by key (location); walk them together to find mismatches
2608 while ((outputs.size() > 0 && a_it != outputs.end()) || (inputs.size() && b_it != inputs.end())) {
2609 bool a_at_end = outputs.size() == 0 || a_it == outputs.end();
2610 bool b_at_end = inputs.size() == 0 || b_it == inputs.end();
2611 auto a_first = a_at_end ? std::make_pair(0u, 0u) : a_it->first;
2612 auto b_first = b_at_end ? std::make_pair(0u, 0u) : b_it->first;
2613
ziga-lunarg8346fe82021-08-22 17:30:50 +02002614 a_first.second += a_component;
2615 b_first.second += b_component;
2616
2617 const auto a_length = a_at_end ? 0 : producer->GetNumComponentsInBaseType(producer->get_def(a_it->second.type_id));
2618 const auto b_length = b_at_end ? 0 : consumer->GetNumComponentsInBaseType(consumer->get_def(b_it->second.type_id));
2619 assert(a_at_end || a_component < a_length);
2620 assert(b_at_end || b_component < b_length);
2621
Chris Forbes47567b72017-06-09 12:09:45 -07002622 if (b_at_end || ((!a_at_end) && (a_first < b_first))) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002623 skip |= LogPerformanceWarning(producer->vk_shader_module(), kVUID_Core_Shader_OutputNotConsumed,
ziga-lunarg8346fe82021-08-22 17:30:50 +02002624 "%s writes to output location %" PRIu32 ".%" PRIu32 " which is not consumed by %s",
2625 producer_stage->name, a_first.first, a_first.second, consumer_stage->name);
2626 if ((b_first.first > a_first.first) || b_at_end || (a_component + 1 == a_length)) {
2627 a_it++;
2628 a_component = 0;
2629 } else {
2630 a_component++;
2631 }
Chris Forbes47567b72017-06-09 12:09:45 -07002632 } else if (a_at_end || a_first > b_first) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002633 skip |= LogError(consumer->vk_shader_module(), kVUID_Core_Shader_InputNotProduced,
ziga-lunarg8346fe82021-08-22 17:30:50 +02002634 "%s consumes input location %" PRIu32 ".%" PRIu32 " which is not written by %s", consumer_stage->name,
2635 b_first.first, b_first.second, producer_stage->name);
2636 if ((a_first.first > b_first.first) || a_at_end || (b_component + 1 == b_length)) {
2637 b_it++;
2638 b_component = 0;
2639 } else {
2640 b_component++;
2641 }
Chris Forbes47567b72017-06-09 12:09:45 -07002642 } else {
2643 // subtleties of arrayed interfaces:
2644 // - if is_patch, then the member is not arrayed, even though the interface may be.
2645 // - if is_block_member, then the extra array level of an arrayed interface is not
2646 // expressed in the member type -- it's expressed in the block type.
ziga-lunarg8346fe82021-08-22 17:30:50 +02002647 if (!TypesMatch(producer, consumer, a_it->second.type_id, b_it->second.type_id)) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002648 skip |= LogError(producer->vk_shader_module(), kVUID_Core_Shader_InterfaceTypeMismatch,
ziga-lunarg8346fe82021-08-22 17:30:50 +02002649 "Type mismatch on location %" PRIu32 ".%" PRIu32 ": '%s' vs '%s'", a_first.first, a_first.second,
sfricke-samsung962cad92021-04-13 00:46:29 -07002650 producer->DescribeType(a_it->second.type_id).c_str(),
2651 consumer->DescribeType(b_it->second.type_id).c_str());
ziga-lunarg8346fe82021-08-22 17:30:50 +02002652 a_it++;
2653 b_it++;
2654 continue;
Chris Forbes47567b72017-06-09 12:09:45 -07002655 }
2656 if (a_it->second.is_patch != b_it->second.is_patch) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002657 skip |= LogError(producer->vk_shader_module(), kVUID_Core_Shader_InterfaceTypeMismatch,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002658 "Decoration mismatch on location %u.%u: is per-%s in %s stage but per-%s in %s stage",
2659 a_first.first, a_first.second, a_it->second.is_patch ? "patch" : "vertex", producer_stage->name,
2660 b_it->second.is_patch ? "patch" : "vertex", consumer_stage->name);
Chris Forbes47567b72017-06-09 12:09:45 -07002661 }
2662 if (a_it->second.is_relaxed_precision != b_it->second.is_relaxed_precision) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002663 skip |= LogError(producer->vk_shader_module(), kVUID_Core_Shader_InterfaceTypeMismatch,
ziga-lunarg8346fe82021-08-22 17:30:50 +02002664 "Decoration mismatch on location %" PRIu32 ".%" PRIu32 ": %s and %s stages differ in precision",
2665 a_first.first, a_first.second, producer_stage->name, consumer_stage->name);
Chris Forbes47567b72017-06-09 12:09:45 -07002666 }
ziga-lunarg8346fe82021-08-22 17:30:50 +02002667 uint32_t a_remaining = a_length - a_component;
2668 uint32_t b_remaining = b_length - b_component;
2669 if (a_remaining == b_remaining) { // Sizes match so we can advance both a_it and b_it
2670 a_it++;
2671 b_it++;
2672 a_component = 0;
2673 b_component = 0;
2674 } else if (a_remaining > b_remaining) { // a has more components remaining
2675 a_component += b_remaining;
2676 b_component = 0;
2677 b_it++;
2678 } else if (b_remaining > a_remaining) { // b has more components remaining
2679 b_component += a_remaining;
2680 a_component = 0;
2681 a_it++;
2682 }
Chris Forbes47567b72017-06-09 12:09:45 -07002683 }
2684 }
2685
Ari Suonpaa696b3432019-03-11 14:02:57 +02002686 if (consumer_stage->stage != VK_SHADER_STAGE_FRAGMENT_BIT) {
sfricke-samsung962cad92021-04-13 00:46:29 -07002687 auto builtins_producer = producer->CollectBuiltinBlockMembers(producer_entrypoint, spv::StorageClassOutput);
2688 auto builtins_consumer = consumer->CollectBuiltinBlockMembers(consumer_entrypoint, spv::StorageClassInput);
Ari Suonpaa696b3432019-03-11 14:02:57 +02002689
2690 if (!builtins_producer.empty() && !builtins_consumer.empty()) {
2691 if (builtins_producer.size() != builtins_consumer.size()) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002692 skip |= LogError(producer->vk_shader_module(), kVUID_Core_Shader_InterfaceTypeMismatch,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002693 "Number of elements inside builtin block differ between stages (%s %d vs %s %d).",
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07002694 producer_stage->name, static_cast<int>(builtins_producer.size()), consumer_stage->name,
2695 static_cast<int>(builtins_consumer.size()));
Ari Suonpaa696b3432019-03-11 14:02:57 +02002696 } else {
2697 auto it_producer = builtins_producer.begin();
2698 auto it_consumer = builtins_consumer.begin();
2699 while (it_producer != builtins_producer.end() && it_consumer != builtins_consumer.end()) {
2700 if (*it_producer != *it_consumer) {
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002701 skip |= LogError(producer->vk_shader_module(), kVUID_Core_Shader_InterfaceTypeMismatch,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07002702 "Builtin variable inside block doesn't match between %s and %s.", producer_stage->name,
2703 consumer_stage->name);
Ari Suonpaa696b3432019-03-11 14:02:57 +02002704 break;
2705 }
2706 it_producer++;
2707 it_consumer++;
2708 }
2709 }
2710 }
2711 }
2712
Chris Forbes47567b72017-06-09 12:09:45 -07002713 return skip;
2714}
2715
John Zulauf14c355b2019-06-27 16:09:37 -06002716static inline uint32_t DetermineFinalGeomStage(const PIPELINE_STATE *pipeline, const VkGraphicsPipelineCreateInfo *pCreateInfo) {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002717 uint32_t stage_mask = 0;
2718 if (pipeline->topology_at_rasterizer == VK_PRIMITIVE_TOPOLOGY_POINT_LIST) {
2719 for (uint32_t i = 0; i < pCreateInfo->stageCount; i++) {
2720 stage_mask |= pCreateInfo->pStages[i].stage;
2721 }
2722 // Determine which shader in which PointSize should be written (the final geometry stage)
Jeff Bolz105d6492018-09-29 15:46:44 -05002723 if (stage_mask & VK_SHADER_STAGE_MESH_BIT_NV) {
2724 stage_mask = VK_SHADER_STAGE_MESH_BIT_NV;
2725 } else if (stage_mask & VK_SHADER_STAGE_GEOMETRY_BIT) {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002726 stage_mask = VK_SHADER_STAGE_GEOMETRY_BIT;
2727 } else if (stage_mask & VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT) {
2728 stage_mask = VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT;
2729 } else if (stage_mask & VK_SHADER_STAGE_VERTEX_BIT) {
2730 stage_mask = VK_SHADER_STAGE_VERTEX_BIT;
Mark Lobodzinski2c984cc2018-07-31 09:57:46 -06002731 }
2732 }
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002733 return stage_mask;
Mark Lobodzinski2c984cc2018-07-31 09:57:46 -06002734}
2735
Chris Forbes47567b72017-06-09 12:09:45 -07002736// Validate that the shaders used by the given pipeline and store the active_slots
2737// that are actually used by the pipeline into pPipeline->active_slots
John Zulaufac4c6e12019-07-01 16:05:58 -06002738bool CoreChecks::ValidateGraphicsPipelineShaderState(const PIPELINE_STATE *pipeline) const {
Jeremy Gebben11af9792021-08-20 10:20:09 -06002739 const auto create_info = pipeline->create_info.graphics.ptr();
Chris Forbes47567b72017-06-09 12:09:45 -07002740
Chris Forbes47567b72017-06-09 12:09:45 -07002741 bool skip = false;
2742
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07002743 uint32_t pointlist_stage_mask = DetermineFinalGeomStage(pipeline, create_info);
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002744
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002745 const PipelineStageState *vertex_stage = nullptr, *fragment_stage = nullptr;
2746 for (auto &stage : pipeline->stage_state) {
2747 skip |= ValidatePipelineShaderStage(pipeline, stage, (pointlist_stage_mask == stage.stage_flag));
2748 if (stage.stage_flag == VK_SHADER_STAGE_VERTEX_BIT) {
2749 vertex_stage = &stage;
2750 }
2751 if (stage.stage_flag == VK_SHADER_STAGE_FRAGMENT_BIT) {
2752 fragment_stage = &stage;
2753 }
Chris Forbes47567b72017-06-09 12:09:45 -07002754 }
2755
2756 // if the shader stages are no good individually, cross-stage validation is pointless.
2757 if (skip) return true;
2758
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07002759 auto vi = create_info->pVertexInputState;
Chris Forbes47567b72017-06-09 12:09:45 -07002760
2761 if (vi) {
Mark Lobodzinskid8d658e2020-01-30 15:05:51 -07002762 skip |= ValidateViConsistency(vi);
Chris Forbes47567b72017-06-09 12:09:45 -07002763 }
2764
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002765 if (vertex_stage && vertex_stage->module->has_valid_spirv && !IsDynamic(pipeline, VK_DYNAMIC_STATE_VERTEX_INPUT_EXT)) {
2766 skip |= ValidateViAgainstVsInputs(vi, vertex_stage->module.get(), vertex_stage->entrypoint);
Chris Forbes47567b72017-06-09 12:09:45 -07002767 }
2768
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002769 for (size_t i = 1; i < pipeline->stage_state.size(); i++) {
2770 const auto &producer = pipeline->stage_state[i - 1];
2771 const auto &consumer = pipeline->stage_state[i];
2772 assert(producer.module);
2773 if (&producer == fragment_stage) {
2774 break;
2775 }
2776 if (consumer.module) {
2777 if (consumer.module->has_valid_spirv && producer.module->has_valid_spirv) {
2778 auto producer_id = GetShaderStageId(producer.stage_flag);
2779 auto consumer_id = GetShaderStageId(consumer.stage_flag);
2780 skip |=
2781 ValidateInterfaceBetweenStages(producer.module.get(), producer.entrypoint, &shader_stage_attribs[producer_id],
2782 consumer.module.get(), consumer.entrypoint, &shader_stage_attribs[consumer_id]);
Chris Forbesdbb43fc2018-02-16 16:59:23 -08002783 }
Chris Forbes47567b72017-06-09 12:09:45 -07002784 }
2785 }
2786
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002787 if (fragment_stage && fragment_stage->module->has_valid_spirv) {
2788 skip |= ValidateFsOutputsAgainstRenderPass(fragment_stage->module.get(), fragment_stage->entrypoint, pipeline,
Nathaniel Cesarioce9b4812020-12-17 08:55:28 -07002789 create_info->subpass);
Chris Forbes47567b72017-06-09 12:09:45 -07002790 }
2791
2792 return skip;
2793}
2794
Tony-LunarGb2ded512021-02-02 16:03:30 -07002795void CoreChecks::RecordGraphicsPipelineShaderDynamicState(PIPELINE_STATE *pipeline_state) {
Tony-LunarGb2ded512021-02-02 16:03:30 -07002796 if (phys_dev_ext_props.fragment_shading_rate_props.primitiveFragmentShadingRateWithMultipleViewports ||
2797 !IsDynamic(pipeline_state, VK_DYNAMIC_STATE_VIEWPORT_WITH_COUNT_EXT)) {
2798 return;
2799 }
Tobias Hector6663c9b2020-11-05 10:18:02 +00002800
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002801 for (auto &stage : pipeline_state->stage_state) {
2802 if (stage.stage_flag == VK_SHADER_STAGE_VERTEX_BIT || stage.stage_flag == VK_SHADER_STAGE_GEOMETRY_BIT ||
2803 stage.stage_flag == VK_SHADER_STAGE_MESH_BIT_NV) {
Tony-LunarGb2ded512021-02-02 16:03:30 -07002804 bool primitiverate_written = false;
Tobias Hector6663c9b2020-11-05 10:18:02 +00002805
Nathaniel Cesario77cd59b2021-10-11 23:52:24 -06002806 for (const auto &set : stage.module->GetBuiltinDecorationList()) {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002807 auto insn = stage.module->at(set.offset);
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002808 if (set.builtin == spv::BuiltInPrimitiveShadingRateKHR) {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002809 primitiverate_written = stage.module->IsBuiltInWritten(insn, stage.entrypoint);
Tobias Hector6663c9b2020-11-05 10:18:02 +00002810 }
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002811 if (primitiverate_written) {
2812 break;
2813 }
Tony-LunarGb2ded512021-02-02 16:03:30 -07002814 }
sfricke-samsungc0eb5282021-02-28 23:05:55 -08002815
Tony-LunarGb2ded512021-02-02 16:03:30 -07002816 if (primitiverate_written) {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002817 pipeline_state->wrote_primitive_shading_rate.insert(stage.stage_flag);
Tony-LunarGb2ded512021-02-02 16:03:30 -07002818 }
2819 }
2820 }
2821}
2822
2823bool CoreChecks::ValidateGraphicsPipelineShaderDynamicState(const PIPELINE_STATE *pipeline, const CMD_BUFFER_STATE *pCB,
2824 const char *caller, const DrawDispatchVuid &vuid) const {
Tony-LunarGb2ded512021-02-02 16:03:30 -07002825 bool skip = false;
2826
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002827 for (auto &stage : pipeline->stage_state) {
2828 if (stage.stage_flag == VK_SHADER_STAGE_VERTEX_BIT || stage.stage_flag == VK_SHADER_STAGE_GEOMETRY_BIT ||
2829 stage.stage_flag == VK_SHADER_STAGE_MESH_BIT_NV) {
Tony-LunarGb2ded512021-02-02 16:03:30 -07002830 if (!phys_dev_ext_props.fragment_shading_rate_props.primitiveFragmentShadingRateWithMultipleViewports &&
2831 IsDynamic(pipeline, VK_DYNAMIC_STATE_VIEWPORT_WITH_COUNT_EXT) && pCB->viewportWithCountCount != 1) {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002832 if (pipeline->wrote_primitive_shading_rate.find(stage.stage_flag) != pipeline->wrote_primitive_shading_rate.end()) {
Tobias Hector6663c9b2020-11-05 10:18:02 +00002833 skip |=
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06002834 LogError(pipeline->pipeline(), vuid.viewport_count_primitive_shading_rate,
Tobias Hector6663c9b2020-11-05 10:18:02 +00002835 "%s: %s shader of currently bound pipeline statically writes to PrimitiveShadingRateKHR built-in"
2836 "but multiple viewports are set by the last call to vkCmdSetViewportWithCountEXT,"
2837 "and the primitiveFragmentShadingRateWithMultipleViewports limit is not supported.",
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002838 caller, string_VkShaderStageFlagBits(stage.stage_flag));
Tobias Hector6663c9b2020-11-05 10:18:02 +00002839 }
2840 }
2841 }
2842 }
2843
2844 return skip;
2845}
2846
sfricke-samsunge72a85e2020-02-29 21:48:37 -08002847bool CoreChecks::ValidateComputePipelineShaderState(PIPELINE_STATE *pipeline) const {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002848 return ValidatePipelineShaderStage(pipeline, pipeline->stage_state[0], false);
Chris Forbes47567b72017-06-09 12:09:45 -07002849}
Chris Forbes4ae55b32017-06-09 14:42:56 -07002850
Ricardo Garcia6f3477e2020-10-21 10:58:53 +02002851uint32_t CoreChecks::CalcShaderStageCount(const PIPELINE_STATE *pipeline, VkShaderStageFlagBits stageBit) const {
2852 uint32_t total = 0;
Jeremy Gebben11af9792021-08-20 10:20:09 -06002853 const auto &create_info = pipeline->create_info.raytracing;
2854 const auto *stages = create_info.ptr()->pStages;
2855 for (uint32_t stage_index = 0; stage_index < create_info.stageCount; stage_index++) {
Ricardo Garcia6f3477e2020-10-21 10:58:53 +02002856 if (stages[stage_index].stage == stageBit) {
2857 total++;
2858 }
2859 }
2860
Jeremy Gebben11af9792021-08-20 10:20:09 -06002861 if (create_info.pLibraryInfo) {
2862 for (uint32_t i = 0; i < create_info.pLibraryInfo->libraryCount; ++i) {
Jeremy Gebbenb20a8242021-11-05 15:14:43 -06002863 const auto library_pipeline = Get<PIPELINE_STATE>(create_info.pLibraryInfo->pLibraries[i]);
Ricardo Garcia6f3477e2020-10-21 10:58:53 +02002864 total += CalcShaderStageCount(library_pipeline, stageBit);
2865 }
2866 }
2867
2868 return total;
2869}
2870
Sebastian Neubauer7c826172021-10-04 12:05:51 +02002871bool CoreChecks::GroupHasValidIndex(const PIPELINE_STATE *pipeline, uint32_t group, uint32_t stage) const {
2872 if (group == VK_SHADER_UNUSED_NV) {
2873 return true;
2874 }
2875
2876 const auto &create_info = pipeline->create_info.raytracing;
2877 const auto *stages = create_info.ptr()->pStages;
2878
2879 if (group < create_info.stageCount) {
2880 return (stages[group].stage & stage) != 0;
2881 }
2882 group -= create_info.stageCount;
2883
2884 // Search libraries
2885 if (create_info.pLibraryInfo) {
2886 for (uint32_t i = 0; i < create_info.pLibraryInfo->libraryCount; ++i) {
Jeremy Gebbenb20a8242021-11-05 15:14:43 -06002887 const auto library_pipeline = Get<PIPELINE_STATE>(create_info.pLibraryInfo->pLibraries[i]);
Sebastian Neubauer7c826172021-10-04 12:05:51 +02002888 const uint32_t stage_count = library_pipeline->create_info.raytracing.ptr()->stageCount;
2889 if (group < stage_count) {
2890 return (library_pipeline->create_info.raytracing.ptr()->pStages[group].stage & stage) != 0;
2891 }
2892 group -= stage_count;
2893 }
2894 }
2895
2896 // group index too large
2897 return false;
2898}
2899
sourav parmarcd5fb182020-07-17 12:58:44 -07002900bool CoreChecks::ValidateRayTracingPipeline(PIPELINE_STATE *pipeline, VkPipelineCreateFlags flags, bool isKHR) const {
John Zulaufe4474e72019-07-01 17:28:27 -06002901 bool skip = false;
Jason Macnak15f95e82019-08-21 21:52:02 -04002902
Jeremy Gebben11af9792021-08-20 10:20:09 -06002903 const auto &create_info = pipeline->create_info.raytracing;
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002904 if (isKHR) {
Jeremy Gebben11af9792021-08-20 10:20:09 -06002905 if (create_info.maxPipelineRayRecursionDepth > phys_dev_ext_props.ray_tracing_propsKHR.maxRayRecursionDepth) {
2906 skip |=
2907 LogError(device, "VUID-VkRayTracingPipelineCreateInfoKHR-maxPipelineRayRecursionDepth-03589",
2908 "vkCreateRayTracingPipelinesKHR: maxPipelineRayRecursionDepth (%d ) must be less than or equal to "
2909 "VkPhysicalDeviceRayTracingPipelinePropertiesKHR::maxRayRecursionDepth %d",
2910 create_info.maxPipelineRayRecursionDepth, phys_dev_ext_props.ray_tracing_propsKHR.maxRayRecursionDepth);
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002911 }
Jeremy Gebben11af9792021-08-20 10:20:09 -06002912 if (create_info.pLibraryInfo) {
2913 for (uint32_t i = 0; i < create_info.pLibraryInfo->libraryCount; ++i) {
Jeremy Gebbenb20a8242021-11-05 15:14:43 -06002914 const auto library_pipelinestate = Get<PIPELINE_STATE>(create_info.pLibraryInfo->pLibraries[i]);
Jeremy Gebben11af9792021-08-20 10:20:09 -06002915 const auto &library_create_info = library_pipelinestate->create_info.raytracing;
2916 if (library_create_info.maxPipelineRayRecursionDepth != create_info.maxPipelineRayRecursionDepth) {
sourav parmarcd5fb182020-07-17 12:58:44 -07002917 skip |= LogError(
2918 device, "VUID-VkRayTracingPipelineCreateInfoKHR-pLibraries-03591",
2919 "vkCreateRayTracingPipelinesKHR: Each element (%d) of the pLibraries member of libraries must have been"
2920 "created with the value of maxPipelineRayRecursionDepth (%d) equal to that in this pipeline (%d) .",
Jeremy Gebben11af9792021-08-20 10:20:09 -06002921 i, library_create_info.maxPipelineRayRecursionDepth, create_info.maxPipelineRayRecursionDepth);
sourav parmarcd5fb182020-07-17 12:58:44 -07002922 }
Jeremy Gebben11af9792021-08-20 10:20:09 -06002923 if (library_create_info.pLibraryInfo && (library_create_info.pLibraryInterface->maxPipelineRayHitAttributeSize !=
2924 create_info.pLibraryInterface->maxPipelineRayHitAttributeSize ||
2925 library_create_info.pLibraryInterface->maxPipelineRayPayloadSize !=
2926 create_info.pLibraryInterface->maxPipelineRayPayloadSize)) {
sourav parmarcd5fb182020-07-17 12:58:44 -07002927 skip |= LogError(device, "VUID-VkRayTracingPipelineCreateInfoKHR-pLibraryInfo-03593",
2928 "vkCreateRayTracingPipelinesKHR: If pLibraryInfo is not NULL, each element of its pLibraries "
2929 "member must have been created with values of the maxPipelineRayPayloadSize and "
2930 "maxPipelineRayHitAttributeSize members of pLibraryInterface equal to those in this pipeline");
2931 }
2932 if ((flags & VK_PIPELINE_CREATE_RAY_TRACING_SHADER_GROUP_HANDLE_CAPTURE_REPLAY_BIT_KHR) &&
Jeremy Gebben11af9792021-08-20 10:20:09 -06002933 !(library_create_info.flags & VK_PIPELINE_CREATE_RAY_TRACING_SHADER_GROUP_HANDLE_CAPTURE_REPLAY_BIT_KHR)) {
sourav parmarcd5fb182020-07-17 12:58:44 -07002934 skip |= LogError(device, "VUID-VkRayTracingPipelineCreateInfoKHR-flags-03594",
2935 "vkCreateRayTracingPipelinesKHR: If flags includes "
2936 "VK_PIPELINE_CREATE_RAY_TRACING_SHADER_GROUP_HANDLE_CAPTURE_REPLAY_BIT_KHR, each element of "
2937 "the pLibraries member of libraries must have been created with the "
2938 "VK_PIPELINE_CREATE_RAY_TRACING_SHADER_GROUP_HANDLE_CAPTURE_REPLAY_BIT_KHR bit set");
2939 }
sourav parmar83c31b12020-05-06 12:30:54 -07002940 }
2941 }
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002942 } else {
Jeremy Gebben11af9792021-08-20 10:20:09 -06002943 if (create_info.maxRecursionDepth > phys_dev_ext_props.ray_tracing_propsNV.maxRecursionDepth) {
sourav parmarcd5fb182020-07-17 12:58:44 -07002944 skip |= LogError(device, "VUID-VkRayTracingPipelineCreateInfoNV-maxRecursionDepth-03457",
2945 "vkCreateRayTracingPipelinesNV: maxRecursionDepth (%d) must be less than or equal to "
2946 "VkPhysicalDeviceRayTracingPropertiesNV::maxRecursionDepth (%d)",
Jeremy Gebben11af9792021-08-20 10:20:09 -06002947 create_info.maxRecursionDepth, phys_dev_ext_props.ray_tracing_propsNV.maxRecursionDepth);
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002948 }
Jason Macnak15f95e82019-08-21 21:52:02 -04002949 }
Jeremy Gebben11af9792021-08-20 10:20:09 -06002950 const auto *groups = create_info.ptr()->pGroups;
Jason Macnak15f95e82019-08-21 21:52:02 -04002951
Jeremy Gebben11af9792021-08-20 10:20:09 -06002952 for (uint32_t stage_index = 0; stage_index < create_info.stageCount; stage_index++) {
Jeremy Gebben84b838b2021-08-23 08:41:39 -06002953 skip |= ValidatePipelineShaderStage(pipeline, pipeline->stage_state[stage_index], false);
Jason Macnak15f95e82019-08-21 21:52:02 -04002954 }
Ricardo Garcia6f3477e2020-10-21 10:58:53 +02002955
Jeremy Gebben11af9792021-08-20 10:20:09 -06002956 if ((create_info.flags & VK_PIPELINE_CREATE_LIBRARY_BIT_KHR) == 0) {
Ricardo Garcia6f3477e2020-10-21 10:58:53 +02002957 const uint32_t raygen_stages_count = CalcShaderStageCount(pipeline, VK_SHADER_STAGE_RAYGEN_BIT_KHR);
2958 if (raygen_stages_count == 0) {
2959 skip |= LogError(
2960 device,
Mike Schuchardt7b152fa2021-08-03 16:30:27 -07002961 isKHR ? "VUID-VkRayTracingPipelineCreateInfoKHR-stage-03425" : "VUID-VkRayTracingPipelineCreateInfoNV-stage-06232",
Ricardo Garcia6f3477e2020-10-21 10:58:53 +02002962 " : The stage member of at least one element of pStages must be VK_SHADER_STAGE_RAYGEN_BIT_KHR.");
2963 }
Jason Macnak15f95e82019-08-21 21:52:02 -04002964 }
2965
Jeremy Gebben11af9792021-08-20 10:20:09 -06002966 for (uint32_t group_index = 0; group_index < create_info.groupCount; group_index++) {
Jason Macnak15f95e82019-08-21 21:52:02 -04002967 const auto &group = groups[group_index];
2968
2969 if (group.type == VK_RAY_TRACING_SHADER_GROUP_TYPE_GENERAL_NV) {
Sebastian Neubauer7c826172021-10-04 12:05:51 +02002970 if (!GroupHasValidIndex(
2971 pipeline, group.generalShader,
2972 VK_SHADER_STAGE_RAYGEN_BIT_NV | VK_SHADER_STAGE_MISS_BIT_NV | VK_SHADER_STAGE_CALLABLE_BIT_NV)) {
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002973 skip |= LogError(device,
2974 isKHR ? "VUID-VkRayTracingShaderGroupCreateInfoKHR-type-03474"
2975 : "VUID-VkRayTracingShaderGroupCreateInfoNV-type-02413",
2976 ": pGroups[%d]", group_index);
Jason Macnak15f95e82019-08-21 21:52:02 -04002977 }
2978 if (group.anyHitShader != VK_SHADER_UNUSED_NV || group.closestHitShader != VK_SHADER_UNUSED_NV ||
2979 group.intersectionShader != VK_SHADER_UNUSED_NV) {
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002980 skip |= LogError(device,
2981 isKHR ? "VUID-VkRayTracingShaderGroupCreateInfoKHR-type-03475"
2982 : "VUID-VkRayTracingShaderGroupCreateInfoNV-type-02414",
2983 ": pGroups[%d]", group_index);
Jason Macnak15f95e82019-08-21 21:52:02 -04002984 }
2985 } else if (group.type == VK_RAY_TRACING_SHADER_GROUP_TYPE_PROCEDURAL_HIT_GROUP_NV) {
Sebastian Neubauer7c826172021-10-04 12:05:51 +02002986 if (!GroupHasValidIndex(pipeline, group.intersectionShader, VK_SHADER_STAGE_INTERSECTION_BIT_NV)) {
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002987 skip |= LogError(device,
2988 isKHR ? "VUID-VkRayTracingShaderGroupCreateInfoKHR-type-03476"
2989 : "VUID-VkRayTracingShaderGroupCreateInfoNV-type-02415",
2990 ": pGroups[%d]", group_index);
Jason Macnak15f95e82019-08-21 21:52:02 -04002991 }
2992 } else if (group.type == VK_RAY_TRACING_SHADER_GROUP_TYPE_TRIANGLES_HIT_GROUP_NV) {
2993 if (group.intersectionShader != VK_SHADER_UNUSED_NV) {
Jeff Bolz443c2ca2020-03-19 12:11:51 -05002994 skip |= LogError(device,
2995 isKHR ? "VUID-VkRayTracingShaderGroupCreateInfoKHR-type-03477"
2996 : "VUID-VkRayTracingShaderGroupCreateInfoNV-type-02416",
2997 ": pGroups[%d]", group_index);
Jason Macnak15f95e82019-08-21 21:52:02 -04002998 }
2999 }
3000
3001 if (group.type == VK_RAY_TRACING_SHADER_GROUP_TYPE_PROCEDURAL_HIT_GROUP_NV ||
3002 group.type == VK_RAY_TRACING_SHADER_GROUP_TYPE_TRIANGLES_HIT_GROUP_NV) {
Sebastian Neubauer7c826172021-10-04 12:05:51 +02003003 if (!GroupHasValidIndex(pipeline, group.anyHitShader, VK_SHADER_STAGE_ANY_HIT_BIT_NV)) {
Jeff Bolz443c2ca2020-03-19 12:11:51 -05003004 skip |= LogError(device,
3005 isKHR ? "VUID-VkRayTracingShaderGroupCreateInfoKHR-anyHitShader-03479"
3006 : "VUID-VkRayTracingShaderGroupCreateInfoNV-anyHitShader-02418",
3007 ": pGroups[%d]", group_index);
Jason Macnak15f95e82019-08-21 21:52:02 -04003008 }
Sebastian Neubauer7c826172021-10-04 12:05:51 +02003009 if (!GroupHasValidIndex(pipeline, group.closestHitShader, VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV)) {
Jeff Bolz443c2ca2020-03-19 12:11:51 -05003010 skip |= LogError(device,
3011 isKHR ? "VUID-VkRayTracingShaderGroupCreateInfoKHR-closestHitShader-03478"
3012 : "VUID-VkRayTracingShaderGroupCreateInfoNV-closestHitShader-02417",
3013 ": pGroups[%d]", group_index);
Jason Macnak15f95e82019-08-21 21:52:02 -04003014 }
3015 }
John Zulaufe4474e72019-07-01 17:28:27 -06003016 }
3017 return skip;
Jeff Bolzfbe51582018-09-13 10:01:35 -05003018}
3019
Dave Houltona9df0ce2018-02-07 10:51:23 -07003020uint32_t ValidationCache::MakeShaderHash(VkShaderModuleCreateInfo const *smci) { return XXH32(smci->pCode, smci->codeSize, 0); }
Chris Forbes9a61e082017-07-24 15:35:29 -07003021
Dave Houltona9df0ce2018-02-07 10:51:23 -07003022static ValidationCache *GetValidationCacheInfo(VkShaderModuleCreateInfo const *pCreateInfo) {
Mark Lobodzinski1f887d32020-12-30 15:31:33 -07003023 const auto validation_cache_ci = LvlFindInChain<VkShaderModuleValidationCacheCreateInfoEXT>(pCreateInfo->pNext);
John Zulauf25ea2432019-04-05 10:07:38 -06003024 if (validation_cache_ci) {
John Zulauf146ee802019-04-05 15:31:06 -06003025 return CastFromHandle<ValidationCache *>(validation_cache_ci->validationCache);
Chris Forbes9a61e082017-07-24 15:35:29 -07003026 }
Chris Forbes9a61e082017-07-24 15:35:29 -07003027 return nullptr;
3028}
3029
Mark Lobodzinskib56bbb92019-02-18 11:49:59 -07003030bool CoreChecks::PreCallValidateCreateShaderModule(VkDevice device, const VkShaderModuleCreateInfo *pCreateInfo,
Jeff Bolz5c801d12019-10-09 10:38:45 -05003031 const VkAllocationCallbacks *pAllocator, VkShaderModule *pShaderModule) const {
Chris Forbes4ae55b32017-06-09 14:42:56 -07003032 bool skip = false;
3033 spv_result_t spv_valid = SPV_SUCCESS;
Chris Forbes4ae55b32017-06-09 14:42:56 -07003034
Mark Lobodzinski90eea5b2020-05-15 12:54:00 -06003035 if (disabled[shader_validation]) {
Chris Forbes4ae55b32017-06-09 14:42:56 -07003036 return false;
3037 }
3038
sfricke-samsung45996a42021-09-16 13:45:27 -07003039 auto have_glsl_shader = IsExtEnabled(device_extensions.vk_nv_glsl_shader);
Chris Forbes4ae55b32017-06-09 14:42:56 -07003040
3041 if (!have_glsl_shader && (pCreateInfo->codeSize % 4)) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003042 skip |= LogError(device, "VUID-VkShaderModuleCreateInfo-pCode-01376",
3043 "SPIR-V module not valid: Codesize must be a multiple of 4 but is " PRINTF_SIZE_T_SPECIFIER ".",
3044 pCreateInfo->codeSize);
Chris Forbes4ae55b32017-06-09 14:42:56 -07003045 } else {
Chris Forbes9a61e082017-07-24 15:35:29 -07003046 auto cache = GetValidationCacheInfo(pCreateInfo);
3047 uint32_t hash = 0;
Tony-LunarG55fdf1e2021-01-13 14:32:56 -07003048 // If app isn't using a shader validation cache, use the default one from CoreChecks
3049 if (!cache) cache = CastFromHandle<ValidationCache *>(core_validation_cache);
Chris Forbes9a61e082017-07-24 15:35:29 -07003050 if (cache) {
3051 hash = ValidationCache::MakeShaderHash(pCreateInfo);
Dave Houltona9df0ce2018-02-07 10:51:23 -07003052 if (cache->Contains(hash)) return false;
Chris Forbes9a61e082017-07-24 15:35:29 -07003053 }
3054
Jeremy Hayesb3e4d532019-08-16 10:08:49 -06003055 // Use SPIRV-Tools validator to try and catch any issues with the module itself. If specialization constants are present,
3056 // the default values will be used during validation.
sfricke-samsung45996a42021-09-16 13:45:27 -07003057 spv_target_env spirv_environment = PickSpirvEnv(api_version, IsExtEnabled(device_extensions.vk_khr_spirv_1_4));
Dave Houlton0ea2d012018-06-21 14:00:26 -06003058 spv_context ctx = spvContextCreate(spirv_environment);
Dave Houltona9df0ce2018-02-07 10:51:23 -07003059 spv_const_binary_t binary{pCreateInfo->pCode, pCreateInfo->codeSize / sizeof(uint32_t)};
Chris Forbes4ae55b32017-06-09 14:42:56 -07003060 spv_diagnostic diag = nullptr;
Tony-LunarG9fe69a42020-07-23 15:09:37 -06003061 spvtools::ValidatorOptions options;
3062 AdjustValidatorOptions(device_extensions, enabled_features, options);
Karl Schultzfda1b382018-08-08 18:56:11 -06003063 spv_valid = spvValidateWithOptions(ctx, options, &binary, &diag);
Chris Forbes4ae55b32017-06-09 14:42:56 -07003064 if (spv_valid != SPV_SUCCESS) {
3065 if (!have_glsl_shader || (pCreateInfo->pCode[0] == spv::MagicNumber)) {
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003066 if (spv_valid == SPV_WARNING) {
3067 skip |= LogWarning(device, kVUID_Core_Shader_InconsistentSpirv, "SPIR-V module not valid: %s",
3068 diag && diag->error ? diag->error : "(no error text)");
3069 } else {
3070 skip |= LogError(device, kVUID_Core_Shader_InconsistentSpirv, "SPIR-V module not valid: %s",
3071 diag && diag->error ? diag->error : "(no error text)");
3072 }
Chris Forbes4ae55b32017-06-09 14:42:56 -07003073 }
Chris Forbes9a61e082017-07-24 15:35:29 -07003074 } else {
3075 if (cache) {
3076 cache->Insert(hash);
3077 }
Chris Forbes4ae55b32017-06-09 14:42:56 -07003078 }
3079
3080 spvDiagnosticDestroy(diag);
3081 spvContextDestroy(ctx);
3082 }
3083
Chris Forbes4ae55b32017-06-09 14:42:56 -07003084 return skip;
Mark Lobodzinski01734072019-02-13 17:39:15 -07003085}
3086
ziga-lunarg11fecb92021-09-20 16:48:06 +02003087bool CoreChecks::ValidateComputeWorkGroupSizes(const SHADER_MODULE_STATE *shader, const spirv_inst_iter &entrypoint,
3088 const PipelineStageState &stage_state) const {
Lockeaa8fdc02019-04-02 11:59:20 -06003089 bool skip = false;
3090 uint32_t local_size_x = 0;
3091 uint32_t local_size_y = 0;
3092 uint32_t local_size_z = 0;
sfricke-samsung962cad92021-04-13 00:46:29 -07003093 if (shader->FindLocalSize(entrypoint, local_size_x, local_size_y, local_size_z)) {
Lockeaa8fdc02019-04-02 11:59:20 -06003094 if (local_size_x > phys_dev_props.limits.maxComputeWorkGroupSize[0]) {
Mike Schuchardt37b8cc12021-10-05 14:31:11 -07003095 skip |= LogError(shader->vk_shader_module(), "VUID-RuntimeSpirv-x-06429",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003096 "%s local_size_x (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[0] (%" PRIu32 ").",
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06003097 report_data->FormatHandle(shader->vk_shader_module()).c_str(), local_size_x,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003098 phys_dev_props.limits.maxComputeWorkGroupSize[0]);
Lockeaa8fdc02019-04-02 11:59:20 -06003099 }
3100 if (local_size_y > phys_dev_props.limits.maxComputeWorkGroupSize[1]) {
Mike Schuchardt37b8cc12021-10-05 14:31:11 -07003101 skip |= LogError(shader->vk_shader_module(), "VUID-RuntimeSpirv-y-06430",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003102 "%s local_size_y (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[1] (%" PRIu32 ").",
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06003103 report_data->FormatHandle(shader->vk_shader_module()).c_str(), local_size_x,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003104 phys_dev_props.limits.maxComputeWorkGroupSize[1]);
Lockeaa8fdc02019-04-02 11:59:20 -06003105 }
3106 if (local_size_z > phys_dev_props.limits.maxComputeWorkGroupSize[2]) {
Mike Schuchardt37b8cc12021-10-05 14:31:11 -07003107 skip |= LogError(shader->vk_shader_module(), "VUID-RuntimeSpirv-z-06431",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003108 "%s local_size_z (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[2] (%" PRIu32 ").",
Jeremy Gebben14b0d1a2021-05-15 20:15:41 -06003109 report_data->FormatHandle(shader->vk_shader_module()).c_str(), local_size_x,
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003110 phys_dev_props.limits.maxComputeWorkGroupSize[2]);
Lockeaa8fdc02019-04-02 11:59:20 -06003111 }
3112
3113 uint32_t limit = phys_dev_props.limits.maxComputeWorkGroupInvocations;
3114 uint64_t invocations = local_size_x * local_size_y;
3115 // Prevent overflow.
3116 bool fail = false;
3117 if (invocations > UINT32_MAX || invocations > limit) {
3118 fail = true;
3119 }
3120 if (!fail) {
3121 invocations *= local_size_z;
3122 if (invocations > UINT32_MAX || invocations > limit) {
3123 fail = true;
3124 }
3125 }
3126 if (fail) {
Mike Schuchardt37b8cc12021-10-05 14:31:11 -07003127 skip |= LogError(shader->vk_shader_module(), "VUID-RuntimeSpirv-x-06432",
Mark Lobodzinski12b9be92020-01-30 15:25:55 -07003128 "%s local_size (%" PRIu32 ", %" PRIu32 ", %" PRIu32
3129 ") exceeds device limit maxComputeWorkGroupInvocations (%" PRIu32 ").",
sfricke-samsung1ff329f2021-09-16 10:06:47 -07003130 report_data->FormatHandle(shader->vk_shader_module()).c_str(), local_size_x, local_size_y,
3131 local_size_z, limit);
Lockeaa8fdc02019-04-02 11:59:20 -06003132 }
ziga-lunarg11fecb92021-09-20 16:48:06 +02003133
3134 const auto subgroup_flags = VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT |
3135 VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT;
3136 if ((stage_state.create_info->flags & subgroup_flags) == subgroup_flags) {
3137 if (SafeModulo(local_size_x, phys_dev_ext_props.subgroup_size_control_props.maxSubgroupSize) != 0) {
3138 skip |= LogError(
3139 shader->vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-flags-02758",
3140 "%s flags contain VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT and "
3141 "VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT bits, but local workgroup size in the X "
3142 "dimension (%" PRIu32
3143 ") is not a multiple of VkPhysicalDeviceSubgroupSizeControlPropertiesEXT::maxSubgroupSize (%" PRIu32 ").",
3144 report_data->FormatHandle(shader->vk_shader_module()).c_str(), local_size_x,
3145 phys_dev_ext_props.subgroup_size_control_props.maxSubgroupSize);
3146 }
3147 } else if ((stage_state.create_info->flags & VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT) &&
3148 (stage_state.create_info->flags & VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT) == 0) {
3149 const auto *required_subgroup_size_features =
3150 LvlFindInChain<VkPipelineShaderStageRequiredSubgroupSizeCreateInfoEXT>(stage_state.create_info->pNext);
3151 if (!required_subgroup_size_features) {
3152 if (SafeModulo(local_size_x, phys_dev_props_core11.subgroupSize) != 0) {
3153 skip |= LogError(
3154 shader->vk_shader_module(), "VUID-VkPipelineShaderStageCreateInfo-flags-02759",
3155 "%s flags contain VK_PIPELINE_SHADER_STAGE_CREATE_REQUIRE_FULL_SUBGROUPS_BIT_EXT bit, and not the"
3156 "VK_PIPELINE_SHADER_STAGE_CREATE_ALLOW_VARYING_SUBGROUP_SIZE_BIT_EXT bit, but local workgroup size in the "
3157 "X dimension (%" PRIu32 ") is not a multiple of VkPhysicalDeviceVulkan11Properties::subgroupSize (%" PRIu32
3158 ").",
3159 report_data->FormatHandle(shader->vk_shader_module()).c_str(), local_size_x,
3160 phys_dev_props_core11.subgroupSize);
3161 }
3162 }
3163 }
Lockeaa8fdc02019-04-02 11:59:20 -06003164 }
3165 return skip;
3166}
Tony-LunarG8a51b7d2020-07-01 15:57:23 -06003167
3168spv_target_env PickSpirvEnv(uint32_t api_version, bool spirv_1_4) {
3169 if (api_version >= VK_API_VERSION_1_2) {
3170 return SPV_ENV_VULKAN_1_2;
3171 } else if (api_version >= VK_API_VERSION_1_1) {
3172 if (spirv_1_4) {
3173 return SPV_ENV_VULKAN_1_1_SPIRV_1_4;
3174 } else {
3175 return SPV_ENV_VULKAN_1_1;
3176 }
3177 }
3178 return SPV_ENV_VULKAN_1_0;
3179}
Tony-LunarG9fe69a42020-07-23 15:09:37 -06003180
sfricke-samsungecc112a2021-09-03 05:32:17 -07003181// Some Vulkan extensions/features are just all done in spirv-val behind optional settings
Jeremy Gebben5d970742021-05-31 16:04:14 -06003182void AdjustValidatorOptions(const DeviceExtensions &device_extensions, const DeviceFeatures &enabled_features,
Tony-LunarG9fe69a42020-07-23 15:09:37 -06003183 spvtools::ValidatorOptions &options) {
sfricke-samsungecc112a2021-09-03 05:32:17 -07003184 // VK_KHR_relaxed_block_layout never had a feature bit so just enabling the extension allows relaxed layout
3185 // Was promotoed in Vulkan 1.1 so anyone using Vulkan 1.1 also gets this for free
sfricke-samsung45996a42021-09-16 13:45:27 -07003186 if (IsExtEnabled(device_extensions.vk_khr_relaxed_block_layout)) {
sfricke-samsungecc112a2021-09-03 05:32:17 -07003187 // --relax-block-layout
Tony-LunarG9fe69a42020-07-23 15:09:37 -06003188 options.SetRelaxBlockLayout(true);
3189 }
sfricke-samsungecc112a2021-09-03 05:32:17 -07003190
3191 // The rest of the settings are controlled from a feature bit, which are set correctly in the state tracking. Regardless of
3192 // Vulkan version used, the feature bit is needed (also described in the spec).
3193
3194 if (enabled_features.core12.uniformBufferStandardLayout == VK_TRUE) {
3195 // --uniform-buffer-standard-layout
Tony-LunarG9fe69a42020-07-23 15:09:37 -06003196 options.SetUniformBufferStandardLayout(true);
3197 }
sfricke-samsungecc112a2021-09-03 05:32:17 -07003198 if (enabled_features.core12.scalarBlockLayout == VK_TRUE) {
3199 // --scalar-block-layout
Tony-LunarG9fe69a42020-07-23 15:09:37 -06003200 options.SetScalarBlockLayout(true);
3201 }
sfricke-samsungecc112a2021-09-03 05:32:17 -07003202 if (enabled_features.workgroup_memory_explicit_layout_features.workgroupMemoryExplicitLayoutScalarBlockLayout) {
3203 // --workgroup-scalar-block-layout
Caio Marcelo de Oliveira Filhod1bfbcd2021-01-27 01:44:04 -08003204 options.SetWorkgroupScalarBlockLayout(true);
3205 }
sfricke-samsungd3c917b2021-10-19 08:24:57 -07003206 if (enabled_features.maintenance4_features.maintenance4) {
3207 // --allow-localsizeid
3208 options.SetAllowLocalSizeId(true);
3209 }
Tony-LunarG9fe69a42020-07-23 15:09:37 -06003210}