gvl_timing 0.2.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db733cb6bbf991888c58e4d4aede834efaaec117a02c7ca2822f19ef1c842eec
-  data.tar.gz: a86891666732c57f85b46e9eaf8fb88cc5f265449a282f6d1bc1e444b4dd6ac4
+  metadata.gz: c074fa63b689a97efe9aace8d5771f2d028c624f321375999cfc2f65e88a90ce
+  data.tar.gz: b33d3f6aa788576f5f7265c9336a4f4dabf916a84608e7af3f85d087a6551b1d
 SHA512:
-  metadata.gz: 89b22673ab84fba5fb065f34180fbe8bd83b23b77432f60e0736bd7f5a137fc0c285aff73f8527207ffd6ec3593301e434047a7dbd70f3b80c19bf265f2b6dd5
-  data.tar.gz: '09fcfe0991aa902049de16ac2b3e9d8cfe5f9142641258cddcb6b60d8159b00636579231a70a7ccb09544977e415dadc5bc9334b0513db4e9b4eac06316028ad'
+  metadata.gz: 447646c28a9412850137bfca22152c51fe8c002fea31fda3f2b3e999c7f628d699eb09360a451edc7f03621e4dd8a3c0a13854c40cc16ee779f3d7b44ad69abb
+  data.tar.gz: ae1217aa724bbc0f1672ce98573af54d54a4e25dc4d413ffa9bd461ef0f0de0ab7ae638b906d130e4edacfcbbc01e57ad836187af9c8d010196f7a682395b15f

data/ext/gvl_timing/extconf.rb

@@ -7,4 +7,7 @@ require "mkmf"
 # selectively, or entirely remove this flag.
 append_cflags("-fvisibility=hidden")
 
+have_header("ruby/thread.h")
+have_struct_member("rb_internal_thread_event_data_t", "thread", ["ruby/thread.h"])
+
 create_makefile("gvl_timing/gvl_timing")

data/ext/gvl_timing/gvl_timing.c

@@ -26,6 +26,8 @@ struct gvl_timer {
     uint64_t cputime_start;
     uint64_t cputime_stop;
 
+    uint64_t yields_count;
+
     uint64_t prev_timestamp;
     enum ruby_gvl_state prev_state;
     VALUE thread;
@@ -40,16 +42,23 @@ void record_timing(struct gvl_timer *timer, enum ruby_gvl_state new_state) {
     timer->timings[timer->prev_state] += (timestamp - timer->prev_timestamp);
     timer->prev_timestamp = timestamp;
     timer->prev_state = new_state;
+
+    if (new_state == GVL_STATE_IDLE) {
+        timer->yields_count++;
+    }
+
 }
 
 void internal_thread_event_cb(rb_event_flag_t event, const rb_internal_thread_event_data_t *event_data, void *data) {
     struct gvl_timer *timer = data;
 
+    VALUE thread;
+#if HAVE_RB_INTERNAL_THREAD_EVENT_DATA_T_THREAD
+    thread = event_data->thread;
+#else
     if (!ruby_native_thread_p()) return;
-    VALUE thread = rb_thread_current();
-//#if HAVE_RB_INTERNAL_THREAD_EVENT_DATA_T_THREAD
-//#else
-//#endif
+    thread = rb_thread_current();
+#endif
     if (thread != timer->thread) {
         return;
     }
@@ -150,6 +159,10 @@ VALUE gvl_timer_idle_duration(VALUE obj) {
     return ULL2NUM(get_timer(obj)->timings[GVL_STATE_IDLE]);
 }
 
+VALUE gvl_timer_yields_count(VALUE obj) {
+    return ULL2NUM(get_timer(obj)->yields_count);
+}
+
 RUBY_FUNC_EXPORTED void
 Init_gvl_timing(void)
 {
@@ -167,4 +180,6 @@ Init_gvl_timing(void)
     rb_define_method(rb_cTimer, "running_duration_ns", gvl_timer_running_duration, 0);
     rb_define_method(rb_cTimer, "stalled_duration_ns", gvl_timer_stalled_duration, 0);
     rb_define_method(rb_cTimer, "idle_duration_ns", gvl_timer_idle_duration, 0);
+
+    rb_define_method(rb_cTimer, "yields_count", gvl_timer_yields_count, 0);
 }

data/lib/gvl_timing/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GVLTiming
-  VERSION = "0.2.0"
+  VERSION = "0.3.1"
 end

data/lib/gvl_timing.rb

@@ -10,8 +10,11 @@ module GVLTiming
     def measure
       timer = Timer.new
       timer.start
-      yield
-      timer.stop
+      begin
+        yield
+      ensure
+        timer.stop
+      end
       timer
     end
     alias_method :time, :measure
@@ -31,7 +34,7 @@ module GVLTiming
     [
       :duration, :cpu_duration,
       :monotonic_start, :monotonic_stop,
-      :cputime_start, :cputime_start,
+      :cputime_start, :cputime_stop,
       :running_duration, :stalled_duration, :idle_duration
     ].each do |name|
       class_eval <<~RUBY
@@ -41,13 +44,15 @@ module GVLTiming
       RUBY
     end
 
+    alias releases_count yields_count
 
     def inspect
-      "#<#{self.class} total=%.2fs running=%.2fs idle=%.2fs stalled=%.2fs>" % [
+      "#<#{self.class} total=%.2fs running=%.2fs idle=%.2fs stalled=%.2fs yields=%d>" % [
         duration,
         running_duration,
         idle_duration,
         stalled_duration,
+        yields_count,
       ]
     end
 

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: gvl_timing
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.1
 platform: ruby
 authors:
 - John Hawthorn
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-06-18 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies: []
 description: Measure time spent in different GVL states
 email:
@@ -36,7 +35,6 @@ metadata:
   homepage_uri: https://github.com/jhawthorn/gvl_timing
   source_code_uri: https://github.com/jhawthorn/gvl_timing
   changelog_uri: https://github.com/jhawthorn/gvl_timing
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -51,8 +49,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.19
-signing_key:
+rubygems_version: 3.6.7
 specification_version: 4
 summary: Measure time spent in different GVL states
 test_files: []