Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
| 2 | |
| 3 | import sys |
| 4 | import os |
| 5 | import subprocess |
| 6 | import tempfile |
| 7 | import re |
| 8 | import itertools |
| 9 | import hashlib |
| 10 | import shutil |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 11 | import argparse |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 12 | |
| 13 | def parse_stats(stats): |
| 14 | m = re.search('([0-9]+) work registers', stats) |
| 15 | registers = int(m.group(1)) if m else 0 |
| 16 | |
| 17 | m = re.search('([0-9]+) uniform registers', stats) |
| 18 | uniform_regs = int(m.group(1)) if m else 0 |
| 19 | |
| 20 | m_list = re.findall('(-?[0-9]+)\s+(-?[0-9]+)\s+(-?[0-9]+)', stats) |
| 21 | alu_short = float(m_list[1][0]) if m_list else 0 |
| 22 | ls_short = float(m_list[1][1]) if m_list else 0 |
| 23 | tex_short = float(m_list[1][2]) if m_list else 0 |
| 24 | alu_long = float(m_list[2][0]) if m_list else 0 |
| 25 | ls_long = float(m_list[2][1]) if m_list else 0 |
| 26 | tex_long = float(m_list[2][2]) if m_list else 0 |
| 27 | |
| 28 | return (registers, uniform_regs, alu_short, ls_short, tex_short, alu_long, ls_long, tex_long) |
| 29 | |
| 30 | def get_shader_type(shader): |
| 31 | _, ext = os.path.splitext(shader) |
| 32 | if ext == '.vert': |
| 33 | return '--vertex' |
| 34 | elif ext == '.frag': |
| 35 | return '--fragment' |
| 36 | elif ext == '.comp': |
| 37 | return '--compute' |
| 38 | elif ext == '.tesc': |
| 39 | return '--tessellation_control' |
| 40 | elif ext == '.tese': |
| 41 | return '--tessellation_evaluation' |
| 42 | elif ext == '.geom': |
| 43 | return '--geometry' |
| 44 | else: |
| 45 | return '' |
| 46 | |
| 47 | def get_shader_stats(shader): |
| 48 | f, path = tempfile.mkstemp() |
| 49 | |
| 50 | os.close(f) |
| 51 | p = subprocess.Popen(['malisc', get_shader_type(shader), '--core', 'Mali-T760', '-V', shader], stdout = subprocess.PIPE, stderr = subprocess.PIPE) |
| 52 | stdout, stderr = p.communicate() |
| 53 | os.remove(path) |
| 54 | |
| 55 | if p.returncode != 0: |
| 56 | print(stderr.decode('utf-8')) |
| 57 | raise OSError('malisc failed') |
| 58 | p.wait() |
| 59 | |
| 60 | returned = stdout.decode('utf-8') |
| 61 | return parse_stats(returned) |
| 62 | |
Hans-Kristian Arntzen | 7652c90 | 2016-04-19 11:13:47 +0200 | [diff] [blame] | 63 | def validate_shader(shader, vulkan): |
| 64 | if vulkan: |
| 65 | subprocess.check_call(['glslangValidator', '-V', shader]) |
| 66 | else: |
| 67 | subprocess.check_call(['glslangValidator', shader]) |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 68 | |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 69 | def cross_compile(shader, vulkan, spirv): |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 70 | spirv_f, spirv_path = tempfile.mkstemp() |
| 71 | glsl_f, glsl_path = tempfile.mkstemp(suffix = os.path.basename(shader)) |
| 72 | os.close(spirv_f) |
| 73 | os.close(glsl_f) |
| 74 | |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 75 | if vulkan or spirv: |
Hans-Kristian Arntzen | dbee4e4 | 2016-05-05 10:16:22 +0200 | [diff] [blame] | 76 | vulkan_glsl_f, vulkan_glsl_path = tempfile.mkstemp(suffix = os.path.basename(shader)) |
| 77 | os.close(vulkan_glsl_f) |
| 78 | |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 79 | if spirv: |
| 80 | subprocess.check_call(['spirv-as', '-o', spirv_path, shader]) |
| 81 | else: |
| 82 | subprocess.check_call(['glslangValidator', '-V' if vulkan else '-G', '-o', spirv_path, shader]) |
| 83 | |
Hans-Kristian Arntzen | 6aa2007 | 2016-05-23 12:25:09 +0200 | [diff] [blame] | 84 | # Workaround Issue #217 in SPIRV-Tools until the issue is resolved. |
| 85 | #if spirv: |
| 86 | # subprocess.check_call(['spirv-val', spirv_path]) |
Hans-Kristian Arntzen | dbee4e4 | 2016-05-05 10:16:22 +0200 | [diff] [blame] | 87 | |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 88 | spirv_cross_path = './spirv-cross' |
| 89 | subprocess.check_call([spirv_cross_path, '--output', glsl_path, spirv_path]) |
Hans-Kristian Arntzen | dbee4e4 | 2016-05-05 10:16:22 +0200 | [diff] [blame] | 90 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 91 | # A shader might not be possible to make valid GLSL from, skip validation for this case. |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 92 | if (not ('nocompat' in glsl_path)) and (not spirv): |
Hans-Kristian Arntzen | dbee4e4 | 2016-05-05 10:16:22 +0200 | [diff] [blame] | 93 | validate_shader(glsl_path, False) |
| 94 | |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 95 | if vulkan or spirv: |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 96 | subprocess.check_call([spirv_cross_path, '--vulkan-semantics', '--output', vulkan_glsl_path, spirv_path]) |
Hans-Kristian Arntzen | dbee4e4 | 2016-05-05 10:16:22 +0200 | [diff] [blame] | 97 | validate_shader(vulkan_glsl_path, vulkan) |
| 98 | |
| 99 | return (spirv_path, glsl_path, vulkan_glsl_path if vulkan else None) |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 100 | |
| 101 | def md5_for_file(path): |
| 102 | md5 = hashlib.md5() |
| 103 | with open(path, 'rb') as f: |
| 104 | for chunk in iter(lambda: f.read(8192), b''): |
| 105 | md5.update(chunk) |
| 106 | return md5.digest() |
| 107 | |
| 108 | def make_reference_dir(path): |
| 109 | base = os.path.dirname(path) |
| 110 | if not os.path.exists(base): |
| 111 | os.makedirs(base) |
| 112 | |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 113 | def reference_path(directory, relpath): |
| 114 | split_paths = os.path.split(directory) |
| 115 | reference_dir = os.path.join(split_paths[0], 'reference/') |
| 116 | reference_dir = os.path.join(reference_dir, split_paths[1]) |
| 117 | return os.path.join(reference_dir, relpath) |
| 118 | |
Hans-Kristian Arntzen | e50e04c | 2016-03-22 14:56:50 +0100 | [diff] [blame] | 119 | def regression_check(shader, glsl, update, keep): |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 120 | reference = reference_path(shader[0], shader[1]) |
| 121 | joined_path = os.path.join(shader[0], shader[1]) |
| 122 | print('Reference shader path:', reference) |
| 123 | |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 124 | if os.path.exists(reference): |
| 125 | if md5_for_file(glsl) != md5_for_file(reference): |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 126 | if update: |
Hans-Kristian Arntzen | e50e04c | 2016-03-22 14:56:50 +0100 | [diff] [blame] | 127 | print('Generated GLSL has changed for {}!'.format(reference)) |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 128 | # If we expect changes, update the reference file. |
| 129 | if os.path.exists(reference): |
| 130 | os.remove(reference) |
| 131 | make_reference_dir(reference) |
| 132 | shutil.move(glsl, reference) |
| 133 | else: |
Hans-Kristian Arntzen | e50e04c | 2016-03-22 14:56:50 +0100 | [diff] [blame] | 134 | print('Generated GLSL in {} does not match reference {}!'.format(glsl, reference)) |
| 135 | # Otherwise, fail the test. Keep the shader file around so we can inspect. |
| 136 | if not keep: |
| 137 | os.remove(glsl) |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 138 | sys.exit(1) |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 139 | else: |
| 140 | os.remove(glsl) |
| 141 | else: |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 142 | print('Found new shader {}. Placing GLSL in {}'.format(joined_path, reference)) |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 143 | make_reference_dir(reference) |
| 144 | shutil.move(glsl, reference) |
| 145 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 146 | def shader_is_vulkan(shader): |
| 147 | return '.vk.' in shader |
| 148 | |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 149 | def shader_is_spirv(shader): |
| 150 | return '.asm.' in shader |
| 151 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 152 | def test_shader(stats, shader, update, keep): |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 153 | joined_path = os.path.join(shader[0], shader[1]) |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 154 | vulkan = shader_is_vulkan(shader[1]) |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 155 | spirv = shader_is_spirv(shader[1]) |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 156 | |
| 157 | print('Testing shader:', joined_path) |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 158 | spirv, glsl, vulkan_glsl = cross_compile(joined_path, vulkan, spirv) |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 159 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 160 | # Only test GLSL stats if we have a shader following GL semantics. |
Hans-Kristian Arntzen | 45ad58a | 2016-05-10 23:39:41 +0200 | [diff] [blame] | 161 | if stats and (not vulkan) and (not spirv): |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 162 | cross_stats = get_shader_stats(glsl) |
| 163 | |
Hans-Kristian Arntzen | e50e04c | 2016-03-22 14:56:50 +0100 | [diff] [blame] | 164 | regression_check(shader, glsl, update, keep) |
Hans-Kristian Arntzen | dbee4e4 | 2016-05-05 10:16:22 +0200 | [diff] [blame] | 165 | if vulkan_glsl: |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 166 | regression_check((shader[0], shader[1] + '.vk'), vulkan_glsl, update, keep) |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 167 | os.remove(spirv) |
| 168 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 169 | if stats and (not vulkan): |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 170 | pristine_stats = get_shader_stats(joined_path) |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 171 | |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 172 | a = [] |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 173 | a.append(shader[1]) |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 174 | for i in pristine_stats: |
| 175 | a.append(str(i)) |
| 176 | for i in cross_stats: |
| 177 | a.append(str(i)) |
| 178 | print(','.join(a), file = stats) |
| 179 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 180 | def test_shaders_helper(stats, shader_dir, update, malisc, keep): |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 181 | for root, dirs, files in os.walk(os.path.join(shader_dir)): |
| 182 | for i in files: |
| 183 | path = os.path.join(root, i) |
| 184 | relpath = os.path.relpath(path, shader_dir) |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 185 | test_shader(stats, (shader_dir, relpath), update, keep) |
Hans-Kristian Arntzen | 78c9a80 | 2016-05-11 19:39:38 +0200 | [diff] [blame] | 186 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 187 | def test_shaders(shader_dir, update, malisc, keep): |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 188 | if malisc: |
| 189 | with open('stats.csv', 'w') as stats: |
| 190 | print('Shader,OrigRegs,OrigUniRegs,OrigALUShort,OrigLSShort,OrigTEXShort,OrigALULong,OrigLSLong,OrigTEXLong,CrossRegs,CrossUniRegs,CrossALUShort,CrossLSShort,CrossTEXShort,CrossALULong,CrossLSLong,CrossTEXLong', file = stats) |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 191 | test_shaders_helper(stats, shader_dir, update, malisc, keep) |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 192 | else: |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 193 | test_shaders_helper(None, shader_dir, update, malisc, keep) |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 194 | |
| 195 | def main(): |
| 196 | parser = argparse.ArgumentParser(description = 'Script for regression testing.') |
| 197 | parser.add_argument('folder', |
| 198 | help = 'Folder containing shader files to test.') |
| 199 | parser.add_argument('--update', |
| 200 | action = 'store_true', |
| 201 | help = 'Updates reference files if there is a mismatch. Use when legitimate changes in output is found.') |
Hans-Kristian Arntzen | e50e04c | 2016-03-22 14:56:50 +0100 | [diff] [blame] | 202 | parser.add_argument('--keep', |
| 203 | action = 'store_true', |
| 204 | help = 'Leave failed GLSL shaders on disk if they fail regression. Useful for debugging.') |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 205 | parser.add_argument('--malisc', |
| 206 | action = 'store_true', |
Hans-Kristian Arntzen | 147e53a | 2016-04-04 09:36:04 +0200 | [diff] [blame] | 207 | help = 'Use malisc offline compiler to determine static cycle counts before and after spirv-cross.') |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 208 | args = parser.parse_args() |
| 209 | |
| 210 | if not args.folder: |
| 211 | sys.stderr.write('Need shader folder.\n') |
| 212 | sys.exit(1) |
| 213 | |
Hans-Kristian Arntzen | 8869a16 | 2016-05-11 19:55:57 +0200 | [diff] [blame] | 214 | test_shaders(args.folder, args.update, args.malisc, args.keep) |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 215 | if args.malisc: |
| 216 | print('Stats in stats.csv!') |
Hans-Kristian Arntzen | e50e04c | 2016-03-22 14:56:50 +0100 | [diff] [blame] | 217 | print('Tests completed!') |
Hans-Kristian Arntzen | 75471fb | 2016-03-02 18:09:16 +0100 | [diff] [blame] | 218 | |
| 219 | if __name__ == '__main__': |
Hans-Kristian Arntzen | 0a5b3a6 | 2016-03-22 14:44:12 +0100 | [diff] [blame] | 220 | main() |