lowlevel-toolkit 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 05352e9b9bcf1c5e9edd57e688fc408ce72897baa33aaeb193a951be63729509
-  data.tar.gz: 379bd52a3d6a5d4f4fe85a907db67b4bcc32dffe758594031caf28b6eb3befe3
+  metadata.gz: 70c3208f01e4a74022c7a9863ef211932af9a4a68b488e4c48617330d669ee19
+  data.tar.gz: 9752724445dc3214669da9018c1f24d553356c54534787827f740f24c37aada8
 SHA512:
-  metadata.gz: 1126decf4abb9208ac5da0c32aad2be86dbd8cbf017f7074087f7066dbe9e22330ccd0b7400e7add3a40ce5f9c896b640e55d2a08a53da2cf58ea3d5ada62a73
-  data.tar.gz: 90bf7dbbe6e8d22a0c608c1a371b0861add4d57e8ecd7804706c6c7ad340fb9580966f94f88ac300223adbbb488065c047ca50222e31b3c9e30f99179564569a
+  metadata.gz: 2645f4993532e8fdc77e01e6f0e055225f3151c46f05b540522f7a2b773277dc29f5ada32589a5f2a3b47a0d7b1f02fb1618e620f9a0b406ad483393e0f1ebee
+  data.tar.gz: 8794ad2fe4f38b9b66dec535da028144f43144f62ed9d116633b0673af1067a8f5458c80123907b8f5f8f4139e4382749cfb216f3845d968dc70be1482d0ac9e

data/ext/lowlevel_toolkit_native_extension/last_allocation_at.c

@@ -9,7 +9,7 @@ static VALUE stack = Qnil;
 static void on_newobj_event(VALUE tpval, RB_UNUSED_VAR(void *_)) {
   if (stack == Qnil ||
     rb_objspace_internal_object_p(rb_tracearg_object(rb_tracearg_from_tracepoint(tpval)))) return;
-  VALUE buffer[MAX_DEPTH];
+  VALUE buffer[MAX_DEPTH]; // Temporary buffer for calling API
   int depth = rb_profile_frames(0, MAX_DEPTH, buffer, NULL);
   rb_ary_clear(stack);
   for (int i = 0; i < depth; i++) rb_ary_push(stack, buffer[i]);
@@ -29,7 +29,7 @@ static VALUE last_allocation_at(RB_UNUSED_VAR(VALUE _)) {
   for (int i = 0; i < RARRAY_LEN(raw_stack); i++) {
     VALUE entry = rb_ary_entry(raw_stack, i);
     VALUE file = rb_profile_frame_path(entry);
-    if (file != Qnil) rb_ary_push(result, rb_ary_new_from_args(2, file, rb_profile_frame_base_label(entry)));
+    if (file != Qnil) rb_ary_push(result, rb_ary_new_from_args(2, file, rb_profile_frame_label(entry)));
   }
   stack = raw_stack; // Resume recording again
   return result;

data/ext/lowlevel_toolkit_native_extension/print_gc_timing.c

@@ -7,6 +7,8 @@ static uint64_t get_monotonic_time_ns(void) {
   return (uint64_t) ts.tv_sec * 1000000000ULL + (uint64_t) ts.tv_nsec;
 }
 
+static inline double ns_to_ms(uint64_t ns) { return ns / 1000000.0; }
+
 static void on_gc_event(VALUE tpval, RB_UNUSED_VAR(void *_)) {
   static uint64_t gc_start_time = 0;
   rb_event_flag_t event = rb_tracearg_event_flag(rb_tracearg_from_tracepoint(tpval));
@@ -14,7 +16,7 @@ static void on_gc_event(VALUE tpval, RB_UNUSED_VAR(void *_)) {
   if (event == RUBY_INTERNAL_EVENT_GC_ENTER) {
     gc_start_time = get_monotonic_time_ns();
   } else if (event == RUBY_INTERNAL_EVENT_GC_EXIT) {
-    fprintf(stdout, "GC worked for %.2f ms\n", ((get_monotonic_time_ns() - gc_start_time) / 1000000.0));
+    fprintf(stdout, "GC worked for %.2f ms\n", ns_to_ms(get_monotonic_time_ns() - gc_start_time));
   }
 }
 

data/ext/lowlevel_toolkit_native_extension/release_gvl_profiler.c

@@ -26,17 +26,16 @@ static void on_thread_event(rb_event_flag_t event_id, const rb_internal_thread_e
 
     // If we wanted to delay processing the backtrace, we could switch to rb_profile_frames() here
     VALUE frames = rb_make_backtrace();
-    VALUE result = (VALUE) data;
 
-    VALUE stats = rb_hash_aref(result, frames);
+    VALUE stats = rb_hash_aref((VALUE) data, frames);
     if (stats == Qnil) {
       stats = rb_ary_new_from_args(2, INT2FIX(0), INT2FIX(0));
-      rb_hash_aset(result, frames, stats);
+      rb_hash_aset((VALUE) data, frames, stats);
     }
 
     uint64_t time_spent = get_monotonic_time_ns() - release_gvl_at;
-    rb_ary_store(stats, 0, ULL2NUM(NUM2ULL(rb_ary_entry(stats, 0)) + time_spent));
-    rb_ary_store(stats, 1, ULL2NUM(NUM2ULL(rb_ary_entry(stats, 1)) + 1));
+    rb_ary_store(stats, 0, ULL2NUM(NUM2ULL(rb_ary_entry(stats, 0)) + time_spent)); // Time
+    rb_ary_store(stats, 1, ULL2NUM(NUM2ULL(rb_ary_entry(stats, 1)) + 1));          // Counts
   }
 }
 

data/ext/lowlevel_toolkit_native_extension/track_wants_gvl.c

@@ -14,6 +14,8 @@ static int total_wants_key;
 static void on_thread_event(rb_event_flag_t event_id, const rb_internal_thread_event_data_t *event_data, void *data) {
   VALUE thread = event_data->thread;
   if (event_id == RUBY_INTERNAL_THREAD_EVENT_STARTED) {
+    // BUG: This is not thread safe... but works sometimes
+    // TODO: Replace with safe data structure
     rb_ary_push((VALUE) data, thread);
   } else if (event_id == RUBY_INTERNAL_THREAD_EVENT_READY) {
     rb_internal_thread_specific_set(thread, wants_gvl_at_key, (void *) get_monotonic_time_ns());

data/ext/lowlevel_toolkit_native_extension/who_called_me.c

@@ -1,11 +1,11 @@
 #include <ruby/ruby.h>
 #include <ruby/debug.h>
 
-static VALUE callback(const rb_debug_inspector_t *dbg_context, void *data) {
-  bool return_self = (data != NULL);
-  return return_self ?
+static VALUE callback(const rb_debug_inspector_t *dbg_context, void *return_self) {
+  return (return_self != NULL) ?
     // Depth: 0 -> us; 1 -> caller; 2 -> caller of caller
-    rb_debug_inspector_frame_self_get(dbg_context, 2) : rb_debug_inspector_frame_binding_get(dbg_context, 2);
+    rb_debug_inspector_frame_self_get(dbg_context, 2) :
+    rb_debug_inspector_frame_binding_get(dbg_context, 2);
 }
 
 static VALUE who_called_me(RB_UNUSED_VAR(VALUE _)) { return rb_debug_inspector_open(callback, (void *) !NULL); }

data/lib/lowlevel_toolkit/version.rb

@@ -26,5 +26,5 @@
 # frozen_string_literal: true
 
 module LowlevelToolkit
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: lowlevel-toolkit
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Ivo Anjo
 bindir: bin
 cert_chain: []
-date: 2025-04-08 00:00:00.000000000 Z
+date: 2025-04-15 00:00:00.000000000 Z
 dependencies: []
 email:
 - ivo@ivoanjo.me