Merge pull request #304 from talex5/trace-calls

Record a trace event for method calls
mirage · Nov 30, 2024 · 252fd9b · 252fd9b
2 parents 574768a + 498a144
commit 252fd9b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -44,7 +44,7 @@ New features:
 
 API changes:
 
-- Eio port (@talex5 #280 #284 #298 #292 #297 #300).
+- Eio port (@talex5 #280 #284 #298 #292 #297 #300 #304).
 
   This switches capnp-rpc from Lwt to Eio.
   One particularly nice side effect of this is that `Service.return_lwt` has gone,

diff --git a/capnp-rpc/service.ml b/capnp-rpc/service.ml
@@ -38,7 +38,10 @@ let local (s:#generic) =
       let call = Msg.Request.readable msg in
       let interface_id = Call.interface_id_get call in
       let method_id = Call.method_id_get call in
-      Log.debug (fun f -> f "Invoking local method %a" pp_method (interface_id, method_id));
+      Log.debug (fun f ->
+          Eio.Private.Trace.log (Fmt.str "%a" pp_method (interface_id, method_id));
+          f "Invoking local method %a" pp_method (interface_id, method_id)
+        );
       let p = Call.params_get call in
       let m : abstract_method_t = s#dispatch ~interface_id ~method_id in
       let release_params () = Core_types.Request_payload.release msg in