fast_stack 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a7b203e805558207a896a66af413dfbbf48fbf0d
-  data.tar.gz: f8aaa3a556df3007d936e21c7983512ab04f1eea
+  metadata.gz: a98a5bc1eccd73f374b7ed882f44d79aaab1cf65
+  data.tar.gz: 5ea09b23cae084b138af88b10e038add9fd9746a
 SHA512:
-  metadata.gz: 6eef1b4399175be510fc9a93c5948d0791b6d2139b3da100cbecd39fabe1fb3d33397d198faf91d2218fe6e20570e843dd71de9478f5fd2430cc8b6da21c7b02
-  data.tar.gz: fd2f07f355adb448f7feeeb716a5c9f748d9fccfe1baf9ad26bebea1bf9ff0894ea7eaddd61bd37a18f9d66aa19ed324e6308061d47ebcf9841bb7d21788cc09
+  metadata.gz: 94de99fd759ed77a9b0b5c02fd05ea5cb65efd0afa3cbf0ce89ac9d6ba6ec566bd370d32b1bc4f5c752931c8a3210ebd9f19a8db65622f241f9549f2dd85b726
+  data.tar.gz: c2fec3c6ffa3bef3573c98d348f165bf8c19f4eb0836e15b25fc2bcd557c7fd3d5abc8ef4c3b126c0932dc8947cd37829860c149185a711a96b51f301971cf02

data/ext/fast_stack/fast_stack.c

@@ -1,4 +1,5 @@
 #include <stdio.h>
+#include <sys/time.h>
 #include <ruby.h>
 #include <ruby/encoding.h>
 
@@ -7,9 +8,9 @@ profiler_start(VALUE module, VALUE usec)
 {
     struct itimerval timer;
     timer.it_interval.tv_sec = 0;
-    timer.it_interval.tv_usec = NUM2LONG(usec);
+    timer.it_interval.tv_usec = (suseconds_t)NUM2LONG(usec);
     timer.it_value = timer.it_interval;
-    setitimer(ITIMER_PROF, &timer, 0);
+    setitimer(ITIMER_REAL, &timer, 0);
 
     return Qnil;
 }

data/lib/fast_stack.rb

@@ -1,54 +1,26 @@
 require 'fast_stack/fast_stack'
 module FastStack
   def self.profile(millisecs=1, &blk)
-    done = false
     stacks = []
     thread = Thread.current
-    delta = millisecs / 1000.0
     last_time = Time.new
 
     new_api = thread.respond_to?(:backtrace_locations)
 
-    # NOTE not extracting to another method to avoid adding a frame
-    #  to the stack
-
-    # this thread catches stuff fast stack does not
-    # the main case is catching sleeping threads
-    profile_thread = Thread.new do
-      until done
+    trap('ALRM') do
+      stack = (new_api ? thread.backtrace_locations : thread.backtrace)
+      # I am not sure if this is ensured to run in the thread
+      # though in my samples it does
+      if thread == Thread.current
+        stack = stack[2..-1]
+        # since we are in the same thread, might as well remove
+        #  our overhead
         start = Time.new
-        if last_time < start - delta
-          last_time = start
-          stacks << (new_api ? thread.backtrace_locations : thread.backtrace)
-        end
-
-        sleep(delta)
-      end
-    end
-
-    trap('PROF') do
-      start = Time.new
-      if last_time < start - delta
-        last_time = start
-        stack = (new_api ? thread.backtrace_locations : thread.backtrace)
-        # I am not sure if this is ensured to run in the thread
-        # though in my samples it does
-        if thread == Thread.current
-          stack = stack[2..-1]
-          # since we are in the same thread, might as well remove
-          #  our overhead
-          start = Time.new
-        end
-        stacks << stack
       end
+      stacks << stack
     end
 
-    begin
-      profile_block(millisecs * 1000, &blk)
-    ensure
-      done = true
-      profile_thread.join
-    end
+    profile_block(millisecs * 1000, &blk)
 
     stacks
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fast_stack
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Sam Saffron
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-02 00:00:00.000000000 Z
+date: 2013-09-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake