simon/moonpool
1
0
Fork 0
mirror of https://github.com/c-cube/moonpool.git synced 2025-12-06 03:05:30 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix fiber: use a single fut/computation in fibers

Browse source 
otherwise we check the wrong computation for cancellation,
see https://discuss.ocaml.org/t/confused-about-moonpool-cancellation/15381
This commit is contained in:
Simon Cruanes 2024-10-03 12:37:51 -04:00
parent 0d8767f45f
commit 6a44598a31
No known key found for this signature in database
GPG key ID: EBFFF6F283F3A2B4
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/fib/fiber.ml
View file

@ -57,9 +57,8 @@ end
include Private_
let create_ ~pfiber ~runner () : 'a t =
let create_ ~pfiber ~runner ~res () : 'a t =
let id = Handle.generate_fresh () in
let res, _ = Fut.make () in
{
state =
A.make
@ -254,8 +253,8 @@ let add_child_ ~protect (self : _ t) (child : _ t) =
done
let spawn_ ~parent ~runner (f : unit -> 'a) : 'a t =
let comp = Picos.Computation.create () in
let pfiber = PF.create ~forbid:false comp in
let res, _ = Fut.make () in
let pfiber = PF.create ~forbid:false (Fut.Private_.as_computation res) in
(* copy local hmap from parent, if present *)
Option.iter
@ -265,7 +264,7 @@ let spawn_ ~parent ~runner (f : unit -> 'a) : 'a t =
(match parent with
| Some p when is_closed p -> failwith "spawn: nursery is closed"
| _ -> ());
let fib = create_ ~pfiber ~runner () in
let fib = create_ ~pfiber ~runner ~res () in
let run () =
(* make sure the fiber is accessible from inside itself *)