-
-
Notifications
You must be signed in to change notification settings - Fork 35.1k
Open
Labels
perf_hooksIssues and PRs related to the implementation of the Performance Timing API.Issues and PRs related to the implementation of the Performance Timing API.
Description
Version
18
Platform
No response
Subsystem
pref_hooks
What steps will reproduce the bug?
Looks like GC operation with type "kGCTypeMinorMarkCompact" will never be observed
according to the v8 docs
https://v8docs.nodesource.com/node-18.2/d2/dc3/namespacev8.html#a83be2143eef5fa2b17127fd69686496b
there are 5 GCTypes
kGCTypeScavenge |
kGCTypeMinorMarkCompact |
kGCTypeMarkSweepCompact |
kGCTypeIncrementalMarking |
kGCTypeProcessWeakCallbacks |
~~kGCTypeAll~~
But pref hooks is mapping only 4 of them
Lines 63 to 68 in 191dce8
| enum PerformanceGCKind { | |
| NODE_PERFORMANCE_GC_MAJOR = v8::GCType::kGCTypeMarkSweepCompact, | |
| NODE_PERFORMANCE_GC_MINOR = v8::GCType::kGCTypeScavenge, | |
| NODE_PERFORMANCE_GC_INCREMENTAL = v8::GCType::kGCTypeIncrementalMarking, | |
| NODE_PERFORMANCE_GC_WEAKCB = v8::GCType::kGCTypeProcessWeakCallbacks | |
| }; |
same here
Lines 241 to 252 in 191dce8
| static const char* GetGCTypeName(v8::GCType gc_type) { | |
| switch (gc_type) { | |
| case v8::GCType::kGCTypeScavenge: | |
| return "Scavenge"; | |
| case v8::GCType::kGCTypeMarkSweepCompact: | |
| return "MarkSweepCompact"; | |
| case v8::GCType::kGCTypeIncrementalMarking: | |
| return "IncrementalMarking"; | |
| case v8::GCType::kGCTypeProcessWeakCallbacks: | |
| return "ProcessWeakCallbacks"; | |
| default: | |
| return "Unknown"; |
How often does it reproduce? Is there a required condition?
No response
What is the expected behavior? Why is that the expected behavior?
No response
What do you see instead?
I would expect this type to be reported also
same as other third-party monitoring tools have
https://github.com/DataDog/dd-native-metrics-js/blob/main/src/metrics/GarbageCollection.hpp#L37
Additional information
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
perf_hooksIssues and PRs related to the implementation of the Performance Timing API.Issues and PRs related to the implementation of the Performance Timing API.