wiptrott-temp-wip-14-Apr-2019

2 files changed, 15 insertions, 1 deletions

diff --git a/benchmark/common.js b/benchmark/common.js
index b4778d7193..71361bc8b6 100644
--- a/benchmark/common.js
+++ b/benchmark/common.js
@@ -103,6 +103,13 @@ Benchmark.prototype._queue = function(options) {
     queue.push({});
   }
 
+  // Shuffle the queue. This mitigates the possibility of artifacts resulting from
+  // operating system throttling or caching.
+  // Ref: https://github.com/nodejs/node/pull/27081#issuecomment-479981874
+  for (let i = queue.length - 1; i > 0; i--) {
+    const j = Math.floor(Math.random() * (i + 1));
+    [queue[i], queue[j]] = [queue[j], queue[i]];
+  }
   return queue;
 };
 
diff --git a/benchmark/compare.js b/benchmark/compare.js
index bd7c4a95cb..0cc655cd93 100644
--- a/benchmark/compare.js
+++ b/benchmark/compare.js
@@ -50,7 +50,14 @@ for (const filename of benchmarks) {
     }
   }
 }
-// queue.length = binary.length * runs * benchmarks.length
+
+// Shuffle the queue. This mitigates the possibility of artifacts resulting from
+// operating system throttling or caching.
+// Ref: https://github.com/nodejs/node/pull/27081#issuecomment-479981874
+for (let i = queue.length - 1; i > 0; i--) {
+  const j = Math.floor(Math.random() * (i + 1));
+  [queue[i], queue[j]] = [queue[j], queue[i]];
+}
 
 // Print csv header
 console.log('"binary", "filename", "configuration", "rate", "time"');