From 11839c852e4916bc04c4cfd7cc1a81ebd3fe85ae Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jesper=20=C3=96qvist?= Date: Tue, 10 May 2016 14:47:05 -0700 Subject: [PATCH] Fix error in perf command line construction Split perf command into command and arguments so that subprocess.call() works as it should. fixes #6 --- utility/rbench.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/utility/rbench.py b/utility/rbench.py index 68a0a14..0c14e3a 100755 --- a/utility/rbench.py +++ b/utility/rbench.py @@ -107,7 +107,7 @@ def parse_args(rvms, warmup_rep, bench_rep): '''Return a dictionary''' def run_bench(config, rvm, meter, warmup_rep, bench_rep, source, rargs, bench_log): - perf_cmd = config.get('GENERAL', 'PERF_CMD') + perf_cmd = config.get('GENERAL', 'PERF_CMD').split() perf_tmp = config.get('GENERAL', 'PERF_TMP') rcmd = config.get(rvm, 'CMD') rcmd_args = config.get(rvm, 'ARGS') @@ -121,9 +121,9 @@ def run_bench(config, rvm, meter, warmup_rep, bench_rep, source, rargs, bench_lo use_system_time = 'FALSE' if meter == 'perf': - warmup_cmd = [perf_cmd, rcmd, rcmd_args, harness, harness_args, + warmup_cmd = perf_cmd + [rcmd, rcmd_args, harness, harness_args, use_system_time, str(warmup_rep), source, rargs] - bench_cmd = [perf_cmd, rcmd, rcmd_args, harness, harness_args, + bench_cmd = perf_cmd + [rcmd, rcmd_args, harness, harness_args, use_system_time, str(warmup_rep+bench_rep), source, rargs] else: #default python warmup_cmd = [rcmd, rcmd_args, harness, harness_args,