lib: remove unnecessary refreshHrtimeBuffer()

We now serialize/deseialize the hrtime buffers properly instead of
throwing them away at serialization and creating new ones at
pre-execution, so there is no need to reset the variables to
the binding property at pre-execution time.

PR-URL: nodejs#51446
Refs: nodejs#48655
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>

Author: joyeecheung

lib/internal/process/per_thread.js

@@ -57,21 +57,13 @@ const { exitCodes: { kNoFailure } } = internalBinding('errors');
 
 const binding = internalBinding('process_methods');
 
-let hrValues;
-let hrBigintValues;
-
-function refreshHrtimeBuffer() {
-  // The 3 entries filled in by the original process.hrtime contains
-  // the upper/lower 32 bits of the second part of the value,
-  // and the remaining nanoseconds of the value.
-  hrValues = binding.hrtimeBuffer;
-  // Use a BigUint64Array in the closure because this is actually a bit
-  // faster than simply returning a BigInt from C++ in V8 7.1.
-  hrBigintValues = new BigUint64Array(binding.hrtimeBuffer.buffer, 0, 1);
-}
-
-// Create the buffers.
-refreshHrtimeBuffer();
+// The 3 entries filled in by the original process.hrtime contains
+// the upper/lower 32 bits of the second part of the value,
+// and the remaining nanoseconds of the value.
+const hrValues = binding.hrtimeBuffer;
+// Use a BigUint64Array because this is actually a bit
+// faster than simply returning a BigInt from C++ in V8 7.1.
+const hrBigintValues = new BigUint64Array(binding.hrtimeBuffer.buffer, 0, 1);
 
 function hrtime(time) {
   binding.hrtime();
@@ -425,5 +417,4 @@ module.exports = {
   wrapProcessMethods,
   hrtime,
   hrtimeBigInt,
-  refreshHrtimeBuffer,
 };

lib/internal/process/pre_execution.js

@@ -211,8 +211,6 @@ function patchProcessObject(expandArgv1) {
   const binding = internalBinding('process_methods');
   binding.patchProcessObject(process);
 
-  require('internal/process/per_thread').refreshHrtimeBuffer();
-
   // Since we replace process.argv[0] below, preserve the original value in case the user needs it.
   ObjectDefineProperty(process, 'argv0', {
     __proto__: null,