diff options
| author |  Brian Salomon <bsalomon@google.com> | 2017-03-17 14:32:05 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-03-17 19:09:37 +0000 |
| commit | 50f66d805bcd424e34e1191a24a66b09e5611fe2 (patch) | |
| tree | 9839294fa8219168ed1c50794325d54c8d83d28d /tools/skpbench | |
| parent | f6ce600bc576f9951648bdda6e77af5bad4e9562 (diff) | |
Be explicit on test and perf bots about GL vs GLES
This changes the names of several shorthand names for gpu configs to be prefixed either with "gl" or "gles" and makes the bots only use such configs. It adds some missing named configs.
Change-Id: Iea4e0e2ddafe0ac08e623111a15be7335156957c
Reviewed-on: https://skia-review.googlesource.com/9833
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
Diffstat (limited to 'tools/skpbench')
| -rwxr-xr-x | tools/skpbench/skpbench.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tools/skpbench/skpbench.py b/tools/skpbench/skpbench.py index 7cc0a231e9..ef44577c53 100755 --- a/tools/skpbench/skpbench.py +++ b/tools/skpbench/skpbench.py @@ -55,7 +55,7 @@ __argparse.add_argument('--gpu', __argparse.add_argument('--fps', action='store_true', help="use fps instead of ms") __argparse.add_argument('-c', '--config', - default='gpu', help="comma- or space-separated list of GPU configs") + default='gl', help="comma- or space-separated list of GPU configs") __argparse.add_argument('-a', '--resultsfile', help="optional file to append results into") __argparse.add_argument('skps', @@ -122,12 +122,12 @@ class SKPBench: return out.rstrip() @classmethod - def run_warmup(cls, warmup_time): + def run_warmup(cls, warmup_time, config): if not warmup_time: return print('running %i second warmup...' % warmup_time, file=sys.stderr) commandline = cls.ARGV + ['--duration', str(warmup_time * 1000), - '--config', 'gpu', + '--config', config, '--skp', 'warmup'] dump_commandline_if_verbose(commandline) output = subprocess.check_output(commandline, stderr=subprocess.STDOUT) @@ -269,7 +269,7 @@ def run_benchmarks(configs, skps, hardware, resultsfile=None): hardware.sleep(exception.sleeptime) if FLAGS.verbosity >= 4: hardware.print_debug_diagnostics() - SKPBench.run_warmup(hardware.warmup_time) + SKPBench.run_warmup(hardware.warmup_time, configs[0]) def main(): # Delimiter is ',' or ' ', skip if nested inside parens (e.g. gpu(a=b,c=d)). @@ -295,7 +295,7 @@ def main(): hardware = Hardware() with hardware: - SKPBench.run_warmup(hardware.warmup_time) + SKPBench.run_warmup(hardware.warmup_time, configs[0]) if FLAGS.resultsfile: with open(FLAGS.resultsfile, mode='a+') as resultsfile: run_benchmarks(configs, skps, hardware, resultsfile=resultsfile) |