diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2009-07-31 10:50:13 +0100 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2009-07-31 10:51:30 +0100 |
commit | e283e53563915daa11566ba006e1bf1e12b0d2fc (patch) | |
tree | ef917e83f905d75091099eaf9e03171a2f37bf7d /perf/cairo-perf-diff-files.c | |
parent | ce8a8424aa4331f119b115123faf7b773b5f25d0 (diff) | |
download | cairo-e283e53563915daa11566ba006e1bf1e12b0d2fc.tar.gz |
[perf] Print min_ticks in report
As the change and ranking is based on the min_ticks, and as this can
sometimes deviate wildly from median_ticks, include min_ticks in the
output.
In particular it helps to explain cases like:
xlib-rgba rectangles_similar-rgba-mag_source-512 10.13 88.41% -> 5.77 0.19%: 1.50x slowdown
which becomes
xlib-rgba rectangles_similar-rgba-mag_source-512 3.83 (10.13 88.41%) -> 5.75 (5.77 0.19%): 1.50x slowdown
(Considering the poor standard deviation on the initial measurement, this
is more likely a sampling error than a true regression.)
Diffstat (limited to 'perf/cairo-perf-diff-files.c')
-rw-r--r-- | perf/cairo-perf-diff-files.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/perf/cairo-perf-diff-files.c b/perf/cairo-perf-diff-files.c index 6870b80b9..dddf684d3 100644 --- a/perf/cairo-perf-diff-files.c +++ b/perf/cairo-perf-diff-files.c @@ -142,11 +142,13 @@ test_diff_print_binary (test_diff_t *diff, double max_change, cairo_perf_report_options_t *options) { - printf ("%5s-%-4s %26s-%-3d %6.2f %4.2f%% -> %6.2f %4.2f%%: %5.2fx ", + printf ("%5s-%-4s %26s-%-3d %6.2f (%.2f %4.2f%%) -> %6.2f (%.2f %4.2f%%): %5.2fx ", diff->tests[0]->backend, diff->tests[0]->content, diff->tests[0]->name, diff->tests[0]->size, + diff->tests[0]->stats.min_ticks / diff->tests[0]->stats.ticks_per_ms, diff->tests[0]->stats.median_ticks / diff->tests[0]->stats.ticks_per_ms, diff->tests[0]->stats.std_dev * 100, + diff->tests[1]->stats.min_ticks / diff->tests[1]->stats.ticks_per_ms, diff->tests[1]->stats.median_ticks / diff->tests[1]->stats.ticks_per_ms, diff->tests[1]->stats.std_dev * 100, fabs (diff->change)); |