-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Statmemprof for multicore [do not merge] #12379
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
… profile inheritance behaviour when threads or domains terminate; and (b) add explicit discard function to manage stopped profiles.
…ace interface, and the race conditions.
Each tool should depend only on the libraries it is actually linked with, rather than on all libraries.
Also give more details on separately-maintained components. Also update links to Github workflows.
…onfig on a child domain should be the same as a parent domain's.
|
Note: I will be squashing these commits into a more coherent set and order. |
|
Thanks for making the PR @NickBarnes. I've marked this as a draft until it is ready for review. As a start, it would be helpful to get answers to a few questions:
|
Single-domain semantic changesI have made a few small semantic changes for single-domain programs. I have discussed these with the relevant developers at Jane Street, who are the main users of statmemprof in OCaml 4 (via their memtrace tool).
Multi-domain and multi-thread semantics
|
|
Thanks for the answers Nick. |
|
I am restructuring this as a number of much smaller PRs which can be reviewed more-or-less independently. #12381 is the first. |
|
I've introduced the label |
Just a quick note: there are statmemprof users outside Jane Street, and going forward it would be nice to give them the opportunity the participate to API evolution discussion. For example some discussions could happen using an issue on github/ocaml or thee new "Discussion", or on Discuss, whatever. |
How useful is this distinction? I suspect that it is not hard to make it work in OCaml 5.0. |
|
Closing since #12923 (which was much better) was merged. |
This is a version of statmemprof for multicore, addressing #11911. It's not fully debugged, and requires more comments and more work on locking (or lock-free data structures) but I'm making this PR to provide visibility into development, and in the hope of some constructive review. I'll post more description in this comment thread.