Home | History | Annotate | Download | only in vulkan-validation-layers
      1 #!/usr/bin/env python3
      2 # Copyright (c) 2015-2016 The Khronos Group Inc.
      3 # Copyright (c) 2015-2016 Valve Corporation
      4 # Copyright (c) 2015-2016 LunarG, Inc.
      5 # Copyright (c) 2015-2016 Google Inc.
      6 #
      7 # Licensed under the Apache License, Version 2.0 (the "License");
      8 # you may not use this file except in compliance with the License.
      9 # You may obtain a copy of the License at
     10 #
     11 #     http://www.apache.org/licenses/LICENSE-2.0
     12 #
     13 # Unless required by applicable law or agreed to in writing, software
     14 # distributed under the License is distributed on an "AS IS" BASIS,
     15 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
     16 # See the License for the specific language governing permissions and
     17 # limitations under the License.
     18 #
     19 # Author: Tobin Ehlis <tobin (at] lunarg.com>
     20 
     21 import argparse
     22 import os
     23 import sys
     24 import vulkan
     25 import platform
     26 
     27 # vk_layer_documentation_generate.py overview
     28 # This script is intended to generate documentation based on vulkan layers
     29 #  It parses known validation layer headers for details of the validation checks
     30 #  It parses validation layer source files for specific code where checks are implemented
     31 #  structs in a human-readable txt format, as well as utility functions
     32 #  to print enum values as strings
     33 
     34 # NOTE : Initially the script is performing validation of a hand-written document
     35 #  Right now it does 3 checks:
     36 #  1. Verify ENUM codes declared in source are documented
     37 #  2. Verify ENUM codes in document are declared in source
     38 #  3. Verify API function names in document are in the actual API header (vulkan.py)
     39 # Currently script will flag errors in all of these cases
     40 
     41 # TODO : Need a formal specification of the syntax for doc generation
     42 #  Initially, these are the basics:
     43 #  1. Validation checks have unique ENUM values defined in validation layer header
     44 #  2. ENUM includes comments for 1-line overview of check and more detailed description
     45 #  3. Actual code implementing checks includes ENUM value in callback
     46 #  4. Code to test checks should include reference to ENUM
     47 
     48 
     49 # TODO : Need list of known validation layers to use as default input
     50 #  Just a couple of flat lists right now, but may need to make this input file
     51 #  or at least a more dynamic data structure
     52 layer_inputs = { 'draw_state' : {'header' : 'layers/core_validation_error_enums.h',
     53                                  'source' : 'layers/core_validation.cpp',
     54                                  'generated' : False,
     55                                  'error_enum' : 'DRAW_STATE_ERROR'},
     56                  'shader_checker' : {'header' : 'layers/core_validation_error_enums.h',
     57                                      'source' : 'layers/core_validation.cpp',
     58                                      'generated' : False,
     59                                      'error_enum' : 'SHADER_CHECKER_ERROR'},
     60                  'mem_tracker' : {'header' : 'layers/core_validation_error_enums.h',
     61                                   'source' : 'layers/core_validation.cpp',
     62                                   'generated' : False,
     63                                   'error_enum' : 'MEM_TRACK_ERROR'},
     64                  'device_limits' : {'header' : 'layers/core_validation_error_enums.h',
     65                                     'source' : 'layers/core_validation.cpp',
     66                                     'generated' : False,
     67                                     'error_enum' : 'DEV_LIMITS_ERROR',},
     68                  'object_tracker' : {'header' : 'layers/object_tracker.h',
     69                                      'source' : 'layers/object_tracker.cpp',
     70                                      'generated' : False,
     71                                      'error_enum' : 'OBJECT_TRACK_ERROR',},
     72                  'threading' : {'header' : 'layers/threading.h',
     73                                 'source' : 'dbuild/layers/threading.cpp',
     74                                 'generated' : True,
     75                                 'error_enum' : 'THREADING_CHECKER_ERROR'},
     76                  'image' : {'header' : 'layers/image.h',
     77                             'source' : 'layers/image.cpp',
     78                             'generated' : False,
     79                             'error_enum' : 'IMAGE_ERROR',},
     80                  'swapchain' : {'header' : 'layers/swapchain.h',
     81                                 'source' : 'layers/swapchain.cpp',
     82                                 'generated' : False,
     83                                 'error_enum' : 'SWAPCHAIN_ERROR',},
     84                  'parameter_validation' : {'header' : 'layers/parameter_validation_utils.h',
     85                                            'source' : 'layers/parameter_validation.cpp',
     86                                            'generated' : False,
     87                                            'error_enum' : 'ErrorCode',},
     88     }
     89 
     90 builtin_headers = [layer_inputs[ln]['header'] for ln in layer_inputs]
     91 builtin_source = [layer_inputs[ln]['source'] for ln in layer_inputs]
     92 builtin_tests = ['tests/layer_validation_tests.cpp', ]
     93 
     94 # List of extensions in layers that are included in documentation, but not in vulkan.py API set
     95 layer_extension_functions = ['objTrackGetObjects', 'objTrackGetObjectsOfType']
     96 
     97 def handle_args():
     98     parser = argparse.ArgumentParser(description='Generate layer documenation from source.')
     99     parser.add_argument('--in_headers', required=False, default=builtin_headers, help='The input layer header files from which code will be generated.')
    100     parser.add_argument('--in_source', required=False, default=builtin_source, help='The input layer source files from which code will be generated.')
    101     parser.add_argument('--test_source', required=False, default=builtin_tests, help='The input test source files from which code will be generated.')
    102     parser.add_argument('--layer_doc', required=False, default='layers/vk_validation_layer_details.md', help='Existing layer document to be validated against actual layers.')
    103     parser.add_argument('--validate', action='store_true', default=False, help='Validate that there are no mismatches between layer documentation and source. This includes cross-checking the validation checks, and making sure documented Vulkan API calls exist.')
    104     parser.add_argument('--print_structs', action='store_true', default=False, help='Primarily a debug option that prints out internal data structs used to generate layer docs.')
    105     parser.add_argument('--print_doc_checks', action='store_true', default=False, help='Primarily a debug option that prints out all of the checks that are documented.')
    106     return parser.parse_args()
    107 
    108 # Little helper class for coloring cmd line output
    109 class bcolors:
    110 
    111     def __init__(self):
    112         self.GREEN = '\033[0;32m'
    113         self.RED = '\033[0;31m'
    114         self.YELLOW = '\033[1;33m'
    115         self.ENDC = '\033[0m'
    116         if 'Linux' != platform.system():
    117             self.GREEN = ''
    118             self.RED = ''
    119             self.YELLOW = ''
    120             self.ENDC = ''
    121 
    122     def green(self):
    123         return self.GREEN
    124 
    125     def red(self):
    126         return self.RED
    127 
    128     def yellow(self):
    129         return self.YELLOW
    130 
    131     def endc(self):
    132         return self.ENDC
    133 
    134 # Class to parse the validation layer test source and store testnames
    135 class TestParser:
    136     def __init__(self, test_file_list, test_group_name=['VkLayerTest', 'VkPositiveLayerTest', 'VkWsiEnabledLayerTest']):
    137         self.test_files = test_file_list
    138         self.tests_set = set()
    139         self.test_trigger_txt_list = []
    140         for tg in test_group_name:
    141             self.test_trigger_txt_list.append('TEST_F(%s' % tg)
    142             #print('Test trigger test list: %s' % (self.test_trigger_txt_list))
    143 
    144     # Parse test files into internal data struct
    145     def parse(self):
    146         # For each test file, parse test names into set
    147         grab_next_line = False # handle testname on separate line than wildcard
    148         for test_file in self.test_files:
    149             with open(test_file) as tf:
    150                 for line in tf:
    151                     if True in [line.strip().startswith(comment) for comment in ['//', '/*']]:
    152                         continue
    153 
    154                     if True in [ttt in line for ttt in self.test_trigger_txt_list]:
    155                         #print('Test wildcard in line: %s' % (line))
    156                         testname = line.split(',')[-1]
    157                         testname = testname.strip().strip(' {)')
    158                         #print('Inserting test: "%s"' % (testname))
    159                         if ('' == testname):
    160                             grab_next_line = True
    161                             continue
    162                         self.tests_set.add(testname)
    163                     if grab_next_line: # test name on its own line
    164                         grab_next_line = False
    165                         testname = testname.strip().strip(' {)')
    166                         self.tests_set.add(testname)
    167 
    168 # Class to parse the layer source code and store details in internal data structs
    169 class LayerParser:
    170     def __init__(self, header_file_list, source_file_list):
    171         self.header_files = header_file_list
    172         self.source_files = source_file_list
    173         self.layer_dict = {}
    174         self.api_dict = {}
    175 
    176     # Parse layer header files into internal dict data structs
    177     def parse(self):
    178         # For each header file, parse details into dicts
    179         # TODO : Should have a global dict element to track overall list of checks
    180         store_enum = False
    181         for layer_name in layer_inputs:
    182             hf = layer_inputs[layer_name]['header']
    183             self.layer_dict[layer_name] = {} # initialize a new dict for this layer
    184             self.layer_dict[layer_name]['CHECKS'] = [] # enum of checks is stored in a list
    185             #print('Parsing header file %s as layer name %s' % (hf, layer_name))
    186             with open(hf) as f:
    187                 for line in f:
    188                     if True in [line.strip().startswith(comment) for comment in ['//', '/*']]:
    189                         #print("Skipping comment line: %s" % line)
    190                         # For now skipping lines starting w/ comment, may use these to capture
    191                         #  documentation in the future
    192                         continue
    193                     # Find enums
    194                     if store_enum:
    195                         if '}' in line: # we're done with enum definition
    196                             store_enum = False
    197                             continue
    198                         # grab the enum name as a unique check
    199                         if ',' in line:
    200                             # TODO : When documentation for a check is contained in the source,
    201                             #  this is where we should also capture that documentation so that
    202                             #  it can then be transformed into desired doc format
    203                             enum_name = line.split(',')[0].strip()
    204                             # Flag an error if we have already seen this enum
    205                             if enum_name in self.layer_dict[layer_name]['CHECKS']:
    206                                 print('ERROR : % layer has duplicate error enum: %s' % (layer_name, enum_name))
    207                             self.layer_dict[layer_name]['CHECKS'].append(enum_name)
    208                     # If the line includes 'enum' and the expected enum name, start capturing enums
    209                     if False not in [ex in line for ex in ['enum', layer_inputs[layer_name]['error_enum']]]:
    210                         store_enum = True
    211 
    212         # For each source file, parse into dicts
    213         for sf in self.source_files:
    214             #print('Parsing source file %s' % sf)
    215             pass
    216             # TODO : In the source file we want to see where checks actually occur
    217             #  Need to build function tree of checks so that we know all of the
    218             #  checks that occur under a top-level Vulkan API call
    219             #  Eventually in the validation we can flag ENUMs that aren't being
    220             #  used in the source, and we can document source code lines as well
    221             #  as Vulkan API calls where each specific ENUM check is made
    222 
    223     def print_structs(self):
    224         print('This is where I print the data structs')
    225         for layer in self.layer_dict:
    226             print('Layer %s has %i checks:\n%s' % (layer, len(self.layer_dict[layer]['CHECKS'])-1, "\n\t".join(self.layer_dict[layer]['CHECKS'])))
    227 
    228 # Class to parse hand-written md layer documentation into a dict and then validate its contents
    229 class LayerDoc:
    230     def __init__(self, source_file):
    231         self.layer_doc_filename = source_file
    232         self.txt_color = bcolors()
    233         # Main data struct to store info from layer doc
    234         self.layer_doc_dict = {}
    235         # Comprehensive list of all validation checks recorded in doc
    236         self.enum_list = []
    237 
    238     # Parse the contents of doc into data struct
    239     def parse(self):
    240         layer_name = 'INIT'
    241         parse_layer_details = False
    242         detail_trigger = '| Check | '
    243         parse_pending_work = False
    244         pending_trigger = ' Pending Work'
    245         parse_overview = False
    246         overview_trigger = ' Overview'
    247         enum_prefix = ''
    248 
    249         with open(self.layer_doc_filename) as f:
    250             for line in f:
    251                 if parse_pending_work:
    252                     if '.' in line and line.strip()[0].isdigit():
    253                         todo_item = line.split('.')[1].strip()
    254                         self.layer_doc_dict[layer_name]['pending'].append(todo_item)
    255                 if pending_trigger in line and '##' in line:
    256                     parse_layer_details = False
    257                     parse_pending_work = True
    258                     parse_overview = False
    259                     self.layer_doc_dict[layer_name]['pending'] = []
    260                 if parse_layer_details:
    261                     # Grab details but skip the fomat line with a bunch of '-' chars
    262                     if '|' in line and line.count('-') < 20:
    263                         detail_sections = line.split('|')
    264                         #print("Details elements from line %s: %s" % (line, detail_sections))
    265                         check_name = '%s%s' % (enum_prefix, detail_sections[3].strip())
    266 
    267                         self.enum_list.append(check_name)
    268                         self.layer_doc_dict[layer_name][check_name] = {}
    269                         self.layer_doc_dict[layer_name][check_name]['summary_txt'] = detail_sections[1].strip()
    270                         self.layer_doc_dict[layer_name][check_name]['details_txt'] = detail_sections[2].strip()
    271                         self.layer_doc_dict[layer_name][check_name]['api_list'] = detail_sections[4].split()
    272                         self.layer_doc_dict[layer_name][check_name]['tests'] = detail_sections[5].split()
    273                         self.layer_doc_dict[layer_name][check_name]['notes'] = detail_sections[6].strip()
    274                         # strip any unwanted commas from api and test names
    275                         self.layer_doc_dict[layer_name][check_name]['api_list'] = [a.strip(',') for a in self.layer_doc_dict[layer_name][check_name]['api_list']]
    276                         test_list = [a.strip(',') for a in self.layer_doc_dict[layer_name][check_name]['tests']]
    277                         self.layer_doc_dict[layer_name][check_name]['tests'] = [a.split('.')[-1] for a in test_list]
    278                 # Trigger details parsing when we have table header
    279                 if detail_trigger in line:
    280                     parse_layer_details = True
    281                     parse_pending_work = False
    282                     parse_overview = False
    283                     enum_txt = line.split('|')[3]
    284                     if '*' in enum_txt:
    285                         enum_prefix = enum_txt.split()[-1].strip('*').strip()
    286                         #print('prefix: %s' % enum_prefix)
    287                 if parse_overview:
    288                     self.layer_doc_dict[layer_name]['overview'] += line
    289                 if overview_trigger in line and '##' in line:
    290                     parse_layer_details = False
    291                     parse_pending_work = False
    292                     parse_overview = True
    293                     layer_name = line.split()[1]
    294                     self.layer_doc_dict[layer_name] = {}
    295                     self.layer_doc_dict[layer_name]['overview'] = ''
    296 
    297     # Verify that checks, tests and api references in layer doc match reality
    298     #  Report API calls from doc that are not found in API
    299     #  Report checks from doc that are not in actual layers
    300     #  Report checks from layers that are not captured in doc
    301     #  Report checks from doc that do not have a valid test
    302     def validate(self, layer_dict, tests_set):
    303         #print("tests_set: %s" % (tests_set))
    304         # Count number of errors found and return it
    305         errors_found = 0
    306         warnings_found = 0
    307         # A few checks that are allowed to not have tests
    308         no_test_checks = ['DRAWSTATE_INTERNAL_ERROR', 'DRAWSTATE_OUT_OF_MEMORY', 'MEMTRACK_INTERNAL_ERROR', 'OBJTRACK_INTERNAL_ERROR']
    309         # First we'll go through the doc datastructures and flag any issues
    310         for chk in self.enum_list:
    311             doc_layer_found = False
    312             for real_layer in layer_dict:
    313                 if chk in layer_dict[real_layer]['CHECKS']:
    314                     #print('Found actual layer check %s in doc' % (chk))
    315                     doc_layer_found = True
    316                     continue
    317             if not doc_layer_found:
    318                 print(self.txt_color.red() + 'Actual layers do not contain documented check: %s' % (chk) + self.txt_color.endc())
    319                 errors_found += 1
    320 
    321         # Now go through API names in doc and verify they're real
    322         # First we're going to transform proto names from vulkan.py into single list
    323         core_api_names = [p.name for p in vulkan.core.protos]
    324         wsi_s_names = [p.name for p in vulkan.ext_khr_surface.protos]
    325         wsi_ds_names = [p.name for p in vulkan.ext_khr_device_swapchain.protos]
    326         dbg_rpt_names = [p.name for p in vulkan.ext_debug_report.protos]
    327         api_names = core_api_names + wsi_s_names + wsi_ds_names + dbg_rpt_names
    328         for ln in self.layer_doc_dict:
    329             for chk in self.layer_doc_dict[ln]:
    330                 if chk in ['overview', 'pending']:
    331                     continue
    332                 for api in self.layer_doc_dict[ln][chk]['api_list']:
    333                     if api[2:] not in api_names and api not in layer_extension_functions:
    334                         print(self.txt_color.red() + 'Doc references invalid function: %s' % (api) + self.txt_color.endc())
    335                         errors_found += 1
    336                 # For now warn on missing or invalid tests
    337                 for test in self.layer_doc_dict[ln][chk]['tests']:
    338                     if '*' in test:
    339                         # naive way to handle wildcards, just make sure we have matches on parts
    340                         test_parts = test.split('*')
    341                         for part in test_parts:
    342                             part_found = False
    343                             for t in tests_set:
    344                                 if part in t:
    345                                     part_found = True
    346                                     break
    347                             if not part_found:
    348                                 print(self.txt_color.red() + 'Validation check %s has missing or invalid test : %s' % (chk, test))
    349                                 errors_found += 1
    350                                 break
    351                     elif test not in tests_set and not chk.endswith('_NONE'):
    352                         if test == 'TODO':
    353                             if chk not in no_test_checks:
    354                                 warnings_found += 1
    355                         else:
    356                             print(self.txt_color.red() + 'Validation check %s has missing or invalid test : %s' % (chk, test))
    357                             errors_found += 1
    358         # Now go through all of the actual checks in the layers and make sure they're covered in the doc
    359         for ln in layer_dict:
    360             for chk in layer_dict[ln]['CHECKS']:
    361                 if chk not in self.enum_list:
    362                     print(self.txt_color.red() + 'Doc is missing check: %s' % (chk) + self.txt_color.endc())
    363                     errors_found += 1
    364 
    365         return (errors_found, warnings_found)
    366 
    367     # Print all of the checks captured in the doc
    368     def print_checks(self):
    369         print('Checks captured in doc:\n%s' % ('\n\t'.join(self.enum_list)))
    370 
    371 def main(argv=None):
    372     # Parse args
    373     opts = handle_args()
    374     # Create parser for layer files
    375     layer_parser = LayerParser(opts.in_headers, opts.in_source)
    376     # Parse files into internal data structs
    377     layer_parser.parse()
    378     # Parse test files
    379     test_parser = TestParser(opts.test_source)
    380     test_parser.parse()
    381 
    382     # Generate requested types of output
    383     if opts.print_structs: # Print details of internal data structs
    384         layer_parser.print_structs()
    385 
    386     layer_doc = LayerDoc(opts.layer_doc)
    387     layer_doc.parse()
    388     if opts.print_doc_checks:
    389         layer_doc.print_checks()
    390 
    391     if opts.validate:
    392         (num_errors, num_warnings) = layer_doc.validate(layer_parser.layer_dict, test_parser.tests_set)
    393         txt_color = bcolors()
    394         if (0 == num_warnings):
    395             print(txt_color.green() + 'No warning cases found between %s and implementation' % (os.path.basename(opts.layer_doc)) + txt_color.endc())
    396         else:
    397             print(txt_color.yellow() + 'Found %s warnings due to missing tests. Missing tests are labeled as "TODO" in "%s."' % (num_warnings, opts.layer_doc))
    398         if (0 == num_errors):
    399             print(txt_color.green() + 'No mismatches found between %s and implementation' % (os.path.basename(opts.layer_doc)) + txt_color.endc())
    400         else:
    401             return num_errors
    402     return 0
    403 
    404 if __name__ == "__main__":
    405     sys.exit(main())
    406 
    407