Tony-LunarG | 7371999 | 2020-01-15 10:20:28 -0700 | [diff] [blame] | 1 | /* Copyright (c) 2015-2020 The Khronos Group Inc. |
| 2 | * Copyright (c) 2015-2020 Valve Corporation |
| 3 | * Copyright (c) 2015-2020 LunarG, Inc. |
| 4 | * Copyright (C) 2015-2020 Google Inc. |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 5 | * |
| 6 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 7 | * you may not use this file except in compliance with the License. |
| 8 | * You may obtain a copy of the License at |
| 9 | * |
| 10 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 11 | * |
| 12 | * Unless required by applicable law or agreed to in writing, software |
| 13 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 14 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 15 | * See the License for the specific language governing permissions and |
| 16 | * limitations under the License. |
| 17 | * |
| 18 | * Author: Cody Northrop <cnorthrop@google.com> |
| 19 | * Author: Michael Lentine <mlentine@google.com> |
| 20 | * Author: Tobin Ehlis <tobine@google.com> |
| 21 | * Author: Chia-I Wu <olv@google.com> |
| 22 | * Author: Chris Forbes <chrisf@ijw.co.nz> |
| 23 | * Author: Mark Lobodzinski <mark@lunarg.com> |
| 24 | * Author: Ian Elliott <ianelliott@google.com> |
| 25 | * Author: Dave Houlton <daveh@lunarg.com> |
| 26 | * Author: Dustin Graves <dustin@lunarg.com> |
| 27 | * Author: Jeremy Hayes <jeremy@lunarg.com> |
| 28 | * Author: Jon Ashburn <jon@lunarg.com> |
| 29 | * Author: Karl Schultz <karl@lunarg.com> |
| 30 | * Author: Mark Young <marky@lunarg.com> |
| 31 | * Author: Mike Schuchardt <mikes@lunarg.com> |
| 32 | * Author: Mike Weiblen <mikew@lunarg.com> |
| 33 | * Author: Tony Barbour <tony@LunarG.com> |
| 34 | * Author: John Zulauf <jzulauf@lunarg.com> |
| 35 | * Author: Shannon McPherson <shannon@lunarg.com> |
| 36 | */ |
| 37 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 38 | #include "chassis.h" |
| 39 | #include "core_validation.h" |
| 40 | |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 41 | // This LUT is created to allow a static listing of each VUID that is covered by drawdispatch commands |
| 42 | // clang-format off |
| 43 | static const std::map<CMD_TYPE, DrawDispatchVuid> drawdispatch_vuid = { |
| 44 | {CMD_DRAW, { |
| 45 | "VUID-vkCmdDraw-commandBuffer-cmdpool", |
| 46 | "VUID-vkCmdDraw-renderpass", |
| 47 | "VUID-vkCmdDraw-None-02700", |
| 48 | "VUID-vkCmdDraw-commandBuffer-02701", |
| 49 | "VUID-vkCmdDraw-None-02720", |
| 50 | "VUID-vkCmdDraw-None-02697", |
| 51 | "VUID-vkCmdDraw-renderPass-02684", |
| 52 | "VUID-vkCmdDraw-subpass-02685" |
| 53 | }}, |
| 54 | {CMD_DRAWINDEXED, { |
| 55 | "VUID-vkCmdDrawIndexed-commandBuffer-cmdpool", |
| 56 | "VUID-vkCmdDrawIndexed-renderpass", |
| 57 | "VUID-vkCmdDrawIndexed-None-02700", |
| 58 | "VUID-vkCmdDrawIndexed-commandBuffer-02701", |
| 59 | "VUID-vkCmdDrawIndexed-None-02720", |
| 60 | "VUID-vkCmdDrawIndexed-None-02697", |
| 61 | "VUID-vkCmdDrawIndexed-renderPass-02684", |
| 62 | "VUID-vkCmdDrawIndexed-subpass-02685" |
| 63 | }}, |
| 64 | {CMD_DRAWINDIRECT, { |
| 65 | "VUID-vkCmdDrawIndirect-commandBuffer-cmdpool", |
| 66 | "VUID-vkCmdDrawIndirect-renderpass", |
| 67 | "VUID-vkCmdDrawIndirect-None-02700", |
| 68 | "VUID-vkCmdDrawIndirect-commandBuffer-02701", |
| 69 | "VUID-vkCmdDrawIndirect-None-02720", |
| 70 | "VUID-vkCmdDrawIndirect-None-02697", |
| 71 | "VUID-vkCmdDrawIndirect-renderPass-02684", |
| 72 | "VUID-vkCmdDrawIndirect-subpass-02685" |
| 73 | }}, |
| 74 | {CMD_DRAWINDEXEDINDIRECT, { |
| 75 | "VUID-vkCmdDrawIndexedIndirect-commandBuffer-cmdpool", |
| 76 | "VUID-vkCmdDrawIndexedIndirect-renderpass", |
| 77 | "VUID-vkCmdDrawIndexedIndirect-None-02700", |
| 78 | "VUID-vkCmdDrawIndexedIndirect-commandBuffer-02701", |
| 79 | "VUID-vkCmdDrawIndexedIndirect-None-02720", |
| 80 | "VUID-vkCmdDrawIndexedIndirect-None-02697", |
| 81 | "VUID-vkCmdDrawIndexedIndirect-renderPass-02684", |
| 82 | "VUID-vkCmdDrawIndexedIndirect-subpass-02685" |
| 83 | }}, |
| 84 | {CMD_DISPATCH, { |
| 85 | "VUID-vkCmdDispatch-commandBuffer-cmdpool", |
| 86 | "VUID-vkCmdDispatch-renderpass", |
| 87 | "VUID-vkCmdDispatch-None-02700", |
| 88 | kVUIDUndefined, // dynamic_state |
| 89 | kVUIDUndefined, // vertex_binding |
| 90 | "VUID-vkCmdDispatch-None-02697", |
| 91 | kVUIDUndefined, // render_pass_compatible |
| 92 | kVUIDUndefined, // subpass_index |
| 93 | }}, |
| 94 | {CMD_DISPATCHINDIRECT, { |
| 95 | "VUID-vkCmdDispatchIndirect-commandBuffer-cmdpool", |
| 96 | "VUID-vkCmdDispatchIndirect-renderpass", |
| 97 | "VUID-vkCmdDispatchIndirect-None-02700", |
| 98 | kVUIDUndefined, // dynamic_state |
| 99 | kVUIDUndefined, // vertex_binding |
| 100 | "VUID-vkCmdDispatchIndirect-None-02697", |
| 101 | kVUIDUndefined, // render_pass_compatible |
| 102 | kVUIDUndefined, // subpass_index |
| 103 | }}, |
| 104 | {CMD_DRAWINDIRECTCOUNT, { |
| 105 | "VUID-vkCmdDrawIndirectCount-commandBuffer-cmdpool", |
| 106 | "VUID-vkCmdDrawIndirectCount-renderpass", |
| 107 | "VUID-vkCmdDrawIndirectCount-None-02700", |
| 108 | "VUID-vkCmdDrawIndirectCount-commandBuffer-02701", |
| 109 | "VUID-vkCmdDrawIndirectCount-None-02720", |
| 110 | "VUID-vkCmdDrawIndirectCount-None-02697", |
| 111 | "VUID-vkCmdDrawIndirectCount-renderPass-02684", |
| 112 | "VUID-vkCmdDrawIndirectCount-subpass-02685" |
| 113 | }}, |
| 114 | {CMD_DRAWINDEXEDINDIRECTCOUNT,{ |
| 115 | "VUID-vkCmdDrawIndexedIndirectCount-commandBuffer-cmdpool", |
| 116 | "VUID-vkCmdDrawIndexedIndirectCount-renderpass", |
| 117 | "VUID-vkCmdDrawIndexedIndirectCount-None-02700", |
| 118 | "VUID-vkCmdDrawIndexedIndirectCount-commandBuffer-02701", |
| 119 | "VUID-vkCmdDrawIndexedIndirectCount-None-02720", |
| 120 | "VUID-vkCmdDrawIndexedIndirectCount-None-02697", |
| 121 | "VUID-vkCmdDrawIndexedIndirectCount-renderPass-02684", |
| 122 | "VUID-vkCmdDrawIndexedIndirectCount-subpass-02685" |
| 123 | }}, |
| 124 | {CMD_TRACERAYSNV, { |
| 125 | "VUID-vkCmdTraceRaysNV-commandBuffer-cmdpool", |
| 126 | "VUID-vkCmdTraceRaysNV-renderpass", |
| 127 | "VUID-vkCmdTraceRaysNV-None-02700", |
| 128 | "VUID-vkCmdTraceRaysNV-commandBuffer-02701", |
| 129 | kVUIDUndefined, // vertex_binding |
| 130 | "VUID-vkCmdTraceRaysNV-None-02697", |
| 131 | kVUIDUndefined, // render_pass_compatible |
| 132 | kVUIDUndefined, // subpass_index |
| 133 | }}, |
Jeff Bolz | 443c2ca | 2020-03-19 12:11:51 -0500 | [diff] [blame^] | 134 | {CMD_TRACERAYSKHR, { |
| 135 | "VUID-vkCmdTraceRaysKHR-commandBuffer-cmdpool", |
| 136 | "VUID-vkCmdTraceRaysKHR-renderpass", |
| 137 | "VUID-vkCmdTraceRaysKHR-None-02700", |
| 138 | "VUID-vkCmdTraceRaysKHR-commandBuffer-02701", |
| 139 | kVUIDUndefined, // vertex_binding |
| 140 | "VUID-vkCmdTraceRaysKHR-None-02697", |
| 141 | kVUIDUndefined, // render_pass_compatible |
| 142 | kVUIDUndefined, // subpass_index |
| 143 | }}, |
| 144 | {CMD_TRACERAYSINDIRECTKHR, { |
| 145 | "VUID-vkCmdTraceRaysIndirectKHR-commandBuffer-cmdpool", |
| 146 | "VUID-vkCmdTraceRaysIndirectKHR-renderpass", |
| 147 | "VUID-vkCmdTraceRaysIndirectKHR-None-02700", |
| 148 | "VUID-vkCmdTraceRaysIndirectKHR-commandBuffer-02701", |
| 149 | kVUIDUndefined, // vertex_binding |
| 150 | "VUID-vkCmdTraceRaysIndirectKHR-None-02697", |
| 151 | kVUIDUndefined, // render_pass_compatible |
| 152 | kVUIDUndefined, // subpass_index |
| 153 | }}, |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 154 | {CMD_DRAWMESHTASKSNV, { |
| 155 | "VUID-vkCmdDrawMeshTasksNV-commandBuffer-cmdpool", |
| 156 | "VUID-vkCmdDrawMeshTasksNV-renderpass", |
| 157 | "VUID-vkCmdDrawMeshTasksNV-None-02700", |
| 158 | "VUID-vkCmdDrawMeshTasksNV-commandBuffer-02701", |
| 159 | kVUIDUndefined, // vertex_binding |
| 160 | "VUID-vkCmdDrawMeshTasksNV-None-02697", |
| 161 | "VUID-vkCmdDrawMeshTasksNV-renderPass-02684", |
| 162 | "VUID-vkCmdDrawMeshTasksNV-subpass-02685" |
| 163 | }}, |
| 164 | {CMD_DRAWMESHTASKSINDIRECTNV, { |
| 165 | "VUID-vkCmdDrawMeshTasksIndirectNV-commandBuffer-cmdpool", |
| 166 | "VUID-vkCmdDrawMeshTasksIndirectNV-renderpass", |
| 167 | "VUID-vkCmdDrawMeshTasksIndirectNV-None-02700", |
| 168 | "VUID-vkCmdDrawMeshTasksIndirectNV-commandBuffer-02701", |
| 169 | kVUIDUndefined, // vertex_binding |
| 170 | "VUID-vkCmdDrawMeshTasksIndirectNV-None-02697", |
| 171 | "VUID-vkCmdDrawMeshTasksIndirectNV-renderPass-02684", |
| 172 | "VUID-vkCmdDrawMeshTasksIndirectNV-subpass-02685" |
| 173 | }}, |
| 174 | {CMD_DRAWMESHTASKSINDIRECTCOUNTNV, { |
| 175 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-commandBuffer-cmdpool", |
| 176 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-renderpass", |
| 177 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-None-02700", |
| 178 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-commandBuffer-02701", |
| 179 | kVUIDUndefined, // vertex_binding |
| 180 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-None-02697", |
| 181 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-renderPass-02684", |
| 182 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-subpass-02685" |
| 183 | }}, |
| 184 | // Used if invalid cmd_type is used |
| 185 | {CMD_NONE, {kVUIDUndefined, kVUIDUndefined, kVUIDUndefined, kVUIDUndefined, kVUIDUndefined, kVUIDUndefined, kVUIDUndefined, kVUIDUndefined}} |
| 186 | }; |
| 187 | // clang-format on |
| 188 | |
| 189 | // Getter function to provide kVUIDUndefined in case an invalid cmd_type is passed in |
| 190 | const DrawDispatchVuid &CoreChecks::GetDrawDispatchVuid(CMD_TYPE cmd_type) const { |
| 191 | if (drawdispatch_vuid.find(cmd_type) != drawdispatch_vuid.cend()) { |
| 192 | return drawdispatch_vuid.at(cmd_type); |
| 193 | } else { |
| 194 | return drawdispatch_vuid.at(CMD_NONE); |
| 195 | } |
| 196 | } |
| 197 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 198 | // Generic function to handle validation for all CmdDraw* type functions |
| 199 | bool CoreChecks::ValidateCmdDrawType(VkCommandBuffer cmd_buffer, bool indexed, VkPipelineBindPoint bind_point, CMD_TYPE cmd_type, |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 200 | const char *caller, VkQueueFlags queue_flags) const { |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 201 | bool skip = false; |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 202 | const DrawDispatchVuid vuid = GetDrawDispatchVuid(cmd_type); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 203 | const CMD_BUFFER_STATE *cb_state = GetCBState(cmd_buffer); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 204 | if (cb_state) { |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 205 | skip |= ValidateCmdQueueFlags(cb_state, caller, queue_flags, vuid.queue_flag); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 206 | skip |= ValidateCmd(cb_state, cmd_type, caller); |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 207 | skip |= ValidateCmdBufDrawState(cb_state, cmd_type, indexed, bind_point, caller); |
| 208 | skip |= (VK_PIPELINE_BIND_POINT_GRAPHICS == bind_point) ? OutsideRenderPass(cb_state, caller, vuid.inside_renderpass) |
| 209 | : InsideRenderPass(cb_state, caller, vuid.inside_renderpass); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 210 | } |
| 211 | return skip; |
| 212 | } |
| 213 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 214 | bool CoreChecks::PreCallValidateCmdDraw(VkCommandBuffer commandBuffer, uint32_t vertexCount, uint32_t instanceCount, |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 215 | uint32_t firstVertex, uint32_t firstInstance) const { |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 216 | return ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAW, "vkCmdDraw()", |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 217 | VK_QUEUE_GRAPHICS_BIT); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 218 | } |
| 219 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 220 | bool CoreChecks::PreCallValidateCmdDrawIndexed(VkCommandBuffer commandBuffer, uint32_t indexCount, uint32_t instanceCount, |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 221 | uint32_t firstIndex, int32_t vertexOffset, uint32_t firstInstance) const { |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 222 | bool skip = ValidateCmdDrawType(commandBuffer, true, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWINDEXED, "vkCmdDrawIndexed()", |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 223 | VK_QUEUE_GRAPHICS_BIT); |
Jeff Bolz | 46c0ea0 | 2019-10-09 13:06:29 -0500 | [diff] [blame] | 224 | const CMD_BUFFER_STATE *cb_state = GetCBState(commandBuffer); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 225 | if (!skip && (cb_state->status & CBSTATUS_INDEX_BUFFER_BOUND)) { |
| 226 | unsigned int index_size = 0; |
| 227 | const auto &index_buffer_binding = cb_state->index_buffer_binding; |
| 228 | if (index_buffer_binding.index_type == VK_INDEX_TYPE_UINT16) { |
| 229 | index_size = 2; |
| 230 | } else if (index_buffer_binding.index_type == VK_INDEX_TYPE_UINT32) { |
| 231 | index_size = 4; |
Piers Daniell | 5070e3e | 2019-08-20 13:39:35 -0600 | [diff] [blame] | 232 | } else if (index_buffer_binding.index_type == VK_INDEX_TYPE_UINT8_EXT) { |
| 233 | index_size = 1; |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 234 | } |
| 235 | VkDeviceSize end_offset = (index_size * ((VkDeviceSize)firstIndex + indexCount)) + index_buffer_binding.offset; |
| 236 | if (end_offset > index_buffer_binding.size) { |
Mark Lobodzinski | cf0e6d3 | 2020-01-30 12:12:58 -0700 | [diff] [blame] | 237 | skip |= |
| 238 | LogError(index_buffer_binding.buffer, "VUID-vkCmdDrawIndexed-indexSize-00463", |
| 239 | "vkCmdDrawIndexed() index size (%d) * (firstIndex (%d) + indexCount (%d)) " |
| 240 | "+ binding offset (%" PRIuLEAST64 ") = an ending offset of %" PRIuLEAST64 |
| 241 | " bytes, which is greater than the index buffer size (%" PRIuLEAST64 ").", |
| 242 | index_size, firstIndex, indexCount, index_buffer_binding.offset, end_offset, index_buffer_binding.size); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 243 | } |
| 244 | } |
| 245 | return skip; |
| 246 | } |
| 247 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 248 | bool CoreChecks::PreCallValidateCmdDrawIndirect(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, uint32_t count, |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 249 | uint32_t stride) const { |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 250 | bool skip = ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWINDIRECT, "vkCmdDrawIndirect()", |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 251 | VK_QUEUE_GRAPHICS_BIT); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 252 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 253 | skip |= ValidateMemoryIsBoundToBuffer(buffer_state, "vkCmdDrawIndirect()", "VUID-vkCmdDrawIndirect-buffer-02708"); |
| 254 | skip |= ValidateBufferUsageFlags(buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, "VUID-vkCmdDrawIndirect-buffer-02709", |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 255 | "vkCmdDrawIndirect()", "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 256 | if (count > 1) { |
| 257 | skip |= ValidateCmdDrawStrideWithStruct(commandBuffer, "VUID-vkCmdDrawIndirect-drawCount-00476", stride, |
| 258 | "VkDrawIndirectCommand", sizeof(VkDrawIndirectCommand)); |
| 259 | skip |= |
| 260 | ValidateCmdDrawStrideWithBuffer(commandBuffer, "VUID-vkCmdDrawIndirect-drawCount-00488", stride, |
| 261 | "VkDrawIndirectCommand", sizeof(VkDrawIndirectCommand), count, offset, buffer_state); |
| 262 | } |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 263 | // TODO: If the drawIndirectFirstInstance feature is not enabled, all the firstInstance members of the |
| 264 | // VkDrawIndirectCommand structures accessed by this command must be 0, which will require access to the contents of 'buffer'. |
| 265 | return skip; |
| 266 | } |
| 267 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 268 | bool CoreChecks::PreCallValidateCmdDrawIndexedIndirect(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 269 | uint32_t count, uint32_t stride) const { |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 270 | bool skip = ValidateCmdDrawType(commandBuffer, true, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWINDEXEDINDIRECT, |
| 271 | "vkCmdDrawIndexedIndirect()", VK_QUEUE_GRAPHICS_BIT); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 272 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 273 | skip |= ValidateMemoryIsBoundToBuffer(buffer_state, "vkCmdDrawIndexedIndirect()", "VUID-vkCmdDrawIndexedIndirect-buffer-02708"); |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 274 | skip |= ValidateBufferUsageFlags(buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 275 | "VUID-vkCmdDrawIndexedIndirect-buffer-02709", "vkCmdDrawIndexedIndirect()", |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 276 | "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 277 | if (count > 1) { |
| 278 | skip |= ValidateCmdDrawStrideWithStruct(commandBuffer, "VUID-vkCmdDrawIndexedIndirect-drawCount-00528", stride, |
| 279 | "VkDrawIndexedIndirectCommand", sizeof(VkDrawIndexedIndirectCommand)); |
| 280 | skip |= ValidateCmdDrawStrideWithBuffer(commandBuffer, "VUID-vkCmdDrawIndexedIndirect-drawCount-00540", stride, |
| 281 | "VkDrawIndexedIndirectCommand", sizeof(VkDrawIndexedIndirectCommand), count, offset, |
| 282 | buffer_state); |
| 283 | } |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 284 | // TODO: If the drawIndirectFirstInstance feature is not enabled, all the firstInstance members of the |
| 285 | // VkDrawIndexedIndirectCommand structures accessed by this command must be 0, which will require access to the contents of |
| 286 | // 'buffer'. |
| 287 | return skip; |
| 288 | } |
| 289 | |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 290 | bool CoreChecks::PreCallValidateCmdDispatch(VkCommandBuffer commandBuffer, uint32_t x, uint32_t y, uint32_t z) const { |
Locke | f39c077 | 2019-04-03 14:40:02 -0600 | [diff] [blame] | 291 | bool skip = false; |
Locke | f39c077 | 2019-04-03 14:40:02 -0600 | [diff] [blame] | 292 | skip |= ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_COMPUTE, CMD_DISPATCH, "vkCmdDispatch()", |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 293 | VK_QUEUE_COMPUTE_BIT); |
Locke | f39c077 | 2019-04-03 14:40:02 -0600 | [diff] [blame] | 294 | return skip; |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 295 | } |
| 296 | |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 297 | bool CoreChecks::PreCallValidateCmdDispatchIndirect(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset) const { |
Mark Lobodzinski | 708aae7 | 2019-12-20 11:57:46 -0700 | [diff] [blame] | 298 | bool skip = ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_COMPUTE, CMD_DISPATCHINDIRECT, |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 299 | "vkCmdDispatchIndirect()", VK_QUEUE_COMPUTE_BIT); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 300 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 301 | skip |= ValidateMemoryIsBoundToBuffer(buffer_state, "vkCmdDispatchIndirect()", "VUID-vkCmdDispatchIndirect-buffer-02708"); |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 302 | skip |= |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 303 | ValidateBufferUsageFlags(buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, "VUID-vkCmdDispatchIndirect-buffer-02709", |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 304 | "vkCmdDispatchIndirect()", "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 305 | return skip; |
| 306 | } |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 307 | bool CoreChecks::ValidateCmdDrawIndirectCount(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
| 308 | VkBuffer countBuffer, VkDeviceSize countBufferOffset, uint32_t maxDrawCount, |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 309 | uint32_t stride, const char *apiName) const { |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 310 | bool skip = false; |
| 311 | if (offset & 3) { |
Mark Lobodzinski | cf0e6d3 | 2020-01-30 12:12:58 -0700 | [diff] [blame] | 312 | skip |= LogError(commandBuffer, "VUID-vkCmdDrawIndirectCount-offset-02710", |
| 313 | "%s() parameter, VkDeviceSize offset (0x%" PRIxLEAST64 "), is not a multiple of 4.", apiName, offset); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 314 | } |
| 315 | |
| 316 | if (countBufferOffset & 3) { |
Mark Lobodzinski | cf0e6d3 | 2020-01-30 12:12:58 -0700 | [diff] [blame] | 317 | skip |= LogError(commandBuffer, "VUID-vkCmdDrawIndirectCount-countBufferOffset-02716", |
| 318 | "%s() parameter, VkDeviceSize countBufferOffset (0x%" PRIxLEAST64 "), is not a multiple of 4.", apiName, |
| 319 | countBufferOffset); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 320 | } |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 321 | skip |= ValidateCmdDrawStrideWithStruct(commandBuffer, "VUID-vkCmdDrawIndirectCount-stride-03110", stride, apiName, |
| 322 | sizeof(VkDrawIndirectCommand)); |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 323 | if (maxDrawCount > 1) { |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 324 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 325 | skip |= ValidateCmdDrawStrideWithBuffer(commandBuffer, "VUID-vkCmdDrawIndirectCount-maxDrawCount-03111", stride, apiName, |
| 326 | sizeof(VkDrawIndirectCommand), maxDrawCount, offset, buffer_state); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 327 | } |
| 328 | |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 329 | skip |= ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWINDIRECTCOUNT, apiName, |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 330 | VK_QUEUE_GRAPHICS_BIT); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 331 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
| 332 | const BUFFER_STATE *count_buffer_state = GetBufferState(countBuffer); |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 333 | skip |= ValidateMemoryIsBoundToBuffer(buffer_state, apiName, "VUID-vkCmdDrawIndirectCount-buffer-02708"); |
| 334 | skip |= ValidateMemoryIsBoundToBuffer(count_buffer_state, apiName, "VUID-vkCmdDrawIndirectCount-countBuffer-02714"); |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 335 | skip |= ValidateBufferUsageFlags(buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 336 | "VUID-vkCmdDrawIndirectCount-buffer-02709", apiName, "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
| 337 | skip |= |
| 338 | ValidateBufferUsageFlags(count_buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
| 339 | "VUID-vkCmdDrawIndirectCount-countBuffer-02715", apiName, "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 340 | return skip; |
| 341 | } |
| 342 | |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 343 | bool CoreChecks::PreCallValidateCmdDrawIndirectCountKHR(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
| 344 | VkBuffer countBuffer, VkDeviceSize countBufferOffset, uint32_t maxDrawCount, |
| 345 | uint32_t stride) const { |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 346 | return ValidateCmdDrawIndirectCount(commandBuffer, buffer, offset, countBuffer, countBufferOffset, maxDrawCount, stride, |
| 347 | "VkCmdDrawIndirectCountKHR"); |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 348 | } |
| 349 | |
| 350 | bool CoreChecks::PreCallValidateCmdDrawIndirectCount(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
| 351 | VkBuffer countBuffer, VkDeviceSize countBufferOffset, uint32_t maxDrawCount, |
| 352 | uint32_t stride) const { |
Tony-LunarG | 83544c7 | 2020-01-17 12:30:19 -0700 | [diff] [blame] | 353 | return ValidateCmdDrawIndirectCount(commandBuffer, buffer, offset, countBuffer, countBufferOffset, maxDrawCount, stride, |
| 354 | "VkCmdDrawIndirectCount"); |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 355 | } |
| 356 | |
| 357 | bool CoreChecks::ValidateCmdDrawIndexedIndirectCount(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
| 358 | VkBuffer countBuffer, VkDeviceSize countBufferOffset, uint32_t maxDrawCount, |
| 359 | uint32_t stride) const { |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 360 | bool skip = false; |
| 361 | if (offset & 3) { |
Mark Lobodzinski | cf0e6d3 | 2020-01-30 12:12:58 -0700 | [diff] [blame] | 362 | skip |= LogError( |
| 363 | commandBuffer, "VUID-vkCmdDrawIndexedIndirectCount-offset-02710", |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 364 | "vkCmdDrawIndexedIndirectCount() parameter, VkDeviceSize offset (0x%" PRIxLEAST64 "), is not a multiple of 4.", offset); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 365 | } |
| 366 | |
| 367 | if (countBufferOffset & 3) { |
Mark Lobodzinski | cf0e6d3 | 2020-01-30 12:12:58 -0700 | [diff] [blame] | 368 | skip |= LogError(commandBuffer, "VUID-vkCmdDrawIndexedIndirectCount-countBufferOffset-02716", |
| 369 | "vkCmdDrawIndexedIndirectCount() parameter, VkDeviceSize countBufferOffset (0x%" PRIxLEAST64 |
| 370 | "), is not a multiple of 4.", |
| 371 | countBufferOffset); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 372 | } |
| 373 | |
Mike Schuchardt | 65847d9 | 2019-12-20 13:50:47 -0800 | [diff] [blame] | 374 | skip |= ValidateCmdDrawStrideWithStruct(commandBuffer, "VUID-vkCmdDrawIndexedIndirectCount-stride-03142", stride, |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 375 | "VkDrawIndirectCommand", sizeof(VkDrawIndexedIndirectCommand)); |
| 376 | if (maxDrawCount > 1) { |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 377 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
Mike Schuchardt | 65847d9 | 2019-12-20 13:50:47 -0800 | [diff] [blame] | 378 | skip |= ValidateCmdDrawStrideWithBuffer(commandBuffer, "VUID-vkCmdDrawIndexedIndirectCount-maxDrawCount-03143", stride, |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 379 | "VkDrawIndirectCommand", sizeof(VkDrawIndexedIndirectCommand), maxDrawCount, offset, |
| 380 | buffer_state); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 381 | } |
| 382 | |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 383 | skip |= ValidateCmdDrawType(commandBuffer, true, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWINDEXEDINDIRECTCOUNT, |
| 384 | "vkCmdDrawIndexedIndirectCount()", VK_QUEUE_GRAPHICS_BIT); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 385 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
| 386 | const BUFFER_STATE *count_buffer_state = GetBufferState(countBuffer); |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 387 | skip |= ValidateMemoryIsBoundToBuffer(buffer_state, "vkCmdDrawIndexedIndirectCount()", |
Mike Schuchardt | 65847d9 | 2019-12-20 13:50:47 -0800 | [diff] [blame] | 388 | "VUID-vkCmdDrawIndexedIndirectCount-buffer-02708"); |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 389 | skip |= ValidateMemoryIsBoundToBuffer(count_buffer_state, "vkCmdDrawIndexedIndirectCount()", |
Mike Schuchardt | 65847d9 | 2019-12-20 13:50:47 -0800 | [diff] [blame] | 390 | "VUID-vkCmdDrawIndexedIndirectCount-countBuffer-02714"); |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 391 | skip |= ValidateBufferUsageFlags(buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 392 | "VUID-vkCmdDrawIndexedIndirectCount-buffer-02709", "vkCmdDrawIndexedIndirectCount()", |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 393 | "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
| 394 | skip |= ValidateBufferUsageFlags(count_buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 395 | "VUID-vkCmdDrawIndexedIndirectCount-countBuffer-02715", "vkCmdDrawIndexedIndirectCount()", |
Mike Schuchardt | 65847d9 | 2019-12-20 13:50:47 -0800 | [diff] [blame] | 396 | "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 397 | return skip; |
| 398 | } |
| 399 | |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 400 | bool CoreChecks::PreCallValidateCmdDrawIndexedIndirectCountKHR(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
| 401 | VkBuffer countBuffer, VkDeviceSize countBufferOffset, |
| 402 | uint32_t maxDrawCount, uint32_t stride) const { |
| 403 | return ValidateCmdDrawIndexedIndirectCount(commandBuffer, buffer, offset, countBuffer, countBufferOffset, maxDrawCount, stride); |
| 404 | } |
| 405 | |
| 406 | bool CoreChecks::PreCallValidateCmdDrawIndexedIndirectCount(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
| 407 | VkBuffer countBuffer, VkDeviceSize countBufferOffset, |
| 408 | uint32_t maxDrawCount, uint32_t stride) const { |
| 409 | return ValidateCmdDrawIndexedIndirectCount(commandBuffer, buffer, offset, countBuffer, countBufferOffset, maxDrawCount, stride); |
| 410 | } |
| 411 | |
Jeff Bolz | f6e872c | 2019-10-22 12:17:28 -0500 | [diff] [blame] | 412 | bool CoreChecks::PreCallValidateCmdTraceRaysNV(VkCommandBuffer commandBuffer, VkBuffer raygenShaderBindingTableBuffer, |
| 413 | VkDeviceSize raygenShaderBindingOffset, VkBuffer missShaderBindingTableBuffer, |
| 414 | VkDeviceSize missShaderBindingOffset, VkDeviceSize missShaderBindingStride, |
| 415 | VkBuffer hitShaderBindingTableBuffer, VkDeviceSize hitShaderBindingOffset, |
| 416 | VkDeviceSize hitShaderBindingStride, VkBuffer callableShaderBindingTableBuffer, |
| 417 | VkDeviceSize callableShaderBindingOffset, VkDeviceSize callableShaderBindingStride, |
| 418 | uint32_t width, uint32_t height, uint32_t depth) const { |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 419 | bool skip = ValidateCmdDrawType(commandBuffer, true, VK_PIPELINE_BIND_POINT_RAY_TRACING_NV, CMD_TRACERAYSNV, |
| 420 | "vkCmdTraceRaysNV()", VK_QUEUE_COMPUTE_BIT); |
Jeff Bolz | f6e872c | 2019-10-22 12:17:28 -0500 | [diff] [blame] | 421 | return skip; |
| 422 | } |
| 423 | |
Jason Macnak | 67407e7 | 2019-07-11 11:05:09 -0700 | [diff] [blame] | 424 | void CoreChecks::PostCallRecordCmdTraceRaysNV(VkCommandBuffer commandBuffer, VkBuffer raygenShaderBindingTableBuffer, |
| 425 | VkDeviceSize raygenShaderBindingOffset, VkBuffer missShaderBindingTableBuffer, |
| 426 | VkDeviceSize missShaderBindingOffset, VkDeviceSize missShaderBindingStride, |
| 427 | VkBuffer hitShaderBindingTableBuffer, VkDeviceSize hitShaderBindingOffset, |
| 428 | VkDeviceSize hitShaderBindingStride, VkBuffer callableShaderBindingTableBuffer, |
| 429 | VkDeviceSize callableShaderBindingOffset, VkDeviceSize callableShaderBindingStride, |
| 430 | uint32_t width, uint32_t height, uint32_t depth) { |
| 431 | CMD_BUFFER_STATE *cb_state = GetCBState(commandBuffer); |
| 432 | UpdateStateCmdDrawDispatchType(cb_state, VK_PIPELINE_BIND_POINT_RAY_TRACING_NV); |
| 433 | cb_state->hasTraceRaysCmd = true; |
| 434 | } |
| 435 | |
Jeff Bolz | 443c2ca | 2020-03-19 12:11:51 -0500 | [diff] [blame^] | 436 | bool CoreChecks::PreCallValidateCmdTraceRaysKHR(VkCommandBuffer commandBuffer, |
| 437 | const VkStridedBufferRegionKHR *pRaygenShaderBindingTable, |
| 438 | const VkStridedBufferRegionKHR *pMissShaderBindingTable, |
| 439 | const VkStridedBufferRegionKHR *pHitShaderBindingTable, |
| 440 | const VkStridedBufferRegionKHR *pCallableShaderBindingTable, uint32_t width, |
| 441 | uint32_t height, uint32_t depth) const { |
| 442 | bool skip = ValidateCmdDrawType(commandBuffer, true, VK_PIPELINE_BIND_POINT_RAY_TRACING_KHR, CMD_TRACERAYSKHR, |
| 443 | "vkCmdTraceRaysKHR()", VK_QUEUE_COMPUTE_BIT); |
| 444 | return skip; |
| 445 | } |
| 446 | |
| 447 | void CoreChecks::PostCallRecordCmdTraceRaysKHR(VkCommandBuffer commandBuffer, |
| 448 | const VkStridedBufferRegionKHR *pRaygenShaderBindingTable, |
| 449 | const VkStridedBufferRegionKHR *pMissShaderBindingTable, |
| 450 | const VkStridedBufferRegionKHR *pHitShaderBindingTable, |
| 451 | const VkStridedBufferRegionKHR *pCallableShaderBindingTable, uint32_t width, |
| 452 | uint32_t height, uint32_t depth) { |
| 453 | CMD_BUFFER_STATE *cb_state = GetCBState(commandBuffer); |
| 454 | UpdateStateCmdDrawDispatchType(cb_state, VK_PIPELINE_BIND_POINT_RAY_TRACING_KHR); |
| 455 | cb_state->hasTraceRaysCmd = true; |
| 456 | } |
| 457 | |
| 458 | bool CoreChecks::PreCallValidateCmdTraceRaysIndirectKHR(VkCommandBuffer commandBuffer, |
| 459 | const VkStridedBufferRegionKHR *pRaygenShaderBindingTable, |
| 460 | const VkStridedBufferRegionKHR *pMissShaderBindingTable, |
| 461 | const VkStridedBufferRegionKHR *pHitShaderBindingTable, |
| 462 | const VkStridedBufferRegionKHR *pCallableShaderBindingTable, |
| 463 | VkBuffer buffer, VkDeviceSize offset) const { |
| 464 | bool skip = ValidateCmdDrawType(commandBuffer, true, VK_PIPELINE_BIND_POINT_RAY_TRACING_KHR, CMD_TRACERAYSKHR, |
| 465 | "vkCmdTraceRaysIndirectKHR()", VK_QUEUE_COMPUTE_BIT); |
| 466 | return skip; |
| 467 | } |
| 468 | |
| 469 | void CoreChecks::PostCallRecordCmdTraceRaysIndirectKHR(VkCommandBuffer commandBuffer, |
| 470 | const VkStridedBufferRegionKHR *pRaygenShaderBindingTable, |
| 471 | const VkStridedBufferRegionKHR *pMissShaderBindingTable, |
| 472 | const VkStridedBufferRegionKHR *pHitShaderBindingTable, |
| 473 | const VkStridedBufferRegionKHR *pCallableShaderBindingTable, VkBuffer buffer, |
| 474 | VkDeviceSize offset) { |
| 475 | CMD_BUFFER_STATE *cb_state = GetCBState(commandBuffer); |
| 476 | BUFFER_STATE *buffer_state = GetBufferState(buffer); |
| 477 | UpdateStateCmdDrawDispatchType(cb_state, VK_PIPELINE_BIND_POINT_RAY_TRACING_KHR); |
| 478 | cb_state->hasTraceRaysCmd = true; |
| 479 | AddCommandBufferBindingBuffer(cb_state, buffer_state); |
| 480 | } |
| 481 | |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 482 | bool CoreChecks::PreCallValidateCmdDrawMeshTasksNV(VkCommandBuffer commandBuffer, uint32_t taskCount, uint32_t firstTask) const { |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 483 | bool skip = ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWMESHTASKSNV, |
| 484 | "vkCmdDrawMeshTasksNV()", VK_QUEUE_GRAPHICS_BIT); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 485 | return skip; |
| 486 | } |
| 487 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 488 | bool CoreChecks::PreCallValidateCmdDrawMeshTasksIndirectNV(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 489 | uint32_t drawCount, uint32_t stride) const { |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 490 | bool skip = ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWMESHTASKSINDIRECTNV, |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 491 | "vkCmdDrawMeshTasksIndirectNV()", VK_QUEUE_GRAPHICS_BIT); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 492 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 493 | skip |= ValidateMemoryIsBoundToBuffer(buffer_state, "vkCmdDrawMeshTasksIndirectNV()", |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 494 | "VUID-vkCmdDrawMeshTasksIndirectNV-buffer-02708"); |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 495 | skip |= ValidateBufferUsageFlags(buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 496 | "VUID-vkCmdDrawMeshTasksIndirectNV-buffer-02709", "vkCmdDrawMeshTasksIndirectNV()", |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 497 | "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 498 | if (drawCount > 1) { |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 499 | skip |= ValidateCmdDrawStrideWithBuffer(commandBuffer, "VUID-vkCmdDrawMeshTasksIndirectNV-drawCount-02157", stride, |
| 500 | "VkDrawMeshTasksIndirectCommandNV", sizeof(VkDrawMeshTasksIndirectCommandNV), |
| 501 | drawCount, offset, buffer_state); |
| 502 | } |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 503 | return skip; |
| 504 | } |
| 505 | |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 506 | bool CoreChecks::PreCallValidateCmdDrawMeshTasksIndirectCountNV(VkCommandBuffer commandBuffer, VkBuffer buffer, VkDeviceSize offset, |
| 507 | VkBuffer countBuffer, VkDeviceSize countBufferOffset, |
Jeff Bolz | 5c801d1 | 2019-10-09 10:38:45 -0500 | [diff] [blame] | 508 | uint32_t maxDrawCount, uint32_t stride) const { |
Mark Lobodzinski | 708aae7 | 2019-12-20 11:57:46 -0700 | [diff] [blame] | 509 | bool skip = ValidateCmdDrawType(commandBuffer, false, VK_PIPELINE_BIND_POINT_GRAPHICS, CMD_DRAWMESHTASKSINDIRECTCOUNTNV, |
Spencer Fricke | 507600b | 2020-03-22 13:23:06 -0700 | [diff] [blame] | 510 | "vkCmdDrawMeshTasksIndirectCountNV()", VK_QUEUE_GRAPHICS_BIT); |
John Zulauf | fbf3c20 | 2019-07-17 14:57:14 -0600 | [diff] [blame] | 511 | const BUFFER_STATE *buffer_state = GetBufferState(buffer); |
| 512 | const BUFFER_STATE *count_buffer_state = GetBufferState(countBuffer); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 513 | skip |= ValidateMemoryIsBoundToBuffer(buffer_state, "vkCmdDrawMeshTasksIndirectCountNV()", |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 514 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-buffer-02708"); |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 515 | skip |= ValidateMemoryIsBoundToBuffer(count_buffer_state, "vkCmdDrawMeshTasksIndirectCountNV()", |
Shannon McPherson | de3eeba | 2019-04-30 16:53:59 -0600 | [diff] [blame] | 516 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-countBuffer-02714"); |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 517 | skip |= ValidateBufferUsageFlags(buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 518 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-buffer-02709", "vkCmdDrawIndexedIndirectCount()", |
Jasper St. Pierre | bf08046 | 2019-04-11 12:54:38 -0700 | [diff] [blame] | 519 | "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
| 520 | skip |= ValidateBufferUsageFlags(count_buffer_state, VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT, true, |
Tony-LunarG | a74d3fe | 2019-11-22 15:43:20 -0700 | [diff] [blame] | 521 | "VUID-vkCmdDrawMeshTasksIndirectCountNV-countBuffer-02715", "vkCmdDrawIndexedIndirectCount()", |
| 522 | "VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT"); |
Locke | e68ac65 | 2019-05-06 10:17:33 -0600 | [diff] [blame] | 523 | skip |= ValidateCmdDrawStrideWithStruct(commandBuffer, "VUID-vkCmdDrawMeshTasksIndirectCountNV-stride-02182", stride, |
| 524 | "VkDrawMeshTasksIndirectCommandNV", sizeof(VkDrawMeshTasksIndirectCommandNV)); |
| 525 | if (maxDrawCount > 1) { |
| 526 | skip |= ValidateCmdDrawStrideWithBuffer(commandBuffer, "VUID-vkCmdDrawMeshTasksIndirectCountNV-maxDrawCount-02183", stride, |
| 527 | "VkDrawMeshTasksIndirectCommandNV", sizeof(VkDrawMeshTasksIndirectCommandNV), |
| 528 | maxDrawCount, offset, buffer_state); |
| 529 | } |
Jasper St. Pierre | 512613a | 2019-04-08 16:25:23 -0700 | [diff] [blame] | 530 | return skip; |
| 531 | } |