[libc-commits] [PATCH] D148288: [libc] Update RPC interface for system utilities on the GPU
Joseph Huber via Phabricator via libc-commits
libc-commits at lists.llvm.org
Mon Apr 17 19:35:43 PDT 2023
jhuber6 added inline comments.
================
Comment at: libc/src/__support/RPC/rpc.h:251
+
+ // Once we acquire the index we need to check if we are in a valid sending
+ // state.
----------------
JonChesterfield wrote:
> This leaks ports if they're closed after a non-even number of send or recv calls
The port can be opened in the Ack / Data state `0, 0` or `1, 1`. it doesn't get stuck at `1, 1` because we treat `1, 1` the same as `0, 0` in the logic.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148288/new/
https://reviews.llvm.org/D148288
More information about the libc-commits
mailing list