Closed as not planned
Description
Continuing from nodejs/help#1031: src/node_counters.cc
and its ilk need to have their initialization split out into a per-isolate and per-context step for embedding to work with them. Some thoughts:
-
node.js won't know if initialization already happened if the isolate is created by the embedder, unless extra bookkeeping is added. Could hang off
MultiIsolatePlatform::RegisterIsolate()
- or could it? -
the tracing code itself isn't multi-isolate ready in the slightest, it's all globals that need to move to
IsolateData
andEnvironment
-
some tracing flavors have (possibly inevitable) process-global state that should be protected by a
uv_once_t
ornode::Mutex
.