diff options
author | Christopher Ferris <cferris@google.com> | 2013-05-03 20:34:46 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2013-05-03 20:34:46 +0000 |
commit | 10057e4dbeae5bc7ed495dc9f2342a6596c82c79 (patch) | |
tree | 33bac57ca37963fa0044a1a2379c1e8373452858 | |
parent | ed6a3afac9f3d34fc2ce2f23baec05f7a43e58cf (diff) | |
parent | bdfb235a97f69d82538449a033e3e5f1637365d0 (diff) | |
download | extras-tools_r22.tar.gz |
Merge "Fix a few incorrect return codes."tools_r22
-rw-r--r-- | tests/memtest/bandwidth.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tests/memtest/bandwidth.cpp b/tests/memtest/bandwidth.cpp index 0e0ec2e0..523cabe3 100644 --- a/tests/memtest/bandwidth.cpp +++ b/tests/memtest/bandwidth.cpp @@ -306,7 +306,7 @@ int per_core_bandwidth(int argc, char** argv) { args[i].core = *it; args[i].bench = createBandwidthBenchmarkObject(values); if (!args[i].bench) { - return 0; + return -1; } } @@ -342,7 +342,7 @@ int multithread_bandwidth(int argc, char** argv) { args[i].core = -1; args[i].bench = createBandwidthBenchmarkObject(values); if (!args[i].bench) { - return 0; + return -1; } } @@ -367,18 +367,18 @@ bool run_bandwidth_benchmark(int argc, char** argv, const char *name, values["num_warm_loops"].int_value = 0; values["num_loops"].int_value = 0; if (!processBandwidthOptions(argc, argv, bandwidth_opts, &values)) { - return -1; + return false; } size_t size = values["size"].int_value; if ((size % 64) != 0) { printf("The size value must be a multiple of 64.\n"); - return 1; + return false; } if (setpriority(PRIO_PROCESS, 0, -20)) { perror("Unable to raise priority of process."); - return -1; + return false; } bool preamble_printed = false; |