diff options
author | Prathamesh Kulkarni <prathamesh.kulkarni@linaro.org> | 2023-06-26 17:24:54 +0530 |
---|---|---|
committer | Prathamesh Kulkarni <prathamesh.kulkarni@linaro.org> | 2023-06-26 17:27:22 +0530 |
commit | 22ee3e59cad6c33708107544edb34ba44a92efc1 (patch) | |
tree | 1415e13a98e714da9d225611c198125292e9d13e /gather-metrics.sh | |
parent | 9e1f6e23901bfb75f5d43e75098053413b4848e7 (diff) |
gather-metrics.sh: Fix up formatting for perf.csv.
The patch fixes the following issues with perf.csv formatting:
(a) For bare metal benchmarks, add a dummy dso column with "na" values.
(b) -F dso,symbol doesn't output dso entry for binaries. Add dummy "na"
values for the same
(c) Strip whitespace from dso names.
Change-Id: I5635ae035abe3ee5067ca30540aceb5739dc2e34
Diffstat (limited to 'gather-metrics.sh')
-rwxr-xr-x | gather-metrics.sh | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/gather-metrics.sh b/gather-metrics.sh index 7b0d2e7..227e99d 100755 --- a/gather-metrics.sh +++ b/gather-metrics.sh @@ -16,7 +16,7 @@ function gather_perf_data () --buildid-dir local --format sample,dso --sort-field sample \ --perf-bin /usr/lib/linux-tools/$hw_tag/perf \ $verbose_opt $num_entries_opt \ - --results-dir "$results_dir" > "$csv_results_dir/perf.csv" + --results-dir "$results_dir" > "$csv_results_dir/perf-tmp.csv" else # No perf logs to parse, just copy the plain results.csv. # Use 'find' because results.csv is located under @@ -27,12 +27,15 @@ function gather_perf_data () if [ "${#this_csv[@]}" -eq 1 ]; then # Remove "size" column from old results.csv cat "${this_csv[@]}" | cut -d"," -f 1-3 \ - > "$csv_results_dir/perf.csv" + > "$csv_results_dir/perf-tmp.csv" else echo "ERROR: Found ${#this_csv[@]} CSV results files in results-perf, expecting a single one." exit 1 fi fi + + $bmk_scripts/fixup-perf-csv.py "$csv_results_dir/perf-tmp.csv" > "$csv_results_dir/perf.csv" + rm "$csv_results_dir/perf-tmp.csv" } function check_option() |