Browse Source

a heuristic parameter adjustment for BenchMarkTest.run: the previous margin
(20%) didn't always work for a (seemingly) slow machine, so raise it to 25%.

should be quite trivial, and skipping formal review. discussed on jabber.


git-svn-id: svn://bind10.isc.org/svn/bind10/trunk@2691 e5f2f494-b856-4b98-b285-d166d9295462

JINMEI Tatuya 14 years ago
parent
commit
03e6e0a37f
1 changed files with 2 additions and 2 deletions
  1. 2 2
      src/lib/bench/tests/benchmark_unittest.cc

+ 2 - 2
src/lib/bench/tests/benchmark_unittest.cc

@@ -70,9 +70,9 @@ TEST(BenchMarkTest, run) {
     const int sub_iterations = 23;
     const int sleep_time = 50000; // will sleep for 50ms
     // we cannot expect particular accuracy on the measured duration, so
-    // we'll include some conservative margin (20%) and perform range
+    // we'll include some conservative margin (25%) and perform range
     // comparison below.
-    const int duration_margin = 10000; // 10ms
+    const int duration_margin = 12500; // 12.5ms
     const int ONE_MILLION = 1000000;
 
     // Prerequisite check: since the tests in this case may depend on subtle