blob: 9c673fc455db848cf8884b07763589bdad113f2f [file] [log] [blame]
Mark Lobodzinski85672672016-10-13 08:36:42 -06001#!/usr/bin/python3 -i
2#
3# Copyright (c) 2015-2016 The Khronos Group Inc.
4# Copyright (c) 2015-2016 Valve Corporation
5# Copyright (c) 2015-2016 LunarG, Inc.
6# Copyright (c) 2015-2016 Google Inc.
7#
8# Licensed under the Apache License, Version 2.0 (the "License");
9# you may not use this file except in compliance with the License.
10# You may obtain a copy of the License at
11#
12# http://www.apache.org/licenses/LICENSE-2.0
13#
14# Unless required by applicable law or agreed to in writing, software
15# distributed under the License is distributed on an "AS IS" BASIS,
16# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
17# See the License for the specific language governing permissions and
18# limitations under the License.
19#
20# Author: Dustin Graves <dustin@lunarg.com>
Mark Lobodzinski26112592017-05-30 12:02:17 -060021# Author: Mark Lobodzinski <mark@lunarg.com>
Mark Lobodzinski85672672016-10-13 08:36:42 -060022
Mark Lobodzinski06954ea2017-06-21 12:21:45 -060023import os,re,sys,string
Mark Lobodzinski85672672016-10-13 08:36:42 -060024import xml.etree.ElementTree as etree
25from generator import *
26from collections import namedtuple
Mark Lobodzinski06954ea2017-06-21 12:21:45 -060027from vuid_mapping import *
28
Jamie Madill8d4cda22017-11-08 13:40:09 -050029# This is a workaround to use a Python 2.7 and 3.x compatible syntax.
30from io import open
Mark Lobodzinski85672672016-10-13 08:36:42 -060031
Mark Lobodzinskid4950072017-08-01 13:02:20 -060032# ParameterValidationGeneratorOptions - subclass of GeneratorOptions.
Mark Lobodzinski85672672016-10-13 08:36:42 -060033#
Mark Lobodzinskid4950072017-08-01 13:02:20 -060034# Adds options used by ParameterValidationOutputGenerator object during Parameter validation layer generation.
Mark Lobodzinski85672672016-10-13 08:36:42 -060035#
36# Additional members
37# prefixText - list of strings to prefix generated header with
38# (usually a copyright statement + calling convention macros).
39# protectFile - True if multiple inclusion protection should be
40# generated (based on the filename) around the entire header.
41# protectFeature - True if #ifndef..#endif protection should be
42# generated around a feature interface in the header file.
43# genFuncPointers - True if function pointer typedefs should be
44# generated
45# protectProto - If conditional protection should be generated
46# around prototype declarations, set to either '#ifdef'
47# to require opt-in (#ifdef protectProtoStr) or '#ifndef'
48# to require opt-out (#ifndef protectProtoStr). Otherwise
49# set to None.
50# protectProtoStr - #ifdef/#ifndef symbol to use around prototype
51# declarations, if protectProto is set
52# apicall - string to use for the function declaration prefix,
53# such as APICALL on Windows.
54# apientry - string to use for the calling convention macro,
55# in typedefs, such as APIENTRY.
56# apientryp - string to use for the calling convention macro
57# in function pointer typedefs, such as APIENTRYP.
58# indentFuncProto - True if prototype declarations should put each
59# parameter on a separate line
60# indentFuncPointer - True if typedefed function pointers should put each
61# parameter on a separate line
62# alignFuncParam - if nonzero and parameters are being put on a
63# separate line, align parameter names at the specified column
Mark Lobodzinskid4950072017-08-01 13:02:20 -060064class ParameterValidationGeneratorOptions(GeneratorOptions):
Mark Lobodzinski85672672016-10-13 08:36:42 -060065 def __init__(self,
66 filename = None,
67 directory = '.',
68 apiname = None,
69 profile = None,
70 versions = '.*',
71 emitversions = '.*',
72 defaultExtensions = None,
73 addExtensions = None,
74 removeExtensions = None,
75 sortProcedure = regSortFeatures,
76 prefixText = "",
77 genFuncPointers = True,
78 protectFile = True,
79 protectFeature = True,
80 protectProto = None,
81 protectProtoStr = None,
82 apicall = '',
83 apientry = '',
84 apientryp = '',
85 indentFuncProto = True,
86 indentFuncPointer = False,
87 alignFuncParam = 0):
88 GeneratorOptions.__init__(self, filename, directory, apiname, profile,
89 versions, emitversions, defaultExtensions,
90 addExtensions, removeExtensions, sortProcedure)
91 self.prefixText = prefixText
92 self.genFuncPointers = genFuncPointers
93 self.protectFile = protectFile
94 self.protectFeature = protectFeature
95 self.protectProto = protectProto
96 self.protectProtoStr = protectProtoStr
97 self.apicall = apicall
98 self.apientry = apientry
99 self.apientryp = apientryp
100 self.indentFuncProto = indentFuncProto
101 self.indentFuncPointer = indentFuncPointer
102 self.alignFuncParam = alignFuncParam
103
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600104# ParameterValidationOutputGenerator - subclass of OutputGenerator.
Mark Lobodzinski85672672016-10-13 08:36:42 -0600105# Generates param checker layer code.
106#
107# ---- methods ----
108# ParamCheckerOutputGenerator(errFile, warnFile, diagFile) - args as for
109# OutputGenerator. Defines additional internal state.
110# ---- methods overriding base class ----
111# beginFile(genOpts)
112# endFile()
113# beginFeature(interface, emit)
114# endFeature()
115# genType(typeinfo,name)
116# genStruct(typeinfo,name)
117# genGroup(groupinfo,name)
118# genEnum(enuminfo, name)
119# genCmd(cmdinfo)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600120class ParameterValidationOutputGenerator(OutputGenerator):
121 """Generate Parameter Validation code based on XML element attributes"""
Mark Lobodzinski85672672016-10-13 08:36:42 -0600122 # This is an ordered list of sections in the header file.
123 ALL_SECTIONS = ['command']
124 def __init__(self,
125 errFile = sys.stderr,
126 warnFile = sys.stderr,
127 diagFile = sys.stdout):
128 OutputGenerator.__init__(self, errFile, warnFile, diagFile)
129 self.INDENT_SPACES = 4
Mark Lobodzinskib6b8bbd2017-02-08 14:37:15 -0700130 self.intercepts = []
131 self.declarations = []
Mark Lobodzinski85672672016-10-13 08:36:42 -0600132 # Commands to ignore
133 self.blacklist = [
134 'vkGetInstanceProcAddr',
135 'vkGetDeviceProcAddr',
136 'vkEnumerateInstanceLayerProperties',
137 'vkEnumerateInstanceExtensionsProperties',
138 'vkEnumerateDeviceLayerProperties',
139 'vkEnumerateDeviceExtensionsProperties',
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600140 'vkCreateDebugReportCallbackKHR',
141 'vkDestroyDebugReportCallbackKHR',
142 'vkEnumerateInstanceLayerProperties',
143 'vkEnumerateInstanceExtensionProperties',
144 'vkEnumerateDeviceLayerProperties',
145 'vkCmdDebugMarkerEndEXT',
146 'vkEnumerateDeviceExtensionProperties',
147 ]
148 self.validate_only = [
149 'vkCreateInstance',
150 'vkDestroyInstance',
151 'vkCreateDevice',
152 'vkDestroyDevice',
153 'vkCreateQueryPool',
Mark Lobodzinski85672672016-10-13 08:36:42 -0600154 'vkCreateDebugReportCallbackEXT',
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600155 'vkDestroyDebugReportCallbackEXT',
156 'vkCreateCommandPool',
157 ]
Dustin Gravesce68f082017-03-30 15:42:16 -0600158 # Structure fields to ignore
159 self.structMemberBlacklist = { 'VkWriteDescriptorSet' : ['dstSet'] }
Mark Lobodzinski85672672016-10-13 08:36:42 -0600160 # Validation conditions for some special case struct members that are conditionally validated
161 self.structMemberValidationConditions = { 'VkPipelineColorBlendStateCreateInfo' : { 'logicOp' : '{}logicOpEnable == VK_TRUE' } }
162 # Header version
163 self.headerVersion = None
164 # Internal state - accumulators for different inner block text
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600165 self.validation = [] # Text comprising the main per-api parameter validation routines
Mark Lobodzinski85672672016-10-13 08:36:42 -0600166 self.structNames = [] # List of Vulkan struct typenames
167 self.stypes = [] # Values from the VkStructureType enumeration
168 self.structTypes = dict() # Map of Vulkan struct typename to required VkStructureType
169 self.handleTypes = set() # Set of handle type names
170 self.commands = [] # List of CommandData records for all Vulkan commands
171 self.structMembers = [] # List of StructMemberData records for all Vulkan structs
172 self.validatedStructs = dict() # Map of structs type names to generated validation code for that struct type
173 self.enumRanges = dict() # Map of enum name to BEGIN/END range values
Mark Lobodzinskif31e0422017-07-25 14:29:42 -0600174 self.enumValueLists = '' # String containing enumerated type map definitions
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600175 self.func_pointers = '' # String containing function pointers for manual PV functions
176 self.typedefs = '' # String containing function pointer typedefs
Mark Lobodzinski85672672016-10-13 08:36:42 -0600177 self.flags = set() # Map of flags typenames
178 self.flagBits = dict() # Map of flag bits typename to list of values
Chris Forbes78ea32d2016-11-28 11:14:17 +1300179 self.newFlags = set() # Map of flags typenames /defined in the current feature/
Mike Schuchardtafd00482017-08-24 15:15:02 -0600180 self.required_extensions = dict() # Dictionary of required extensions for each item in the current extension
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600181 self.extension_type = '' # Type of active feature (extension), device or instance
182 self.extension_names = dict() # Dictionary of extension names to extension name defines
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600183 self.valid_vuids = set() # Set of all valid VUIDs
Mark Lobodzinski85672672016-10-13 08:36:42 -0600184 # Named tuples to store struct and command data
185 self.StructType = namedtuple('StructType', ['name', 'value'])
186 self.CommandParam = namedtuple('CommandParam', ['type', 'name', 'ispointer', 'isstaticarray', 'isbool', 'israngedenum',
187 'isconst', 'isoptional', 'iscount', 'noautovalidity', 'len', 'extstructs',
188 'condition', 'cdecl'])
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600189 self.CommandData = namedtuple('CommandData', ['name', 'params', 'cdecl', 'extension_type', 'result'])
Mark Lobodzinski85672672016-10-13 08:36:42 -0600190 self.StructMemberData = namedtuple('StructMemberData', ['name', 'members'])
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600191
192 self.vuid_file = None
193 # Cover cases where file is built from scripts directory, Lin/Win, or Android build structure
194 vuid_filename_locations = [
Mark Lobodzinskifc20c4d2017-07-03 15:50:39 -0600195 './vk_validation_error_messages.h',
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600196 '../layers/vk_validation_error_messages.h',
197 '../../layers/vk_validation_error_messages.h',
198 '../../../layers/vk_validation_error_messages.h',
199 ]
200 for vuid_filename in vuid_filename_locations:
201 if os.path.isfile(vuid_filename):
Lenny Komowb79f04a2017-09-18 17:07:00 -0600202 self.vuid_file = open(vuid_filename, "r", encoding="utf8")
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600203 break
204 if self.vuid_file == None:
205 print("Error: Could not find vk_validation_error_messages.h")
Jamie Madill3935f7c2017-11-08 13:50:14 -0500206 sys.exit(1)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600207 #
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600208 # Generate Copyright comment block for file
209 def GenerateCopyright(self):
210 copyright = '/* *** THIS FILE IS GENERATED - DO NOT EDIT! ***\n'
211 copyright += ' * See parameter_validation_generator.py for modifications\n'
212 copyright += ' *\n'
213 copyright += ' * Copyright (c) 2015-2017 The Khronos Group Inc.\n'
214 copyright += ' * Copyright (c) 2015-2017 LunarG, Inc.\n'
215 copyright += ' * Copyright (C) 2015-2017 Google Inc.\n'
216 copyright += ' *\n'
217 copyright += ' * Licensed under the Apache License, Version 2.0 (the "License");\n'
218 copyright += ' * you may not use this file except in compliance with the License.\n'
219 copyright += ' * Copyright (c) 2015-2017 Valve Corporation\n'
220 copyright += ' * You may obtain a copy of the License at\n'
221 copyright += ' *\n'
222 copyright += ' * http://www.apache.org/licenses/LICENSE-2.0\n'
223 copyright += ' *\n'
224 copyright += ' * Unless required by applicable law or agreed to in writing, software\n'
225 copyright += ' * distributed under the License is distributed on an "AS IS" BASIS,\n'
226 copyright += ' * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n'
227 copyright += ' * See the License for the specific language governing permissions and\n'
228 copyright += ' * limitations under the License.\n'
229 copyright += ' *\n'
230 copyright += ' * Author: Mark Lobodzinski <mark@LunarG.com>\n'
231 copyright += ' */\n\n'
232 return copyright
233 #
234 # Increases the global indent variable
Mark Lobodzinski85672672016-10-13 08:36:42 -0600235 def incIndent(self, indent):
236 inc = ' ' * self.INDENT_SPACES
237 if indent:
238 return indent + inc
239 return inc
240 #
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600241 # Decreases the global indent variable
Mark Lobodzinski85672672016-10-13 08:36:42 -0600242 def decIndent(self, indent):
243 if indent and (len(indent) > self.INDENT_SPACES):
244 return indent[:-self.INDENT_SPACES]
245 return ''
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600246 #
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600247 # Convert decimal number to 8 digit hexadecimal lower-case representation
248 def IdToHex(self, dec_num):
249 if dec_num > 4294967295:
250 print ("ERROR: Decimal # %d can't be represented in 8 hex digits" % (dec_num))
Jamie Madill3935f7c2017-11-08 13:50:14 -0500251 sys.exit(1)
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600252 hex_num = hex(dec_num)
253 return hex_num[2:].zfill(8)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600254 #
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600255 # Called at file creation time
Mark Lobodzinski85672672016-10-13 08:36:42 -0600256 def beginFile(self, genOpts):
257 OutputGenerator.beginFile(self, genOpts)
258 # C-specific
259 #
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600260 # Open vk_validation_error_messages.h file to verify computed VUIDs
261 for line in self.vuid_file:
262 # Grab hex number from enum definition
263 vuid_list = line.split('0x')
264 # If this is a valid enumeration line, remove trailing comma and CR
265 if len(vuid_list) == 2:
266 vuid_num = vuid_list[1][:-2]
267 # Make sure this is a good hex number before adding to set
268 if len(vuid_num) == 8 and all(c in string.hexdigits for c in vuid_num):
269 self.valid_vuids.add(vuid_num)
270 #
Mark Lobodzinski85672672016-10-13 08:36:42 -0600271 # User-supplied prefix text, if any (list of strings)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600272 s = self.GenerateCopyright()
273 write(s, file=self.outFile)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600274 #
275 # Headers
276 write('#include <string>', file=self.outFile)
277 self.newline()
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600278 write('#include "vk_loader_platform.h"', file=self.outFile)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600279 write('#include "vulkan/vulkan.h"', file=self.outFile)
280 write('#include "vk_layer_extension_utils.h"', file=self.outFile)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600281 write('#include "parameter_validation.h"', file=self.outFile)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600282 #
283 # Macros
284 self.newline()
285 write('#ifndef UNUSED_PARAMETER', file=self.outFile)
286 write('#define UNUSED_PARAMETER(x) (void)(x)', file=self.outFile)
287 write('#endif // UNUSED_PARAMETER', file=self.outFile)
288 #
289 # Namespace
290 self.newline()
291 write('namespace parameter_validation {', file = self.outFile)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600292 self.newline()
293 write('extern std::mutex global_lock;', file = self.outFile)
294 write('extern std::unordered_map<void *, layer_data *> layer_data_map;', file = self.outFile)
295 write('extern std::unordered_map<void *, instance_layer_data *> instance_layer_data_map;', file = self.outFile)
296 self.newline()
Mark Lobodzinski78a12a92017-08-08 14:16:51 -0600297 #
298 # FuncPtrMap
299 self.func_pointers += 'std::unordered_map<std::string, void *> custom_functions = {\n'
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600300 #
301 # Called at end-time for final content output
Mark Lobodzinski85672672016-10-13 08:36:42 -0600302 def endFile(self):
303 # C-specific
304 self.newline()
Mark Lobodzinskif31e0422017-07-25 14:29:42 -0600305 write(self.enumValueLists, file=self.outFile)
306 self.newline()
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600307 write(self.typedefs, file=self.outFile)
308 self.newline()
Mark Lobodzinski78a12a92017-08-08 14:16:51 -0600309 self.func_pointers += '};\n'
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600310 write(self.func_pointers, file=self.outFile)
311 self.newline()
312 ext_template = 'template <typename T>\n'
313 ext_template += 'bool OutputExtensionError(const T *layer_data, const std::string &api_name, const std::string &extension_name) {\n'
314 ext_template += ' return log_msg(layer_data->report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0, __LINE__,\n'
315 ext_template += ' EXTENSION_NOT_ENABLED, LayerName, "Attemped to call %s() but its required extension %s has not been enabled\\n",\n'
316 ext_template += ' api_name.c_str(), extension_name.c_str());\n'
317 ext_template += '}\n'
318 write(ext_template, file=self.outFile)
319 self.newline()
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600320 commands_text = '\n'.join(self.validation)
321 write(commands_text, file=self.outFile)
322 self.newline()
Mark Lobodzinskib6b8bbd2017-02-08 14:37:15 -0700323 # Output declarations and record intercepted procedures
324 write('// Declarations', file=self.outFile)
325 write('\n'.join(self.declarations), file=self.outFile)
Mark Lobodzinskide43e642017-06-07 14:00:31 -0600326 write('// Map of all APIs to be intercepted by this layer', file=self.outFile)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600327 write('const std::unordered_map<std::string, void*> name_to_funcptr_map = {', file=self.outFile)
Mark Lobodzinskib6b8bbd2017-02-08 14:37:15 -0700328 write('\n'.join(self.intercepts), file=self.outFile)
329 write('};\n', file=self.outFile)
330 self.newline()
Mark Lobodzinski85672672016-10-13 08:36:42 -0600331 # Namespace
332 write('} // namespace parameter_validation', file = self.outFile)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600333 # Finish processing in superclass
334 OutputGenerator.endFile(self)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600335 #
336 # Processing at beginning of each feature or extension
Mark Lobodzinski85672672016-10-13 08:36:42 -0600337 def beginFeature(self, interface, emit):
338 # Start processing in superclass
339 OutputGenerator.beginFeature(self, interface, emit)
340 # C-specific
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600341 # Accumulate includes, defines, types, enums, function pointer typedefs, end function prototypes separately for this
342 # feature. They're only printed in endFeature().
Mark Lobodzinski85672672016-10-13 08:36:42 -0600343 self.headerVersion = None
Mark Lobodzinski85672672016-10-13 08:36:42 -0600344 self.structNames = []
345 self.stypes = []
346 self.structTypes = dict()
Mark Lobodzinski85672672016-10-13 08:36:42 -0600347 self.commands = []
348 self.structMembers = []
Chris Forbes78ea32d2016-11-28 11:14:17 +1300349 self.newFlags = set()
Mike Schuchardtafd00482017-08-24 15:15:02 -0600350
351 # Get base list of extension dependencies for all items in this extension
352 base_required_extensions = []
Mark Lobodzinski26112592017-05-30 12:02:17 -0600353 if self.featureName != "VK_VERSION_1_0":
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600354 # Save Name Define to get correct enable name later
355 nameElem = interface[0][1]
356 name = nameElem.get('name')
357 self.extension_names[self.featureName] = name
358 # This extension is the first dependency for this command
Mike Schuchardtafd00482017-08-24 15:15:02 -0600359 base_required_extensions.append(self.featureName)
360 # Add any defined extension dependencies to the base dependency list for this extension
361 requires = interface.get('requires')
362 if requires is not None:
363 base_required_extensions.extend(requires.split(','))
364
365 # Build dictionary of extension dependencies for each item in this extension
366 self.required_extensions = dict()
367 for require_element in interface.findall('require'):
368 # Copy base extension dependency list
369 required_extensions = list(base_required_extensions)
370 # Add any additional extension dependencies specified in this require block
371 additional_extensions = require_element.get('extension')
372 if additional_extensions:
373 required_extensions.extend(additional_extensions.split(','))
374 # Save full extension list for all named items
375 for element in require_element.findall('*[@name]'):
376 self.required_extensions[element.get('name')] = required_extensions
377
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600378 # And note if this is an Instance or Device extension
379 self.extension_type = interface.get('type')
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600380 #
381 # Called at the end of each extension (feature)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600382 def endFeature(self):
383 # C-specific
384 # Actually write the interface to the output file.
385 if (self.emit):
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600386 # If type declarations are needed by other features based on this one, it may be necessary to suppress the ExtraProtect,
Mark Lobodzinski85672672016-10-13 08:36:42 -0600387 # or move it below the 'for section...' loop.
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600388 ifdef = ''
Mark Lobodzinski85672672016-10-13 08:36:42 -0600389 if (self.featureExtraProtect != None):
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600390 ifdef = '#ifdef %s\n' % self.featureExtraProtect
391 self.validation.append(ifdef)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600392 # Generate the struct member checking code from the captured data
393 self.processStructMemberData()
394 # Generate the command parameter checking code from the captured data
395 self.processCmdData()
396 # Write the declaration for the HeaderVersion
397 if self.headerVersion:
398 write('const uint32_t GeneratedHeaderVersion = {};'.format(self.headerVersion), file=self.outFile)
399 self.newline()
400 # Write the declarations for the VkFlags values combining all flag bits
Chris Forbes78ea32d2016-11-28 11:14:17 +1300401 for flag in sorted(self.newFlags):
Mark Lobodzinski85672672016-10-13 08:36:42 -0600402 flagBits = flag.replace('Flags', 'FlagBits')
403 if flagBits in self.flagBits:
404 bits = self.flagBits[flagBits]
405 decl = 'const {} All{} = {}'.format(flag, flagBits, bits[0])
406 for bit in bits[1:]:
407 decl += '|' + bit
408 decl += ';'
409 write(decl, file=self.outFile)
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600410 endif = '\n'
Mark Lobodzinski85672672016-10-13 08:36:42 -0600411 if (self.featureExtraProtect != None):
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -0600412 endif = '#endif // %s\n' % self.featureExtraProtect
413 self.validation.append(endif)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600414 # Finish processing in superclass
415 OutputGenerator.endFeature(self)
416 #
Mark Lobodzinski85672672016-10-13 08:36:42 -0600417 # Type generation
418 def genType(self, typeinfo, name):
419 OutputGenerator.genType(self, typeinfo, name)
420 typeElem = typeinfo.elem
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600421 # If the type is a struct type, traverse the imbedded <member> tags generating a structure. Otherwise, emit the tag text.
Mark Lobodzinski85672672016-10-13 08:36:42 -0600422 category = typeElem.get('category')
423 if (category == 'struct' or category == 'union'):
424 self.structNames.append(name)
425 self.genStruct(typeinfo, name)
426 elif (category == 'handle'):
427 self.handleTypes.add(name)
428 elif (category == 'bitmask'):
429 self.flags.add(name)
Chris Forbes78ea32d2016-11-28 11:14:17 +1300430 self.newFlags.add(name)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600431 elif (category == 'define'):
432 if name == 'VK_HEADER_VERSION':
433 nameElem = typeElem.find('name')
434 self.headerVersion = noneStr(nameElem.tail).strip()
435 #
436 # Struct parameter check generation.
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600437 # This is a special case of the <type> tag where the contents are interpreted as a set of <member> tags instead of freeform C
438 # type declarations. The <member> tags are just like <param> tags - they are a declaration of a struct or union member.
439 # Only simple member declarations are supported (no nested structs etc.)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600440 def genStruct(self, typeinfo, typeName):
441 OutputGenerator.genStruct(self, typeinfo, typeName)
442 conditions = self.structMemberValidationConditions[typeName] if typeName in self.structMemberValidationConditions else None
443 members = typeinfo.elem.findall('.//member')
444 #
445 # Iterate over members once to get length parameters for arrays
446 lens = set()
447 for member in members:
448 len = self.getLen(member)
449 if len:
450 lens.add(len)
451 #
452 # Generate member info
453 membersInfo = []
454 for member in members:
455 # Get the member's type and name
456 info = self.getTypeNameTuple(member)
457 type = info[0]
458 name = info[1]
459 stypeValue = ''
460 cdecl = self.makeCParamDecl(member, 0)
461 # Process VkStructureType
462 if type == 'VkStructureType':
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600463 # Extract the required struct type value from the comments embedded in the original text defining the
464 # 'typeinfo' element
Mark Lobodzinski85672672016-10-13 08:36:42 -0600465 rawXml = etree.tostring(typeinfo.elem).decode('ascii')
466 result = re.search(r'VK_STRUCTURE_TYPE_\w+', rawXml)
467 if result:
468 value = result.group(0)
469 else:
470 value = self.genVkStructureType(typeName)
471 # Store the required type value
472 self.structTypes[typeName] = self.StructType(name=name, value=value)
473 #
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600474 # Store pointer/array/string info -- Check for parameter name in lens set
Mark Lobodzinski85672672016-10-13 08:36:42 -0600475 iscount = False
476 if name in lens:
477 iscount = True
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600478 # The pNext members are not tagged as optional, but are treated as optional for parameter NULL checks. Static array
479 # members are also treated as optional to skip NULL pointer validation, as they won't be NULL.
Mark Lobodzinski85672672016-10-13 08:36:42 -0600480 isstaticarray = self.paramIsStaticArray(member)
481 isoptional = False
482 if self.paramIsOptional(member) or (name == 'pNext') or (isstaticarray):
483 isoptional = True
Dustin Gravesce68f082017-03-30 15:42:16 -0600484 # Determine if value should be ignored by code generation.
485 noautovalidity = False
486 if (member.attrib.get('noautovalidity') is not None) or ((typeName in self.structMemberBlacklist) and (name in self.structMemberBlacklist[typeName])):
487 noautovalidity = True
Mark Lobodzinski85672672016-10-13 08:36:42 -0600488 membersInfo.append(self.CommandParam(type=type, name=name,
489 ispointer=self.paramIsPointer(member),
490 isstaticarray=isstaticarray,
491 isbool=True if type == 'VkBool32' else False,
492 israngedenum=True if type in self.enumRanges else False,
493 isconst=True if 'const' in cdecl else False,
494 isoptional=isoptional,
495 iscount=iscount,
Dustin Gravesce68f082017-03-30 15:42:16 -0600496 noautovalidity=noautovalidity,
Mark Lobodzinski85672672016-10-13 08:36:42 -0600497 len=self.getLen(member),
Mike Schuchardta40d0b02017-07-23 12:47:47 -0600498 extstructs=self.registry.validextensionstructs[typeName] if name == 'pNext' else None,
Mark Lobodzinski85672672016-10-13 08:36:42 -0600499 condition=conditions[name] if conditions and name in conditions else None,
500 cdecl=cdecl))
501 self.structMembers.append(self.StructMemberData(name=typeName, members=membersInfo))
502 #
Mark Lobodzinskif31e0422017-07-25 14:29:42 -0600503 # Capture group (e.g. C "enum" type) info to be used for param check code generation.
Mark Lobodzinski85672672016-10-13 08:36:42 -0600504 # These are concatenated together with other types.
505 def genGroup(self, groupinfo, groupName):
506 OutputGenerator.genGroup(self, groupinfo, groupName)
507 groupElem = groupinfo.elem
Mark Lobodzinski85672672016-10-13 08:36:42 -0600508 # Store the sType values
509 if groupName == 'VkStructureType':
510 for elem in groupElem.findall('enum'):
511 self.stypes.append(elem.get('name'))
512 elif 'FlagBits' in groupName:
513 bits = []
514 for elem in groupElem.findall('enum'):
515 bits.append(elem.get('name'))
516 if bits:
517 self.flagBits[groupName] = bits
518 else:
519 # Determine if begin/end ranges are needed (we don't do this for VkStructureType, which has a more finely grained check)
520 expandName = re.sub(r'([0-9a-z_])([A-Z0-9][^A-Z0-9]?)',r'\1_\2',groupName).upper()
521 expandPrefix = expandName
522 expandSuffix = ''
523 expandSuffixMatch = re.search(r'[A-Z][A-Z]+$',groupName)
524 if expandSuffixMatch:
525 expandSuffix = '_' + expandSuffixMatch.group()
526 # Strip off the suffix from the prefix
527 expandPrefix = expandName.rsplit(expandSuffix, 1)[0]
528 isEnum = ('FLAG_BITS' not in expandPrefix)
529 if isEnum:
530 self.enumRanges[groupName] = (expandPrefix + '_BEGIN_RANGE' + expandSuffix, expandPrefix + '_END_RANGE' + expandSuffix)
Mark Lobodzinskif31e0422017-07-25 14:29:42 -0600531 # Create definition for a list containing valid enum values for this enumerated type
532 enum_entry = 'const std::vector<%s> All%sEnums = {' % (groupName, groupName)
533 for enum in groupElem:
534 name = enum.get('name')
Mark Lobodzinski117d88f2017-07-27 12:09:08 -0600535 if name is not None and enum.get('supported') != 'disabled':
536 enum_entry += '%s, ' % name
Mark Lobodzinskif31e0422017-07-25 14:29:42 -0600537 enum_entry += '};\n'
538 self.enumValueLists += enum_entry
Mark Lobodzinski85672672016-10-13 08:36:42 -0600539 #
Mark Lobodzinskif31e0422017-07-25 14:29:42 -0600540 # Capture command parameter info to be used for param check code generation.
Mark Lobodzinski85672672016-10-13 08:36:42 -0600541 def genCmd(self, cmdinfo, name):
542 OutputGenerator.genCmd(self, cmdinfo, name)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600543 decls = self.makeCDecls(cmdinfo.elem)
544 typedef = decls[1]
545 typedef = typedef.split(')',1)[1]
546 if name not in self.blacklist:
Mark Lobodzinskib6b8bbd2017-02-08 14:37:15 -0700547 if (self.featureExtraProtect != None):
548 self.declarations += [ '#ifdef %s' % self.featureExtraProtect ]
549 self.intercepts += [ '#ifdef %s' % self.featureExtraProtect ]
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600550 if (name not in self.validate_only):
551 self.func_pointers += '#ifdef %s\n' % self.featureExtraProtect
552 self.typedefs += '#ifdef %s\n' % self.featureExtraProtect
553 if (name not in self.validate_only):
554 self.typedefs += 'typedef bool (*PFN_manual_%s)%s\n' % (name, typedef)
Mark Lobodzinski78a12a92017-08-08 14:16:51 -0600555 self.func_pointers += ' {"%s", nullptr},\n' % name
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600556 self.intercepts += [ ' {"%s", (void*)%s},' % (name,name) ]
Mark Lobodzinskib6b8bbd2017-02-08 14:37:15 -0700557 # Strip off 'vk' from API name
558 self.declarations += [ '%s' % decls[0].replace("VKAPI_CALL vk", "VKAPI_CALL ") ]
559 if (self.featureExtraProtect != None):
560 self.intercepts += [ '#endif' ]
561 self.declarations += [ '#endif' ]
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600562 if (name not in self.validate_only):
563 self.func_pointers += '#endif\n'
564 self.typedefs += '#endif\n'
Mark Lobodzinski85672672016-10-13 08:36:42 -0600565 if name not in self.blacklist:
566 params = cmdinfo.elem.findall('param')
567 # Get list of array lengths
568 lens = set()
569 for param in params:
570 len = self.getLen(param)
571 if len:
572 lens.add(len)
573 # Get param info
574 paramsInfo = []
575 for param in params:
576 paramInfo = self.getTypeNameTuple(param)
577 cdecl = self.makeCParamDecl(param, 0)
578 # Check for parameter name in lens set
579 iscount = False
580 if paramInfo[1] in lens:
581 iscount = True
582 paramsInfo.append(self.CommandParam(type=paramInfo[0], name=paramInfo[1],
583 ispointer=self.paramIsPointer(param),
584 isstaticarray=self.paramIsStaticArray(param),
585 isbool=True if paramInfo[0] == 'VkBool32' else False,
586 israngedenum=True if paramInfo[0] in self.enumRanges else False,
587 isconst=True if 'const' in cdecl else False,
588 isoptional=self.paramIsOptional(param),
589 iscount=iscount,
590 noautovalidity=True if param.attrib.get('noautovalidity') is not None else False,
591 len=self.getLen(param),
592 extstructs=None,
593 condition=None,
594 cdecl=cdecl))
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600595 # Save return value information, if any
596 result_type = ''
597 resultinfo = cmdinfo.elem.find('proto/type')
598 if (resultinfo != None and resultinfo.text != 'void'):
599 result_type = resultinfo.text
600 self.commands.append(self.CommandData(name=name, params=paramsInfo, cdecl=self.makeCDecls(cmdinfo.elem)[0], extension_type=self.extension_type, result=result_type))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600601 #
602 # Check if the parameter passed in is a pointer
603 def paramIsPointer(self, param):
604 ispointer = 0
605 paramtype = param.find('type')
606 if (paramtype.tail is not None) and ('*' in paramtype.tail):
607 ispointer = paramtype.tail.count('*')
608 elif paramtype.text[:4] == 'PFN_':
609 # Treat function pointer typedefs as a pointer to a single value
610 ispointer = 1
611 return ispointer
612 #
613 # Check if the parameter passed in is a static array
614 def paramIsStaticArray(self, param):
615 isstaticarray = 0
616 paramname = param.find('name')
617 if (paramname.tail is not None) and ('[' in paramname.tail):
618 isstaticarray = paramname.tail.count('[')
619 return isstaticarray
620 #
621 # Check if the parameter passed in is optional
622 # Returns a list of Boolean values for comma separated len attributes (len='false,true')
623 def paramIsOptional(self, param):
624 # See if the handle is optional
625 isoptional = False
626 # Simple, if it's optional, return true
627 optString = param.attrib.get('optional')
628 if optString:
629 if optString == 'true':
630 isoptional = True
631 elif ',' in optString:
632 opts = []
633 for opt in optString.split(','):
634 val = opt.strip()
635 if val == 'true':
636 opts.append(True)
637 elif val == 'false':
638 opts.append(False)
639 else:
640 print('Unrecognized len attribute value',val)
641 isoptional = opts
642 return isoptional
643 #
644 # Check if the handle passed in is optional
645 # Uses the same logic as ValidityOutputGenerator.isHandleOptional
646 def isHandleOptional(self, param, lenParam):
647 # Simple, if it's optional, return true
648 if param.isoptional:
649 return True
650 # If no validity is being generated, it usually means that validity is complex and not absolute, so let's say yes.
651 if param.noautovalidity:
652 return True
653 # If the parameter is an array and we haven't already returned, find out if any of the len parameters are optional
654 if lenParam and lenParam.isoptional:
655 return True
656 return False
657 #
658 # Generate a VkStructureType based on a structure typename
659 def genVkStructureType(self, typename):
660 # Add underscore between lowercase then uppercase
661 value = re.sub('([a-z0-9])([A-Z])', r'\1_\2', typename)
Mark Young39389872017-01-19 21:10:49 -0700662 value = value.replace('D3_D12', 'D3D12')
663 value = value.replace('Device_IDProp', 'Device_ID_Prop')
Mark Lobodzinski85672672016-10-13 08:36:42 -0600664 # Change to uppercase
665 value = value.upper()
666 # Add STRUCTURE_TYPE_
667 return re.sub('VK_', 'VK_STRUCTURE_TYPE_', value)
668 #
669 # Get the cached VkStructureType value for the specified struct typename, or generate a VkStructureType
670 # value assuming the struct is defined by a different feature
671 def getStructType(self, typename):
672 value = None
673 if typename in self.structTypes:
674 value = self.structTypes[typename].value
675 else:
676 value = self.genVkStructureType(typename)
677 self.logMsg('diag', 'ParameterValidation: Generating {} for {} structure type that was not defined by the current feature'.format(value, typename))
678 return value
679 #
680 # Retrieve the value of the len tag
681 def getLen(self, param):
682 result = None
683 len = param.attrib.get('len')
684 if len and len != 'null-terminated':
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600685 # For string arrays, 'len' can look like 'count,null-terminated', indicating that we have a null terminated array of
686 # strings. We strip the null-terminated from the 'len' field and only return the parameter specifying the string count
Mark Lobodzinski85672672016-10-13 08:36:42 -0600687 if 'null-terminated' in len:
688 result = len.split(',')[0]
689 else:
690 result = len
691 result = str(result).replace('::', '->')
692 return result
693 #
694 # Retrieve the type and name for a parameter
695 def getTypeNameTuple(self, param):
696 type = ''
697 name = ''
698 for elem in param:
699 if elem.tag == 'type':
700 type = noneStr(elem.text)
701 elif elem.tag == 'name':
702 name = noneStr(elem.text)
703 return (type, name)
704 #
705 # Find a named parameter in a parameter list
706 def getParamByName(self, params, name):
707 for param in params:
708 if param.name == name:
709 return param
710 return None
711 #
712 # Extract length values from latexmath. Currently an inflexible solution that looks for specific
713 # patterns that are found in vk.xml. Will need to be updated when new patterns are introduced.
714 def parseLateXMath(self, source):
715 name = 'ERROR'
716 decoratedName = 'ERROR'
717 if 'mathit' in source:
Mark Lobodzinski36c33862017-02-13 10:15:53 -0700718 # Matches expressions similar to 'latexmath:[\lceil{\mathit{rasterizationSamples} \over 32}\rceil]'
719 match = re.match(r'latexmath\s*\:\s*\[\s*\\l(\w+)\s*\{\s*\\mathit\s*\{\s*(\w+)\s*\}\s*\\over\s*(\d+)\s*\}\s*\\r(\w+)\s*\]', source)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600720 if not match or match.group(1) != match.group(4):
721 raise 'Unrecognized latexmath expression'
722 name = match.group(2)
723 decoratedName = '{}({}/{})'.format(*match.group(1, 2, 3))
724 else:
Mark Lobodzinski36c33862017-02-13 10:15:53 -0700725 # Matches expressions similar to 'latexmath : [dataSize \over 4]'
Mark Young0f183a82017-02-28 09:58:04 -0700726 match = re.match(r'latexmath\s*\:\s*\[\s*(\w+)\s*\\over\s*(\d+)\s*\]', source)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600727 name = match.group(1)
728 decoratedName = '{}/{}'.format(*match.group(1, 2))
729 return name, decoratedName
730 #
731 # Get the length paramater record for the specified parameter name
732 def getLenParam(self, params, name):
733 lenParam = None
734 if name:
735 if '->' in name:
736 # The count is obtained by dereferencing a member of a struct parameter
737 lenParam = self.CommandParam(name=name, iscount=True, ispointer=False, isbool=False, israngedenum=False, isconst=False,
738 isstaticarray=None, isoptional=False, type=None, noautovalidity=False, len=None, extstructs=None,
739 condition=None, cdecl=None)
740 elif 'latexmath' in name:
741 lenName, decoratedName = self.parseLateXMath(name)
742 lenParam = self.getParamByName(params, lenName)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600743 else:
744 lenParam = self.getParamByName(params, name)
745 return lenParam
746 #
747 # Convert a vulkan.h command declaration into a parameter_validation.h definition
748 def getCmdDef(self, cmd):
Mark Lobodzinski85672672016-10-13 08:36:42 -0600749 # Strip the trailing ';' and split into individual lines
750 lines = cmd.cdecl[:-1].split('\n')
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600751 cmd_hdr = '\n'.join(lines)
752 return cmd_hdr
Mark Lobodzinski85672672016-10-13 08:36:42 -0600753 #
754 # Generate the code to check for a NULL dereference before calling the
755 # validation function
756 def genCheckedLengthCall(self, name, exprs):
757 count = name.count('->')
758 if count:
759 checkedExpr = []
760 localIndent = ''
761 elements = name.split('->')
762 # Open the if expression blocks
763 for i in range(0, count):
764 checkedExpr.append(localIndent + 'if ({} != NULL) {{\n'.format('->'.join(elements[0:i+1])))
765 localIndent = self.incIndent(localIndent)
766 # Add the validation expression
767 for expr in exprs:
768 checkedExpr.append(localIndent + expr)
769 # Close the if blocks
770 for i in range(0, count):
771 localIndent = self.decIndent(localIndent)
772 checkedExpr.append(localIndent + '}\n')
773 return [checkedExpr]
774 # No if statements were required
775 return exprs
776 #
777 # Generate code to check for a specific condition before executing validation code
778 def genConditionalCall(self, prefix, condition, exprs):
779 checkedExpr = []
780 localIndent = ''
781 formattedCondition = condition.format(prefix)
782 checkedExpr.append(localIndent + 'if ({})\n'.format(formattedCondition))
783 checkedExpr.append(localIndent + '{\n')
784 localIndent = self.incIndent(localIndent)
785 for expr in exprs:
786 checkedExpr.append(localIndent + expr)
787 localIndent = self.decIndent(localIndent)
788 checkedExpr.append(localIndent + '}\n')
789 return [checkedExpr]
790 #
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600791 # Get VUID identifier from implicit VUID tag
792 def GetVuid(self, vuid_string):
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -0600793 if '->' in vuid_string:
794 return "VALIDATION_ERROR_UNDEFINED"
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600795 vuid_num = self.IdToHex(convertVUID(vuid_string))
796 if vuid_num in self.valid_vuids:
797 vuid = "VALIDATION_ERROR_%s" % vuid_num
798 else:
799 vuid = "VALIDATION_ERROR_UNDEFINED"
800 return vuid
801 #
Mark Lobodzinski85672672016-10-13 08:36:42 -0600802 # Generate the sType check string
Mark Lobodzinski9cf24dd2017-06-28 14:23:22 -0600803 def makeStructTypeCheck(self, prefix, value, lenValue, valueRequired, lenValueRequired, lenPtrRequired, funcPrintName, lenPrintName, valuePrintName, postProcSpec, struct_type_name):
Mark Lobodzinski85672672016-10-13 08:36:42 -0600804 checkExpr = []
805 stype = self.structTypes[value.type]
806 if lenValue:
Mark Lobodzinski9cf24dd2017-06-28 14:23:22 -0600807 vuid_name = struct_type_name if struct_type_name is not None else funcPrintName
808 vuid = self.GetVuid("VUID-%s-%s-parameter" % (vuid_name, value.name))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600809 # This is an array with a pointer to a count value
810 if lenValue.ispointer:
811 # When the length parameter is a pointer, there is an extra Boolean parameter in the function call to indicate if it is required
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600812 checkExpr.append('skip |= validate_struct_type_array(local_data->report_data, "{}", {ppp}"{ldn}"{pps}, {ppp}"{dn}"{pps}, "{sv}", {pf}{ln}, {pf}{vn}, {sv}, {}, {}, {}, {});\n'.format(
Mark Lobodzinski9cf24dd2017-06-28 14:23:22 -0600813 funcPrintName, lenPtrRequired, lenValueRequired, valueRequired, vuid, ln=lenValue.name, ldn=lenPrintName, dn=valuePrintName, vn=value.name, sv=stype.value, pf=prefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600814 # This is an array with an integer count value
815 else:
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600816 checkExpr.append('skip |= validate_struct_type_array(local_data->report_data, "{}", {ppp}"{ldn}"{pps}, {ppp}"{dn}"{pps}, "{sv}", {pf}{ln}, {pf}{vn}, {sv}, {}, {}, {});\n'.format(
Mark Lobodzinski9cf24dd2017-06-28 14:23:22 -0600817 funcPrintName, lenValueRequired, valueRequired, vuid, ln=lenValue.name, ldn=lenPrintName, dn=valuePrintName, vn=value.name, sv=stype.value, pf=prefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600818 # This is an individual struct
819 else:
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600820 vuid = self.GetVuid("VUID-%s-sType-sType" % value.type)
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600821 checkExpr.append('skip |= validate_struct_type(local_data->report_data, "{}", {ppp}"{}"{pps}, "{sv}", {}{vn}, {sv}, {}, {});\n'.format(
Mark Lobodzinski06954ea2017-06-21 12:21:45 -0600822 funcPrintName, valuePrintName, prefix, valueRequired, vuid, vn=value.name, sv=stype.value, vt=value.type, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600823 return checkExpr
824 #
825 # Generate the handle check string
826 def makeHandleCheck(self, prefix, value, lenValue, valueRequired, lenValueRequired, funcPrintName, lenPrintName, valuePrintName, postProcSpec):
827 checkExpr = []
828 if lenValue:
829 if lenValue.ispointer:
830 # This is assumed to be an output array with a pointer to a count value
831 raise('Unsupported parameter validation case: Output handle array elements are not NULL checked')
832 else:
833 # This is an array with an integer count value
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600834 checkExpr.append('skip |= validate_handle_array(local_data->report_data, "{}", {ppp}"{ldn}"{pps}, {ppp}"{dn}"{pps}, {pf}{ln}, {pf}{vn}, {}, {});\n'.format(
Mark Lobodzinski85672672016-10-13 08:36:42 -0600835 funcPrintName, lenValueRequired, valueRequired, ln=lenValue.name, ldn=lenPrintName, dn=valuePrintName, vn=value.name, pf=prefix, **postProcSpec))
836 else:
837 # This is assumed to be an output handle pointer
838 raise('Unsupported parameter validation case: Output handles are not NULL checked')
839 return checkExpr
840 #
841 # Generate check string for an array of VkFlags values
842 def makeFlagsArrayCheck(self, prefix, value, lenValue, valueRequired, lenValueRequired, funcPrintName, lenPrintName, valuePrintName, postProcSpec):
843 checkExpr = []
844 flagBitsName = value.type.replace('Flags', 'FlagBits')
845 if not flagBitsName in self.flagBits:
846 raise('Unsupported parameter validation case: array of reserved VkFlags')
847 else:
848 allFlags = 'All' + flagBitsName
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600849 checkExpr.append('skip |= validate_flags_array(local_data->report_data, "{}", {ppp}"{}"{pps}, {ppp}"{}"{pps}, "{}", {}, {pf}{}, {pf}{}, {}, {});\n'.format(funcPrintName, lenPrintName, valuePrintName, flagBitsName, allFlags, lenValue.name, value.name, lenValueRequired, valueRequired, pf=prefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600850 return checkExpr
851 #
852 # Generate pNext check string
Mark Lobodzinski3c828522017-06-26 13:05:57 -0600853 def makeStructNextCheck(self, prefix, value, funcPrintName, valuePrintName, postProcSpec, struct_type_name):
Mark Lobodzinski85672672016-10-13 08:36:42 -0600854 checkExpr = []
855 # Generate an array of acceptable VkStructureType values for pNext
856 extStructCount = 0
857 extStructVar = 'NULL'
858 extStructNames = 'NULL'
Mark Lobodzinski3c828522017-06-26 13:05:57 -0600859 vuid = self.GetVuid("VUID-%s-pNext-pNext" % struct_type_name)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600860 if value.extstructs:
Mike Schuchardtc73d07e2017-07-12 10:10:01 -0600861 extStructVar = 'allowed_structs_{}'.format(struct_type_name)
862 extStructCount = 'ARRAY_SIZE({})'.format(extStructVar)
Mike Schuchardta40d0b02017-07-23 12:47:47 -0600863 extStructNames = '"' + ', '.join(value.extstructs) + '"'
864 checkExpr.append('const VkStructureType {}[] = {{ {} }};\n'.format(extStructVar, ', '.join([self.getStructType(s) for s in value.extstructs])))
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600865 checkExpr.append('skip |= validate_struct_pnext(local_data->report_data, "{}", {ppp}"{}"{pps}, {}, {}{}, {}, {}, GeneratedHeaderVersion, {});\n'.format(
Mark Lobodzinski3c828522017-06-26 13:05:57 -0600866 funcPrintName, valuePrintName, extStructNames, prefix, value.name, extStructCount, extStructVar, vuid, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600867 return checkExpr
868 #
869 # Generate the pointer check string
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -0600870 def makePointerCheck(self, prefix, value, lenValue, valueRequired, lenValueRequired, lenPtrRequired, funcPrintName, lenPrintName, valuePrintName, postProcSpec, struct_type_name):
Mark Lobodzinski85672672016-10-13 08:36:42 -0600871 checkExpr = []
Mark Lobodzinskidead0b62017-06-28 13:22:03 -0600872 vuid_tag_name = struct_type_name if struct_type_name is not None else funcPrintName
Mark Lobodzinski85672672016-10-13 08:36:42 -0600873 if lenValue:
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -0600874 count_required_vuid = self.GetVuid("VUID-%s-%s-arraylength" % (vuid_tag_name, lenValue.name))
875 array_required_vuid = self.GetVuid("VUID-%s-%s-parameter" % (vuid_tag_name, value.name))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600876 # This is an array with a pointer to a count value
877 if lenValue.ispointer:
878 # If count and array parameters are optional, there will be no validation
879 if valueRequired == 'true' or lenPtrRequired == 'true' or lenValueRequired == 'true':
880 # When the length parameter is a pointer, there is an extra Boolean parameter in the function call to indicate if it is required
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600881 checkExpr.append('skip |= validate_array(local_data->report_data, "{}", {ppp}"{ldn}"{pps}, {ppp}"{dn}"{pps}, {pf}{ln}, {pf}{vn}, {}, {}, {}, {}, {});\n'.format(
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -0600882 funcPrintName, lenPtrRequired, lenValueRequired, valueRequired, count_required_vuid, array_required_vuid, ln=lenValue.name, ldn=lenPrintName, dn=valuePrintName, vn=value.name, pf=prefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600883 # This is an array with an integer count value
884 else:
885 # If count and array parameters are optional, there will be no validation
886 if valueRequired == 'true' or lenValueRequired == 'true':
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -0600887 if value.type != 'char':
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600888 checkExpr.append('skip |= validate_array(local_data->report_data, "{}", {ppp}"{ldn}"{pps}, {ppp}"{dn}"{pps}, {pf}{ln}, {pf}{vn}, {}, {}, {}, {});\n'.format(
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -0600889 funcPrintName, lenValueRequired, valueRequired, count_required_vuid, array_required_vuid, ln=lenValue.name, ldn=lenPrintName, dn=valuePrintName, vn=value.name, pf=prefix, **postProcSpec))
890 else:
891 # Arrays of strings receive special processing
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600892 checkExpr.append('skip |= validate_string_array(local_data->report_data, "{}", {ppp}"{ldn}"{pps}, {ppp}"{dn}"{pps}, {pf}{ln}, {pf}{vn}, {}, {}, {}, {});\n'.format(
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -0600893 funcPrintName, lenValueRequired, valueRequired, count_required_vuid, array_required_vuid, ln=lenValue.name, ldn=lenPrintName, dn=valuePrintName, vn=value.name, pf=prefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600894 if checkExpr:
895 if lenValue and ('->' in lenValue.name):
896 # Add checks to ensure the validation call does not dereference a NULL pointer to obtain the count
897 checkExpr = self.genCheckedLengthCall(lenValue.name, checkExpr)
898 # This is an individual struct that is not allowed to be NULL
899 elif not value.isoptional:
900 # Function pointers need a reinterpret_cast to void*
Mark Lobodzinskidead0b62017-06-28 13:22:03 -0600901 ptr_required_vuid = self.GetVuid("VUID-%s-%s-parameter" % (vuid_tag_name, value.name))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600902 if value.type[:4] == 'PFN_':
Mark Lobodzinski02fa1972017-06-28 14:46:14 -0600903 allocator_dict = {'pfnAllocation': '002004f0',
904 'pfnReallocation': '002004f2',
905 'pfnFree': '002004f4',
906 'pfnInternalAllocation': '002004f6'
907 }
908 vuid = allocator_dict.get(value.name)
909 if vuid is not None:
910 ptr_required_vuid = 'VALIDATION_ERROR_%s' % vuid
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600911 checkExpr.append('skip |= validate_required_pointer(local_data->report_data, "{}", {ppp}"{}"{pps}, reinterpret_cast<const void*>({}{}), {});\n'.format(funcPrintName, valuePrintName, prefix, value.name, ptr_required_vuid, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600912 else:
Mark Lobodzinskid4950072017-08-01 13:02:20 -0600913 checkExpr.append('skip |= validate_required_pointer(local_data->report_data, "{}", {ppp}"{}"{pps}, {}{}, {});\n'.format(funcPrintName, valuePrintName, prefix, value.name, ptr_required_vuid, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600914 return checkExpr
915 #
916 # Process struct member validation code, performing name suibstitution if required
917 def processStructMemberCode(self, line, funcName, memberNamePrefix, memberDisplayNamePrefix, postProcSpec):
918 # Build format specifier list
919 kwargs = {}
920 if '{postProcPrefix}' in line:
921 # If we have a tuple that includes a format string and format parameters, need to use ParameterName class
922 if type(memberDisplayNamePrefix) is tuple:
923 kwargs['postProcPrefix'] = 'ParameterName('
924 else:
925 kwargs['postProcPrefix'] = postProcSpec['ppp']
926 if '{postProcSuffix}' in line:
927 # If we have a tuple that includes a format string and format parameters, need to use ParameterName class
928 if type(memberDisplayNamePrefix) is tuple:
929 kwargs['postProcSuffix'] = ', ParameterName::IndexVector{{ {}{} }})'.format(postProcSpec['ppi'], memberDisplayNamePrefix[1])
930 else:
931 kwargs['postProcSuffix'] = postProcSpec['pps']
932 if '{postProcInsert}' in line:
933 # If we have a tuple that includes a format string and format parameters, need to use ParameterName class
934 if type(memberDisplayNamePrefix) is tuple:
935 kwargs['postProcInsert'] = '{}{}, '.format(postProcSpec['ppi'], memberDisplayNamePrefix[1])
936 else:
937 kwargs['postProcInsert'] = postProcSpec['ppi']
938 if '{funcName}' in line:
939 kwargs['funcName'] = funcName
940 if '{valuePrefix}' in line:
941 kwargs['valuePrefix'] = memberNamePrefix
942 if '{displayNamePrefix}' in line:
943 # Check for a tuple that includes a format string and format parameters to be used with the ParameterName class
944 if type(memberDisplayNamePrefix) is tuple:
945 kwargs['displayNamePrefix'] = memberDisplayNamePrefix[0]
946 else:
947 kwargs['displayNamePrefix'] = memberDisplayNamePrefix
948
949 if kwargs:
950 # Need to escape the C++ curly braces
951 if 'IndexVector' in line:
952 line = line.replace('IndexVector{ ', 'IndexVector{{ ')
953 line = line.replace(' }),', ' }}),')
954 return line.format(**kwargs)
955 return line
956 #
957 # Process struct validation code for inclusion in function or parent struct validation code
958 def expandStructCode(self, lines, funcName, memberNamePrefix, memberDisplayNamePrefix, indent, output, postProcSpec):
959 for line in lines:
960 if output:
961 output[-1] += '\n'
962 if type(line) is list:
963 for sub in line:
964 output.append(self.processStructMemberCode(indent + sub, funcName, memberNamePrefix, memberDisplayNamePrefix, postProcSpec))
965 else:
966 output.append(self.processStructMemberCode(indent + line, funcName, memberNamePrefix, memberDisplayNamePrefix, postProcSpec))
967 return output
968 #
969 # Process struct pointer/array validation code, perfoeming name substitution if required
970 def expandStructPointerCode(self, prefix, value, lenValue, funcName, valueDisplayName, postProcSpec):
971 expr = []
972 expr.append('if ({}{} != NULL)\n'.format(prefix, value.name))
973 expr.append('{')
974 indent = self.incIndent(None)
975 if lenValue:
976 # Need to process all elements in the array
977 indexName = lenValue.name.replace('Count', 'Index')
978 expr[-1] += '\n'
Mark Young39389872017-01-19 21:10:49 -0700979 if lenValue.ispointer:
980 # If the length value is a pointer, de-reference it for the count.
981 expr.append(indent + 'for (uint32_t {iname} = 0; {iname} < *{}{}; ++{iname})\n'.format(prefix, lenValue.name, iname=indexName))
982 else:
983 expr.append(indent + 'for (uint32_t {iname} = 0; {iname} < {}{}; ++{iname})\n'.format(prefix, lenValue.name, iname=indexName))
Mark Lobodzinski85672672016-10-13 08:36:42 -0600984 expr.append(indent + '{')
985 indent = self.incIndent(indent)
986 # Prefix for value name to display in error message
Mark Lobodzinski6f82eb52016-12-05 07:38:41 -0700987 if value.ispointer == 2:
988 memberNamePrefix = '{}{}[{}]->'.format(prefix, value.name, indexName)
989 memberDisplayNamePrefix = ('{}[%i]->'.format(valueDisplayName), indexName)
990 else:
991 memberNamePrefix = '{}{}[{}].'.format(prefix, value.name, indexName)
992 memberDisplayNamePrefix = ('{}[%i].'.format(valueDisplayName), indexName)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600993 else:
994 memberNamePrefix = '{}{}->'.format(prefix, value.name)
995 memberDisplayNamePrefix = '{}->'.format(valueDisplayName)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600996 # Expand the struct validation lines
997 expr = self.expandStructCode(self.validatedStructs[value.type], funcName, memberNamePrefix, memberDisplayNamePrefix, indent, expr, postProcSpec)
Mark Lobodzinski85672672016-10-13 08:36:42 -0600998 if lenValue:
999 # Close if and for scopes
1000 indent = self.decIndent(indent)
1001 expr.append(indent + '}\n')
1002 expr.append('}\n')
1003 return expr
1004 #
1005 # Generate the parameter checking code
1006 def genFuncBody(self, funcName, values, valuePrefix, displayNamePrefix, structTypeName):
1007 lines = [] # Generated lines of code
1008 unused = [] # Unused variable names
1009 for value in values:
1010 usedLines = []
1011 lenParam = None
1012 #
1013 # Prefix and suffix for post processing of parameter names for struct members. Arrays of structures need special processing to include the array index in the full parameter name.
1014 postProcSpec = {}
1015 postProcSpec['ppp'] = '' if not structTypeName else '{postProcPrefix}'
1016 postProcSpec['pps'] = '' if not structTypeName else '{postProcSuffix}'
1017 postProcSpec['ppi'] = '' if not structTypeName else '{postProcInsert}'
1018 #
1019 # Generate the full name of the value, which will be printed in the error message, by adding the variable prefix to the value name
1020 valueDisplayName = '{}{}'.format(displayNamePrefix, value.name)
1021 #
1022 # Check for NULL pointers, ignore the inout count parameters that
1023 # will be validated with their associated array
1024 if (value.ispointer or value.isstaticarray) and not value.iscount:
Mark Lobodzinski85672672016-10-13 08:36:42 -06001025 # Parameters for function argument generation
1026 req = 'true' # Paramerter cannot be NULL
1027 cpReq = 'true' # Count pointer cannot be NULL
1028 cvReq = 'true' # Count value cannot be 0
1029 lenDisplayName = None # Name of length parameter to print with validation messages; parameter name with prefix applied
Mark Lobodzinski85672672016-10-13 08:36:42 -06001030 # Generate required/optional parameter strings for the pointer and count values
1031 if value.isoptional:
1032 req = 'false'
1033 if value.len:
1034 # The parameter is an array with an explicit count parameter
1035 lenParam = self.getLenParam(values, value.len)
1036 lenDisplayName = '{}{}'.format(displayNamePrefix, lenParam.name)
1037 if lenParam.ispointer:
1038 # Count parameters that are pointers are inout
1039 if type(lenParam.isoptional) is list:
1040 if lenParam.isoptional[0]:
1041 cpReq = 'false'
1042 if lenParam.isoptional[1]:
1043 cvReq = 'false'
1044 else:
1045 if lenParam.isoptional:
1046 cpReq = 'false'
1047 else:
1048 if lenParam.isoptional:
1049 cvReq = 'false'
1050 #
1051 # The parameter will not be processes when tagged as 'noautovalidity'
1052 # For the pointer to struct case, the struct pointer will not be validated, but any
1053 # members not tagged as 'noatuvalidity' will be validated
1054 if value.noautovalidity:
1055 # Log a diagnostic message when validation cannot be automatically generated and must be implemented manually
1056 self.logMsg('diag', 'ParameterValidation: No validation for {} {}'.format(structTypeName if structTypeName else funcName, value.name))
1057 else:
Mark Lobodzinski85672672016-10-13 08:36:42 -06001058 # If this is a pointer to a struct with an sType field, verify the type
1059 if value.type in self.structTypes:
Mark Lobodzinski9cf24dd2017-06-28 14:23:22 -06001060 usedLines += self.makeStructTypeCheck(valuePrefix, value, lenParam, req, cvReq, cpReq, funcName, lenDisplayName, valueDisplayName, postProcSpec, structTypeName)
Mark Lobodzinski85672672016-10-13 08:36:42 -06001061 # If this is an input handle array that is not allowed to contain NULL handles, verify that none of the handles are VK_NULL_HANDLE
1062 elif value.type in self.handleTypes and value.isconst and not self.isHandleOptional(value, lenParam):
1063 usedLines += self.makeHandleCheck(valuePrefix, value, lenParam, req, cvReq, funcName, lenDisplayName, valueDisplayName, postProcSpec)
1064 elif value.type in self.flags and value.isconst:
1065 usedLines += self.makeFlagsArrayCheck(valuePrefix, value, lenParam, req, cvReq, funcName, lenDisplayName, valueDisplayName, postProcSpec)
1066 elif value.isbool and value.isconst:
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001067 usedLines.append('skip |= validate_bool32_array(local_data->report_data, "{}", {ppp}"{}"{pps}, {ppp}"{}"{pps}, {pf}{}, {pf}{}, {}, {});\n'.format(funcName, lenDisplayName, valueDisplayName, lenParam.name, value.name, cvReq, req, pf=valuePrefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001068 elif value.israngedenum and value.isconst:
Mark Lobodzinskiaff801e2017-07-25 15:29:57 -06001069 enum_value_list = 'All%sEnums' % value.type
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001070 usedLines.append('skip |= validate_ranged_enum_array(local_data->report_data, "{}", {ppp}"{}"{pps}, {ppp}"{}"{pps}, "{}", {}, {pf}{}, {pf}{}, {}, {});\n'.format(funcName, lenDisplayName, valueDisplayName, value.type, enum_value_list, lenParam.name, value.name, cvReq, req, pf=valuePrefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001071 elif value.name == 'pNext':
1072 # We need to ignore VkDeviceCreateInfo and VkInstanceCreateInfo, as the loader manipulates them in a way that is not documented in vk.xml
1073 if not structTypeName in ['VkDeviceCreateInfo', 'VkInstanceCreateInfo']:
Mark Lobodzinski3c828522017-06-26 13:05:57 -06001074 usedLines += self.makeStructNextCheck(valuePrefix, value, funcName, valueDisplayName, postProcSpec, structTypeName)
Mark Lobodzinski85672672016-10-13 08:36:42 -06001075 else:
Mark Lobodzinski1e707bd2017-06-28 10:54:55 -06001076 usedLines += self.makePointerCheck(valuePrefix, value, lenParam, req, cvReq, cpReq, funcName, lenDisplayName, valueDisplayName, postProcSpec, structTypeName)
Mark Lobodzinski85672672016-10-13 08:36:42 -06001077 # If this is a pointer to a struct (input), see if it contains members that need to be checked
1078 if value.type in self.validatedStructs and value.isconst:
1079 usedLines.append(self.expandStructPointerCode(valuePrefix, value, lenParam, funcName, valueDisplayName, postProcSpec))
1080 # Non-pointer types
1081 else:
Mark Lobodzinski85672672016-10-13 08:36:42 -06001082 # The parameter will not be processes when tagged as 'noautovalidity'
1083 # For the struct case, the struct type will not be validated, but any
1084 # members not tagged as 'noatuvalidity' will be validated
1085 if value.noautovalidity:
1086 # Log a diagnostic message when validation cannot be automatically generated and must be implemented manually
1087 self.logMsg('diag', 'ParameterValidation: No validation for {} {}'.format(structTypeName if structTypeName else funcName, value.name))
1088 else:
Mark Lobodzinski024b2822017-06-27 13:22:05 -06001089 vuid_name_tag = structTypeName if structTypeName is not None else funcName
Mark Lobodzinski85672672016-10-13 08:36:42 -06001090 if value.type in self.structTypes:
1091 stype = self.structTypes[value.type]
Mark Lobodzinski06954ea2017-06-21 12:21:45 -06001092 vuid = self.GetVuid("VUID-%s-sType-sType" % value.type)
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001093 usedLines.append('skip |= validate_struct_type(local_data->report_data, "{}", {ppp}"{}"{pps}, "{sv}", &({}{vn}), {sv}, false, {});\n'.format(
Mark Lobodzinski06954ea2017-06-21 12:21:45 -06001094 funcName, valueDisplayName, valuePrefix, vuid, vn=value.name, sv=stype.value, vt=value.type, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001095 elif value.type in self.handleTypes:
1096 if not self.isHandleOptional(value, None):
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001097 usedLines.append('skip |= validate_required_handle(local_data->report_data, "{}", {ppp}"{}"{pps}, {}{});\n'.format(funcName, valueDisplayName, valuePrefix, value.name, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001098 elif value.type in self.flags:
1099 flagBitsName = value.type.replace('Flags', 'FlagBits')
1100 if not flagBitsName in self.flagBits:
Mark Lobodzinskid0b0c512017-06-28 12:06:41 -06001101 vuid = self.GetVuid("VUID-%s-%s-zerobitmask" % (vuid_name_tag, value.name))
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001102 usedLines.append('skip |= validate_reserved_flags(local_data->report_data, "{}", {ppp}"{}"{pps}, {pf}{}, {});\n'.format(funcName, valueDisplayName, value.name, vuid, pf=valuePrefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001103 else:
Mark Lobodzinskie643fcc2017-06-26 16:27:15 -06001104 if value.isoptional:
1105 flagsRequired = 'false'
Mark Lobodzinski024b2822017-06-27 13:22:05 -06001106 vuid = self.GetVuid("VUID-%s-%s-parameter" % (vuid_name_tag, value.name))
Mark Lobodzinskie643fcc2017-06-26 16:27:15 -06001107 else:
1108 flagsRequired = 'true'
Mark Lobodzinskie643fcc2017-06-26 16:27:15 -06001109 vuid = self.GetVuid("VUID-%s-%s-requiredbitmask" % (vuid_name_tag, value.name))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001110 allFlagsName = 'All' + flagBitsName
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001111 usedLines.append('skip |= validate_flags(local_data->report_data, "{}", {ppp}"{}"{pps}, "{}", {}, {pf}{}, {}, false, {});\n'.format(funcName, valueDisplayName, flagBitsName, allFlagsName, value.name, flagsRequired, vuid, pf=valuePrefix, **postProcSpec))
Mike Schuchardt47619c82017-05-31 09:14:22 -06001112 elif value.type in self.flagBits:
1113 flagsRequired = 'false' if value.isoptional else 'true'
1114 allFlagsName = 'All' + value.type
Mark Lobodzinski024b2822017-06-27 13:22:05 -06001115 vuid = self.GetVuid("VUID-%s-%s-parameter" % (vuid_name_tag, value.name))
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001116 usedLines.append('skip |= validate_flags(local_data->report_data, "{}", {ppp}"{}"{pps}, "{}", {}, {pf}{}, {}, true, {});\n'.format(funcName, valueDisplayName, value.type, allFlagsName, value.name, flagsRequired, vuid, pf=valuePrefix, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001117 elif value.isbool:
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001118 usedLines.append('skip |= validate_bool32(local_data->report_data, "{}", {ppp}"{}"{pps}, {}{});\n'.format(funcName, valueDisplayName, valuePrefix, value.name, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001119 elif value.israngedenum:
Mark Lobodzinski42eb3c32017-06-28 11:47:22 -06001120 vuid = self.GetVuid("VUID-%s-%s-parameter" % (vuid_name_tag, value.name))
Mark Lobodzinski74cb45f2017-07-25 15:10:29 -06001121 enum_value_list = 'All%sEnums' % value.type
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001122 usedLines.append('skip |= validate_ranged_enum(local_data->report_data, "{}", {ppp}"{}"{pps}, "{}", {}, {}{}, {});\n'.format(funcName, valueDisplayName, value.type, enum_value_list, valuePrefix, value.name, vuid, **postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001123 # If this is a struct, see if it contains members that need to be checked
1124 if value.type in self.validatedStructs:
1125 memberNamePrefix = '{}{}.'.format(valuePrefix, value.name)
1126 memberDisplayNamePrefix = '{}.'.format(valueDisplayName)
1127 usedLines.append(self.expandStructCode(self.validatedStructs[value.type], funcName, memberNamePrefix, memberDisplayNamePrefix, '', [], postProcSpec))
Mark Lobodzinski85672672016-10-13 08:36:42 -06001128 # Append the parameter check to the function body for the current command
1129 if usedLines:
1130 # Apply special conditional checks
1131 if value.condition:
1132 usedLines = self.genConditionalCall(valuePrefix, value.condition, usedLines)
1133 lines += usedLines
1134 elif not value.iscount:
1135 # If no expression was generated for this value, it is unreferenced by the validation function, unless
1136 # it is an array count, which is indirectly referenced for array valiadation.
1137 unused.append(value.name)
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001138 if not lines:
1139 lines.append('// No xml-driven validation\n')
Mark Lobodzinski85672672016-10-13 08:36:42 -06001140 return lines, unused
1141 #
1142 # Generate the struct member check code from the captured data
1143 def processStructMemberData(self):
1144 indent = self.incIndent(None)
1145 for struct in self.structMembers:
1146 #
1147 # The string returned by genFuncBody will be nested in an if check for a NULL pointer, so needs its indent incremented
1148 lines, unused = self.genFuncBody('{funcName}', struct.members, '{valuePrefix}', '{displayNamePrefix}', struct.name)
1149 if lines:
1150 self.validatedStructs[struct.name] = lines
1151 #
1152 # Generate the command param check code from the captured data
1153 def processCmdData(self):
1154 indent = self.incIndent(None)
1155 for command in self.commands:
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001156 just_validate = False
1157 if command.name in self.validate_only:
1158 just_validate = True
Mark Lobodzinski85672672016-10-13 08:36:42 -06001159 # Skip first parameter if it is a dispatch handle (everything except vkCreateInstance)
1160 startIndex = 0 if command.name == 'vkCreateInstance' else 1
1161 lines, unused = self.genFuncBody(command.name, command.params[startIndex:], '', '', None)
Mark Lobodzinski3f10bfe2017-08-23 15:23:23 -06001162 # Cannot validate extension dependencies for device extension APIs having a physical device as their dispatchable object
Mike Schuchardtafd00482017-08-24 15:15:02 -06001163 if (command.name in self.required_extensions) and (self.extension_type != 'device' or command.params[0].type != 'VkPhysicalDevice'):
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -06001164 ext_test = ''
Mike Schuchardtafd00482017-08-24 15:15:02 -06001165 for ext in self.required_extensions[command.name]:
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -06001166 ext_name_define = ''
1167 ext_enable_name = ''
1168 for extension in self.registry.extensions:
1169 if extension.attrib['name'] == ext:
1170 ext_name_define = extension[0][1].get('name')
1171 ext_enable_name = ext_name_define.lower()
1172 ext_enable_name = re.sub('_extension_name', '', ext_enable_name)
1173 break
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001174 ext_test = 'if (!local_data->extensions.%s) skip |= OutputExtensionError(local_data, "%s", %s);\n' % (ext_enable_name, command.name, ext_name_define)
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -06001175 lines.insert(0, ext_test)
Mark Lobodzinski85672672016-10-13 08:36:42 -06001176 if lines:
1177 cmdDef = self.getCmdDef(command) + '\n'
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001178 # For a validation-only routine, change the function declaration
1179 if just_validate:
1180 jv_def = '// Generated function handles validation only -- API definition is in non-generated source\n'
1181 jv_def += 'extern %s\n\n' % command.cdecl
1182 cmdDef = 'bool parameter_validation_' + cmdDef.split('VKAPI_CALL ',1)[1]
1183 if command.name == 'vkCreateInstance':
1184 cmdDef = cmdDef.replace('(\n', '(\n VkInstance instance,\n')
1185 cmdDef = jv_def + cmdDef
Mark Lobodzinski85672672016-10-13 08:36:42 -06001186 cmdDef += '{\n'
Mark Lobodzinski26112592017-05-30 12:02:17 -06001187
Gabríel Arthúr Péturssona3b5d672017-08-19 16:44:45 +00001188 # Add list of commands to skip -- just generate the routine signature and put the manual source in parameter_validation_utils.cpp
1189 if command.params[0].type in ["VkInstance", "VkPhysicalDevice"] or command.name == 'vkCreateInstance':
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001190 map_name = 'instance_layer_data_map'
1191 map_type = 'instance_layer_data'
1192 else:
1193 map_name = 'layer_data_map'
1194 map_type = 'layer_data'
1195 instance_param = command.params[0].name
1196 if command.name == 'vkCreateInstance':
1197 instance_param = 'instance'
1198 layer_data = ' %s *local_data = GetLayerDataPtr(get_dispatch_key(%s), %s);\n' % (map_type, instance_param, map_name)
1199 cmdDef += layer_data
1200 cmdDef += '%sbool skip = false;\n' % indent
1201 if not just_validate:
1202 if command.result != '':
Jamie Madillfc315192017-11-08 14:11:26 -05001203 if command.result == "VkResult":
1204 cmdDef += indent + '%s result = VK_ERROR_VALIDATION_FAILED_EXT;\n' % command.result
1205 elif command.result == "VkBool32":
1206 cmdDef += indent + '%s result = VK_FALSE;\n' % command.result
1207 else:
1208 raise Exception("Unknown result type: " + command.result)
1209
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001210 cmdDef += '%sstd::unique_lock<std::mutex> lock(global_lock);\n' % indent
Mark Lobodzinski85672672016-10-13 08:36:42 -06001211 for line in lines:
1212 cmdDef += '\n'
1213 if type(line) is list:
1214 for sub in line:
1215 cmdDef += indent + sub
1216 else:
1217 cmdDef += indent + line
1218 cmdDef += '\n'
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001219 if not just_validate:
1220 # Generate parameter list for manual fcn and down-chain calls
1221 params_text = ''
1222 for param in command.params:
1223 params_text += '%s, ' % param.name
1224 params_text = params_text[:-2]
1225 # Generate call to manual function if its function pointer is non-null
Mark Lobodzinski78a12a92017-08-08 14:16:51 -06001226 cmdDef += '%sPFN_manual_%s custom_func = (PFN_manual_%s)custom_functions["%s"];\n' % (indent, command.name, command.name, command.name)
1227 cmdDef += '%sif (custom_func != nullptr) {\n' % indent
1228 cmdDef += ' %sskip |= custom_func(%s);\n' % (indent, params_text)
Mark Lobodzinskid4950072017-08-01 13:02:20 -06001229 cmdDef += '%s}\n\n' % indent
1230 # Release the validation lock
1231 cmdDef += '%slock.unlock();\n' % indent
1232 # Generate skip check and down-chain call
1233 cmdDef += '%sif (!skip) {\n' % indent
1234 down_chain_call = ' %s' % indent
1235 if command.result != '':
1236 down_chain_call += ' result = '
1237 # Generate down-chain API call
1238 api_call = '%s(%s);' % (command.name, params_text)
1239 down_chain_call += 'local_data->dispatch_table.%s\n' % api_call[2:]
1240 cmdDef += down_chain_call
1241 cmdDef += '%s}\n' % indent
1242 if command.result != '':
1243 cmdDef += '%sreturn result;\n' % indent
1244 else:
1245 cmdDef += '%sreturn skip;\n' % indent
Mark Lobodzinski85672672016-10-13 08:36:42 -06001246 cmdDef += '}\n'
Mark Lobodzinskid8b7e022017-06-01 15:10:13 -06001247 self.validation.append(cmdDef)