Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Side by Side Diff: tools/bench_record.cpp

Issue 346753003: Revert of Move BenchTimer to tools as Timer (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tools/bench_playback.cpp ('k') | tools/skpdiff/skpdiff_util.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2014 Google Inc. 2 * Copyright 2014 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkCommandLineFlags.h" 8 #include "SkCommandLineFlags.h"
9 #include "SkForceLinking.h" 9 #include "SkForceLinking.h"
10 #include "SkGraphics.h" 10 #include "SkGraphics.h"
11 #include "SkOSFile.h" 11 #include "SkOSFile.h"
12 #include "SkPicture.h" 12 #include "SkPicture.h"
13 #include "SkPictureRecorder.h" 13 #include "SkPictureRecorder.h"
14 #include "SkStream.h" 14 #include "SkStream.h"
15 #include "SkString.h" 15 #include "SkString.h"
16 16
17 #include "BenchTimer.h"
17 #include "LazyDecodeBitmap.h" 18 #include "LazyDecodeBitmap.h"
18 #include "Stats.h" 19 #include "Stats.h"
19 #include "Timer.h" 20
21 typedef WallTimer Timer;
20 22
21 __SK_FORCE_IMAGE_DECODER_LINKING; 23 __SK_FORCE_IMAGE_DECODER_LINKING;
22 24
23 DEFINE_string2(skps, r, "skps", "Directory containing SKPs to read and re-record ."); 25 DEFINE_string2(skps, r, "skps", "Directory containing SKPs to read and re-record .");
24 DEFINE_int32(samples, 10, "Number of times to re-record each SKP."); 26 DEFINE_int32(samples, 10, "Number of times to re-record each SKP.");
25 DEFINE_int32(tileGridSize, 512, "Set the tile grid size. Has no effect if bbh is not set to tilegrid."); 27 DEFINE_int32(tileGridSize, 512, "Set the tile grid size. Has no effect if bbh is not set to tilegrid.");
26 DEFINE_string(bbh, "", "Turn on the bbh and select the type, one of rtree, tileg rid, quadtree"); 28 DEFINE_string(bbh, "", "Turn on the bbh and select the type, one of rtree, tileg rid, quadtree");
27 DEFINE_bool(skr, false, "Record SKR instead of SKP."); 29 DEFINE_bool(skr, false, "Record SKR instead of SKP.");
28 DEFINE_string(match, "", "The usual filters on file names of SKPs to bench."); 30 DEFINE_string(match, "", "The usual filters on file names of SKPs to bench.");
29 DEFINE_string(timescale, "us", "Print times in ms, us, or ns"); 31 DEFINE_string(timescale, "us", "Print times in ms, us, or ns");
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 } 74 }
73 75
74 static void bench_record(const SkPicture& src, 76 static void bench_record(const SkPicture& src,
75 const double timerOverhead, 77 const double timerOverhead,
76 const char* name, 78 const char* name,
77 SkBBHFactory* bbhFactory) { 79 SkBBHFactory* bbhFactory) {
78 // Rerecord once to warm up any caches. Otherwise the first sample can be v ery noisy. 80 // Rerecord once to warm up any caches. Otherwise the first sample can be v ery noisy.
79 rerecord(src, bbhFactory); 81 rerecord(src, bbhFactory);
80 82
81 // Rerecord once to see how many times we should loop to make timer overhead insignificant. 83 // Rerecord once to see how many times we should loop to make timer overhead insignificant.
82 WallTimer timer; 84 Timer timer;
83 const double scale = timescale();
84 do { 85 do {
85 timer.start(); 86 timer.start(timescale());
86 rerecord(src, bbhFactory); 87 rerecord(src, bbhFactory);
87 timer.end(); 88 timer.end();
88 } while (timer.fWall * scale < timerOverhead); // Loop just in case somethi ng bizarre happens. 89 } while (timer.fWall < timerOverhead); // Loop just in case something biza rre happens.
89 90
90 // We want (timer overhead / measurement) to be less than FLAGS_overheadGoal . 91 // We want (timer overhead / measurement) to be less than FLAGS_overheadGoal .
91 // So in each sample, we'll loop enough times to have made that true for our first measurement. 92 // So in each sample, we'll loop enough times to have made that true for our first measurement.
92 const int loops = (int)ceil(timerOverhead / timer.fWall / FLAGS_overheadGoal ); 93 const int loops = (int)ceil(timerOverhead / timer.fWall / FLAGS_overheadGoal );
93 94
94 SkAutoTMalloc<double> samples(FLAGS_samples); 95 SkAutoTMalloc<double> samples(FLAGS_samples);
95 for (int i = 0; i < FLAGS_samples; i++) { 96 for (int i = 0; i < FLAGS_samples; i++) {
96 timer.start(); 97 timer.start(timescale());
97 for (int j = 0; j < loops; j++) { 98 for (int j = 0; j < loops; j++) {
98 rerecord(src, bbhFactory); 99 rerecord(src, bbhFactory);
99 } 100 }
100 timer.end(); 101 timer.end();
101 samples[i] = timer.fWall * scale / loops; 102 samples[i] = timer.fWall / loops;
102 } 103 }
103 104
104 Stats stats(samples.get(), FLAGS_samples); 105 Stats stats(samples.get(), FLAGS_samples);
105 if (FLAGS_verbose == 0) { 106 if (FLAGS_verbose == 0) {
106 printf("%g\t%s\n", stats.min, name); 107 printf("%g\t%s\n", stats.min, name);
107 } else if (FLAGS_verbose == 1) { 108 } else if (FLAGS_verbose == 1) {
108 // Get a rough idea of how noisy the measurements were. 109 // Get a rough idea of how noisy the measurements were.
109 const double noisePercent = 100 * sqrt(stats.var) / stats.mean; 110 const double noisePercent = 100 * sqrt(stats.var) / stats.mean;
110 printf("%g\t%g\t%g\t±%.0f%%\t%s\n", stats.min, stats.mean, stats.max, no isePercent, name); 111 printf("%g\t%g\t%g\t±%.0f%%\t%s\n", stats.min, stats.mean, stats.max, no isePercent, name);
111 } else if (FLAGS_verbose == 2) { 112 } else if (FLAGS_verbose == 2) {
(...skipping 12 matching lines...) Expand all
124 125
125 if (FLAGS_bbh.count() > 1) { 126 if (FLAGS_bbh.count() > 1) {
126 SkDebugf("Multiple bbh arguments supplied.\n"); 127 SkDebugf("Multiple bbh arguments supplied.\n");
127 return 1; 128 return 1;
128 } 129 }
129 130
130 SkAutoTDelete<SkBBHFactory> bbhFactory(parse_FLAGS_bbh()); 131 SkAutoTDelete<SkBBHFactory> bbhFactory(parse_FLAGS_bbh());
131 132
132 // Each run will use this timer overhead estimate to guess how many times it should run. 133 // Each run will use this timer overhead estimate to guess how many times it should run.
133 static const int kOverheadLoops = 10000000; 134 static const int kOverheadLoops = 10000000;
134 WallTimer timer; 135 Timer timer;
135 double overheadEstimate = 0.0; 136 double overheadEstimate = 0.0;
136 const double scale = timescale();
137 for (int i = 0; i < kOverheadLoops; i++) { 137 for (int i = 0; i < kOverheadLoops; i++) {
138 timer.start(); 138 timer.start(timescale());
139 timer.end(); 139 timer.end();
140 overheadEstimate += timer.fWall * scale; 140 overheadEstimate += timer.fWall;
141 } 141 }
142 overheadEstimate /= kOverheadLoops; 142 overheadEstimate /= kOverheadLoops;
143 143
144 SkOSFile::Iter it(FLAGS_skps[0], ".skp"); 144 SkOSFile::Iter it(FLAGS_skps[0], ".skp");
145 SkString filename; 145 SkString filename;
146 bool failed = false; 146 bool failed = false;
147 while (it.next(&filename)) { 147 while (it.next(&filename)) {
148 if (SkCommandLineFlags::ShouldSkip(FLAGS_match, filename.c_str())) { 148 if (SkCommandLineFlags::ShouldSkip(FLAGS_match, filename.c_str())) {
149 continue; 149 continue;
150 } 150 }
(...skipping 16 matching lines...) Expand all
167 bench_record(*src, overheadEstimate, filename.c_str(), bbhFactory.get()) ; 167 bench_record(*src, overheadEstimate, filename.c_str(), bbhFactory.get()) ;
168 } 168 }
169 return failed ? 1 : 0; 169 return failed ? 1 : 0;
170 } 170 }
171 171
172 #if !defined SK_BUILD_FOR_IOS 172 #if !defined SK_BUILD_FOR_IOS
173 int main(int argc, char * const argv[]) { 173 int main(int argc, char * const argv[]) {
174 return tool_main(argc, (char**) argv); 174 return tool_main(argc, (char**) argv);
175 } 175 }
176 #endif 176 #endif
OLDNEW
« no previous file with comments | « tools/bench_playback.cpp ('k') | tools/skpdiff/skpdiff_util.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698