perftools.rb 0.3.6 → 0.3.7

data/README

@@ -21,6 +21,14 @@ google-perftools for ruby code
 
     $ CPUPROFILE=/tmp/my_app_profile RUBYOPT="-r`gem which perftools | tail -1`" ruby my_app.rb
 
+  Change the sampling interval (defaults to 100 times a second, valid range is 1-4000):
+
+    $ CPUPROFILE_FREQUENCY=500 ruby my_app.rb
+
+  Use walltime instead of cputime profiling:
+
+    $ CPUPROFILE_REALTIME=1 ruby my_app.rb
+
 
 === Reporting
 

data/ext/extconf.rb

@@ -45,7 +45,8 @@ Dir.chdir('src') do
       ['perftools-gc', true],
       ['perftools-osx', RUBY_PLATFORM =~ /darwin/],
       ['perftools-osx-106', RUBY_PLATFORM =~ /darwin10/],
-      ['perftools-debug', true]
+      ['perftools-debug', true],
+      ['perftools-realtime', true]
     ].each do |patch, apply|
       if apply
         sys("patch -p1 < ../../../patches/#{patch}.patch")

data/ext/perftools.c

@@ -54,7 +54,7 @@ static VALUE I__send__;
     */
 
     for (; frame && (n = frame->node); frame = frame->prev) {
-      if (frame->prev && frame->prev->last_func) {
+      if (frame->prev > 0xff && frame->prev->last_func) {
         if (frame->prev->node == n) {
           if (frame->prev->last_func == frame->last_func) continue;
         }

data/patches/perftools-realtime.patch

@@ -0,0 +1,68 @@
+diff --git a/src/profile-handler.cc b/src/profile-handler.cc
+index 370d012..619b980 100644
+--- a/src/profile-handler.cc
++++ b/src/profile-handler.cc
+@@ -133,6 +133,9 @@ class ProfileHandler {
+   // SIGPROF interrupt frequency, read-only after construction.
+   int32 frequency_;
+ 
++  // use SIGALRM/ITIMER_REAL
++  bool realtime_;
++
+   // Counts the number of callbacks registered.
+   int32 callback_count_ GUARDED_BY(control_lock_);
+ 
+@@ -241,6 +244,13 @@ ProfileHandler::ProfileHandler()
+       callback_count_(0),
+       timer_sharing_(TIMERS_UNTOUCHED) {
+   SpinLockHolder cl(&control_lock_);
++
++  const char* rt = getenv("CPUPROFILE_REALTIME");
++  if (rt != NULL)
++    realtime_ = true;
++  else
++    realtime_ = false;
++
+   // Get frequency of interrupts (if specified)
+   char junk;
+   const char* fr = getenv("CPUPROFILE_FREQUENCY");
+@@ -396,18 +406,18 @@ void ProfileHandler::StartTimer() {
+   timer.it_interval.tv_sec = 0;
+   timer.it_interval.tv_usec = 1000000 / frequency_;
+   timer.it_value = timer.it_interval;
+-  setitimer(ITIMER_PROF, &timer, 0);
++  setitimer(realtime_ ? ITIMER_REAL : ITIMER_PROF, &timer, 0);
+ }
+ 
+ void ProfileHandler::StopTimer() {
+   struct itimerval timer;
+   memset(&timer, 0, sizeof timer);
+-  setitimer(ITIMER_PROF, &timer, 0);
++  setitimer(realtime_ ? ITIMER_REAL : ITIMER_PROF, &timer, 0);
+ }
+ 
+ bool ProfileHandler::IsTimerRunning() {
+   struct itimerval current_timer;
+-  RAW_CHECK(0 == getitimer(ITIMER_PROF, &current_timer), "getitimer");
++  RAW_CHECK(0 == getitimer(realtime_ ? ITIMER_REAL : ITIMER_PROF, &current_timer), "getitimer");
+   return (current_timer.it_value.tv_sec != 0 ||
+           current_timer.it_value.tv_usec != 0);
+ }
+@@ -417,7 +427,7 @@ void ProfileHandler::EnableHandler() {
+   sa.sa_sigaction = SignalHandler;
+   sa.sa_flags = SA_RESTART | SA_SIGINFO;
+   sigemptyset(&sa.sa_mask);
+-  RAW_CHECK(sigaction(SIGPROF, &sa, NULL) == 0, "sigprof (enable)");
++  RAW_CHECK(sigaction(realtime_ ? SIGALRM : SIGPROF, &sa, NULL) == 0, "sigprof (enable)");
+ }
+ 
+ void ProfileHandler::DisableHandler() {
+@@ -425,7 +435,7 @@ void ProfileHandler::DisableHandler() {
+   sa.sa_handler = SIG_IGN;
+   sa.sa_flags = SA_RESTART;
+   sigemptyset(&sa.sa_mask);
+-  RAW_CHECK(sigaction(SIGPROF, &sa, NULL) == 0, "sigprof (disable)");
++  RAW_CHECK(sigaction(realtime_ ? SIGALRM : SIGPROF, &sa, NULL) == 0, "sigprof (disable)");
+ }
+ 
+ void ProfileHandler::SignalHandler(int sig, siginfo_t* sinfo, void* ucontext) {

data/patches/perftools.patch

@@ -1,9 +1,3 @@
-commit 4ba520684b8171488971ba2039730122a0bc7faf
-Author: Aman Gupta <aman@tmm1.net>
-Date:   Mon Oct 26 16:10:31 2009 -0700
-
-    perftools
-
 diff --git a/Makefile.in b/Makefile.in
 index 232d2ec..566d77c 100644
 --- a/Makefile.in

data/perftools.rb.gemspec

@@ -1,7 +1,7 @@
 spec = Gem::Specification.new do |s|
   s.name = 'perftools.rb'
-  s.version = '0.3.6'
-  s.date = '2009-10-26'
+  s.version = '0.3.7'
+  s.date = '2009-11-03'
   s.rubyforge_project = 'perftools-rb'
   s.summary = 'google-perftools for ruby code'
   s.description = 'A sampling profiler for ruby code based on patches to google-perftools'
@@ -28,6 +28,7 @@ spec = Gem::Specification.new do |s|
     "patches/perftools-osx-106.patch",
     "patches/perftools-osx.patch",
     "patches/perftools-pprof.patch",
+    "patches/perftools-realtime.patch",
     "patches/perftools-notests.patch",
     "patches/perftools.patch",
     "perftools.rb.gemspec"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: perftools.rb
 version: !ruby/object:Gem::Version 
-  version: 0.3.6
+  version: 0.3.7
 platform: ruby
 authors: 
 - Aman Gupta
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-10-26 00:00:00 -07:00
+date: 2009-11-03 00:00:00 -08:00
 default_executable: 
 dependencies: []
 
@@ -32,6 +32,7 @@ files:
 - patches/perftools-osx-106.patch
 - patches/perftools-osx.patch
 - patches/perftools-pprof.patch
+- patches/perftools-realtime.patch
 - patches/perftools-notests.patch
 - patches/perftools.patch
 - perftools.rb.gemspec