chore: bump use i32 for pid in instrument-hooks api

Author: GuillaumeLagrange

crates/codspeed/instrument-hooks

@@ -4,7 +4,11 @@ pub const MARKER_TYPE_SAMPLE_START: u32 = 0;
 pub const MARKER_TYPE_SAMPLE_END: u32 = 1;
 pub const MARKER_TYPE_BENCHMARK_START: u32 = 2;
 pub const MARKER_TYPE_BENCHMARK_END: u32 = 3;
-pub type InstrumentHooks = *mut u64;
+#[repr(C)]
+#[derive(Debug, Copy, Clone)]
+pub struct InstrumentHooks {
+    _unused: [u8; 0],
+}
 extern "C" {
     pub fn instrument_hooks_init() -> *mut InstrumentHooks;
 }
@@ -44,7 +48,7 @@ extern "C" {
 extern "C" {
     pub fn instrument_hooks_add_marker(
         arg1: *mut InstrumentHooks,
-        pid: u32,
+        pid: i32,
         marker_type: u8,
         timestamp: u64,
     ) -> u8;
@@ -56,7 +60,7 @@ pub const instrument_hooks_feature_t_FEATURE_DISABLE_CALLGRIND_MARKERS: instrume
     0;
 pub type instrument_hooks_feature_t = ::std::os::raw::c_uint;
 extern "C" {
-    pub fn instrument_hooks_set_feature(feature: instrument_hooks_feature_t, enabled: bool);
+    pub fn instrument_hooks_set_feature(feature: u64, enabled: bool);
 }
 extern "C" {
     pub fn instrument_hooks_set_environment(
@@ -76,5 +80,5 @@ extern "C" {
     ) -> u8;
 }
 extern "C" {
-    pub fn instrument_hooks_write_environment(arg1: *mut InstrumentHooks, pid: u32) -> u8;
+    pub fn instrument_hooks_write_environment(arg1: *mut InstrumentHooks, pid: i32) -> u8;
 }

crates/codspeed/src/instrument_hooks/bindings.rs

@@ -129,7 +129,7 @@ mod linux_impl {
 
         #[inline(always)]
         pub fn add_benchmark_timestamps(&self, start: u64, end: u64) {
-            let pid = std::process::id();
+            let pid = std::process::id() as i32;
 
             unsafe {
                 ffi::instrument_hooks_add_marker(
@@ -191,7 +191,7 @@ mod linux_impl {
         }
 
         pub fn write_environment(&self) -> Result<(), u8> {
-            let pid = std::process::id();
+            let pid = std::process::id() as i32;
             let result = unsafe { ffi::instrument_hooks_write_environment(self.0, pid) };
             if result == 0 {
                 Ok(())
@@ -203,7 +203,7 @@ mod linux_impl {
         pub fn disable_callgrind_markers() {
             unsafe {
                 ffi::instrument_hooks_set_feature(
-                    ffi::instrument_hooks_feature_t_FEATURE_DISABLE_CALLGRIND_MARKERS,
+                    ffi::instrument_hooks_feature_t_FEATURE_DISABLE_CALLGRIND_MARKERS.into(),
                     true,
                 )
             };

crates/codspeed/src/instrument_hooks/mod.rs

@@ -7,4 +7,5 @@ bindgen "$SCRIPT_DIR/../../instrument-hooks/includes/core.h" \
   -o "$SCRIPT_DIR/bindings.rs" \
   --rust-target 1.74 \
   --allowlist-function "instrument_hooks_.*" \
-  --allowlist-var "MARKER_TYPE_.*"
+  --allowlist-var "MARKER_TYPE_.*" \
+  --allowlist-type "instrument_hooks_feature_t"