summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Todd Kjos <tkjos@google.com> 2017-10-03 13:56:33 -0700
committer Todd Kjos <tkjos@google.com> 2017-10-03 21:01:42 +0000
commit3b17a3128f1bc044eadee17b546b29394ef791e3 (patch)
tree4f824a0a4fa509101b8581f070ec8fd5e62879b3
parentbe1a67cacfda4d9f4311d4257a57b023780f10f8 (diff)
Fix binderThroughputTest -m option
The -m option allows the user to specify the estimated max latency to calibrate buckets used for percentile calculations. The option didn't work because the time_per_bucket was not being set so the default value was used. Test: verified correct -m operation Bug: 65052581 Change-Id: I96d6ba0271edaffda0bd9cccfef5d559d3bf111c Signed-off-by: Todd Kjos <tkjos@google.com>
-rw-r--r--libs/binder/tests/binderThroughputTest.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/libs/binder/tests/binderThroughputTest.cpp b/libs/binder/tests/binderThroughputTest.cpp
index 455f2c418d..e35f491984 100644
--- a/libs/binder/tests/binderThroughputTest.cpp
+++ b/libs/binder/tests/binderThroughputTest.cpp
@@ -381,6 +381,7 @@ int main(int argc, char *argv[])
// No need to run training round in this case.
if (atoi(argv[i+1]) > 0) {
max_time_bucket = strtoull(argv[i+1], (char **)NULL, 10) * 1000;
+ time_per_bucket = max_time_bucket / num_buckets;
i++;
} else {
cout << "Max latency -m must be positive." << endl;