Revision: 7059
Author: [email protected]
Date: Fri Mar 4 02:12:33 2011
Log: Start sampler for the runtime profiler even when we can't setup a
signal handler.
Review URL: http://codereview.chromium.org/6625013
http://code.google.com/p/v8/source/detail?r=7059
Modified:
/branches/bleeding_edge/src/platform-linux.cc
=======================================
--- /branches/bleeding_edge/src/platform-linux.cc Fri Mar 4 01:36:04 2011
+++ /branches/bleeding_edge/src/platform-linux.cc Fri Mar 4 02:12:33 2011
@@ -584,7 +584,9 @@
// This is also initialized by the first argument to pthread_create()
but we
// don't know which thread will run first (the original thread or the new
// one) so we initialize it here too.
- prctl(PR_SET_NAME, reinterpret_cast<unsigned long>(thread->name()), 0,
0, 0);
+ prctl(PR_SET_NAME,
+ reinterpret_cast<unsigned long>(thread->name()), // NOLINT
+ 0, 0, 0);
thread->thread_handle_data()->thread_ = pthread_self();
ASSERT(thread->IsValid());
thread->Run();
@@ -871,6 +873,7 @@
}
void SendProfilingSignal() {
+ if (!signal_handler_installed_) return;
// Glibc doesn't provide a wrapper for tgkill(2).
syscall(SYS_tgkill, vm_tgid_, vm_tid_, SIGPROF);
}
@@ -937,8 +940,8 @@
sa.sa_sigaction = ProfilerSignalHandler;
sigemptyset(&sa.sa_mask);
sa.sa_flags = SA_RESTART | SA_SIGINFO;
- if (sigaction(SIGPROF, &sa, &data_->old_signal_handler_) != 0) return;
- data_->signal_handler_installed_ = true;
+ data_->signal_handler_installed_ =
+ sigaction(SIGPROF, &sa, &data_->old_signal_handler_) == 0;
// Start a thread that sends SIGPROF signal to VM thread.
// Sending the signal ourselves instead of relying on itimer provides
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
