[PATCH] D151022: [llvm-exegesis] Introduce SubprocessMemory Utility Class

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 30 08:22:33 PDT 2023


bjope added a comment.

In D151022#4456689 <https://reviews.llvm.org/D151022#4456689>, @aidengrossman wrote:

> I was not considering multi-user setups. I'll look into writing a patch up for that. The process should clean up after its own shared memory. `shm_unlink` should get called in `~SubprocessMemory`, but there's nothing checking that the call actually succeeds. I'll look at doing some more investigation there and probably add an assertion/check within the test.

Any progress on this? It is a bit annoying that the /dev/shm/auxmem* isn't cleaned up automatically. I currently need to manually clean such things up from all our servers where our downstream build bots are running.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151022/new/

https://reviews.llvm.org/D151022



More information about the llvm-commits mailing list