src: fix v8::CpuProfiler idle sampling
Ensure that samples are evenly distributed, which is required for v8::CpuProfiler users to work correctly (v8-profiler / node-inspector). PR-URL: https://github.com/nodejs/node/pull/2324 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
This commit is contained in:
parent
68dc69a382
commit
8e141356ff
13
src/node.cc
13
src/node.cc
@ -3567,10 +3567,6 @@ inline void PlatformInit() {
|
||||
RegisterSignalHandler(SIGINT, SignalExit, true);
|
||||
RegisterSignalHandler(SIGTERM, SignalExit, true);
|
||||
|
||||
// Block SIGPROF signals when sleeping in epoll_wait/kevent/etc. Avoids the
|
||||
// performance penalty of frequent EINTR wakeups when the profiler is running.
|
||||
uv_loop_configure(uv_default_loop(), UV_LOOP_BLOCK_SIGNAL, SIGPROF);
|
||||
|
||||
// Raise the open file descriptor limit.
|
||||
struct rlimit lim;
|
||||
if (getrlimit(RLIMIT_NOFILE, &lim) == 0 && lim.rlim_cur != lim.rlim_max) {
|
||||
@ -3635,6 +3631,15 @@ void Init(int* argc,
|
||||
}
|
||||
}
|
||||
|
||||
#ifdef __POSIX__
|
||||
// Block SIGPROF signals when sleeping in epoll_wait/kevent/etc. Avoids the
|
||||
// performance penalty of frequent EINTR wakeups when the profiler is running.
|
||||
// Only do this for v8.log profiling, as it breaks v8::CpuProfiler users.
|
||||
if (v8_is_profiling) {
|
||||
uv_loop_configure(uv_default_loop(), UV_LOOP_BLOCK_SIGNAL, SIGPROF);
|
||||
}
|
||||
#endif
|
||||
|
||||
#if defined(NODE_HAVE_I18N_SUPPORT)
|
||||
if (icu_data_dir == nullptr) {
|
||||
// if the parameter isn't given, use the env variable.
|
||||
|
Loading…
x
Reference in New Issue
Block a user