Skip to content
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

addressing issue #17 : https://github.com/ocaml-multicore/parallel-programming-in-multicore-ocaml/issues/17 #19

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cbadjatya
Copy link

@cbadjatya cbadjatya commented Oct 21, 2023

Added a section for instrumentation with Olly.

Copy link
Collaborator

@Sudha247 Sudha247 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, @cbadjatya! This is a helpful change to point users to the latest tooling for instrumentation. I've left some suggestions below.

Olly can be used in the following manner `olly trace program_trace.json 'program.exe [args]'`. The json file can then be viewed in your browser.

Olly also allows you to measure the GC latency and view a parseable histogram summary of pause durations. Latency can be measured using
`olly latency 'program.exe [args]'`.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

latency is deprecated; use gc-stats instead.


*Olly* Olly is part of the `runtime_events_tools` package. Install Olly on your switch using `opam install runtime_event_tools`.

Olly can be used in the following manner `olly trace program_trace.json 'program.exe [args]'`. The json file can then be viewed in your browser.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

olly has two modes now, json and fuchsia.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants