Skip to content

Commit

Permalink
src: disable concurrent sparkplug compilation
Browse files Browse the repository at this point in the history
It introduces process hangs on some platforms because Node.js doesn't
tear down V8 correctly.
Disable it while we work on a solution.

Refs: nodejs#47297
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=13902
  • Loading branch information
targos committed Apr 6, 2023
1 parent 0e79635 commit 88c91bf
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/node.cc
Original file line number Diff line number Diff line change
Expand Up @@ -790,6 +790,10 @@ static ExitCode InitializeNodeWithArgsInternal(
// is security relevant, for Node it's less important.
V8::SetFlagsFromString("--no-freeze-flags-after-init");

// TODO(targos): Remove when /~https://github.com/nodejs/node/issues/47297 is
// fixed.
V8::SetFlagsFromString("--no-concurrent-sparkplug");

#if defined(NODE_V8_OPTIONS)
// Should come before the call to V8::SetFlagsFromCommandLine()
// so the user can disable a flag --foo at run-time by passing
Expand Down

0 comments on commit 88c91bf

Please sign in to comment.