v8: export more fields in getHeapStatistics

export total_global_handles_size, used_global_handles_size, external_memory in getHeapStatistics
nodejs · Apr 19, 2022 · 67dfc51 · 67dfc51
1 parent 77373aa
commit 67dfc51
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 3 deletions.
diff --git a/doc/api/v8.md b/doc/api/v8.md
@@ -180,6 +180,9 @@ Returns an object with the following properties:
 * `does_zap_garbage` {number}
 * `number_of_native_contexts` {number}
 * `number_of_detached_contexts` {number}
+* `total_global_handles_size` {number}
+* `used_global_handles_size` {number}
+* `external_memory` {number}
 
 `does_zap_garbage` is a 0/1 boolean, which signifies whether the
 `--zap_code_space` option is enabled or not. This makes V8 overwrite heap
@@ -209,7 +212,10 @@ being non-zero indicates a potential memory leak.
   peak_malloced_memory: 1127496,
   does_zap_garbage: 0,
   number_of_native_contexts: 1,
-  number_of_detached_contexts: 0
+  number_of_detached_contexts: 0,
+  total_global_handles_size: 8192,
+  used_global_handles_size: 3296,
+  external_memory: 318824
 }
 ```
 

diff --git a/lib/v8.js b/lib/v8.js
@@ -107,6 +107,9 @@ const {
   kPeakMallocedMemoryIndex,
   kNumberOfNativeContextsIndex,
   kNumberOfDetachedContextsIndex,
+  kTotalGlobalHandlesSizeIndex,
+  kUsedGlobalHandlesSizeIndex,
+  kExternalMemoryIndex,
 
   // Properties for heap spaces statistics buffer extraction.
   kHeapSpaces,
@@ -165,7 +168,10 @@ function getHeapStatistics() {
     peak_malloced_memory: buffer[kPeakMallocedMemoryIndex],
     does_zap_garbage: buffer[kDoesZapGarbageIndex],
     number_of_native_contexts: buffer[kNumberOfNativeContextsIndex],
-    number_of_detached_contexts: buffer[kNumberOfDetachedContextsIndex]
+    number_of_detached_contexts: buffer[kNumberOfDetachedContextsIndex],
+    total_global_handles_size: buffer[kTotalGlobalHandlesSizeIndex],
+    used_global_handles_size: buffer[kUsedGlobalHandlesSizeIndex],
+    external_memory: buffer[kExternalMemoryIndex]
   };
 }
 

diff --git a/src/node_v8.cc b/src/node_v8.cc
@@ -59,7 +59,10 @@ using v8::Value;
   V(7, peak_malloced_memory, kPeakMallocedMemoryIndex)                        \
   V(8, does_zap_garbage, kDoesZapGarbageIndex)                                \
   V(9, number_of_native_contexts, kNumberOfNativeContextsIndex)               \
-  V(10, number_of_detached_contexts, kNumberOfDetachedContextsIndex)
+  V(10, number_of_detached_contexts, kNumberOfDetachedContextsIndex)          \
+  V(11, total_global_handles_size, kTotalGlobalHandlesSizeIndex)              \
+  V(12, used_global_handles_size, kUsedGlobalHandlesSizeIndex)                \
+  V(13, external_memory, kExternalMemoryIndex)
 
 #define V(a, b, c) +1
 static constexpr size_t kHeapStatisticsPropertiesCount =

diff --git a/test/parallel/test-v8-stats.js b/test/parallel/test-v8-stats.js
@@ -6,15 +6,18 @@ const v8 = require('v8');
 const s = v8.getHeapStatistics();
 const keys = [
   'does_zap_garbage',
+  'external_memory',
   'heap_size_limit',
   'malloced_memory',
   'number_of_detached_contexts',
   'number_of_native_contexts',
   'peak_malloced_memory',
   'total_available_size',
+  'total_global_handles_size',
   'total_heap_size',
   'total_heap_size_executable',
   'total_physical_size',
+  'used_global_handles_size',
   'used_heap_size'];
 assert.deepStrictEqual(Object.keys(s).sort(), keys);
 keys.forEach(function(key) {