diff options
| author | Matt Loring <mattloring@google.com> | 2017-03-14 01:17:57 +0300 |
|---|---|---|
| committer | Myles Borins <mylesborins@google.com> | 2017-09-12 04:16:59 +0300 |
| commit | fadcbab61794d8c9266283b6400b2b1e6ab5c14a (patch) | |
| tree | 3bb936b4472f7300b9c64e95cca54fd11dc5d3da /node.gyp | |
| parent | bd40cc6ef82e1b81e37f406fc813906fe40d29f2 (diff) | |
src: Node implementation of v8::Platform
Node.js currently uses the V8 implementation of the DefaultPlatform
which schedules VM tasks on a V8 managed thread pool. Since the Node.js
event loop is not aware of these tasks, the Node.js process may exit
while there are outstanding VM tasks. This will become problematic once
asynchronous wasm compilation lands in V8.
This PR introduces a Node.js specific implementation of the v8::Platform
on top of libuv so that the event loop is aware of outstanding VM tasks.
PR-URL: https://github.com/nodejs/node/pull/14001
Fixes: https://github.com/nodejs/node/issues/3665
Fixes: https://github.com/nodejs/node/issues/8496
Fixes: https://github.com/nodejs/node/issues/12980
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Diffstat (limited to 'node.gyp')
| -rw-r--r-- | node.gyp | 4 |
1 files changed, 4 insertions, 0 deletions
@@ -189,6 +189,7 @@ 'src/node_http_parser.cc', 'src/node_main.cc', 'src/node_os.cc', + 'src/node_platform.cc', 'src/node_revert.cc', 'src/node_serdes.cc', 'src/node_url.cc', @@ -237,6 +238,7 @@ 'src/node_internals.h', 'src/node_javascript.h', 'src/node_mutex.h', + 'src/node_platform.h', 'src/node_root_certs.h', 'src/node_version.h', 'src/node_watchdog.h', @@ -655,6 +657,8 @@ 'defines': [ 'NODE_WANT_INTERNALS=1' ], 'sources': [ + 'src/node_platform.cc', + 'src/node_platform.h', 'test/cctest/test_base64.cc', 'test/cctest/test_environment.cc', 'test/cctest/test_util.cc', |