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

Link information from JLM and MonitorContended events #56

Open
mpirvu opened this issue Mar 22, 2021 · 0 comments · May be fixed by #72
Open

Link information from JLM and MonitorContended events #56

mpirvu opened this issue Mar 22, 2021 · 0 comments · May be fixed by #72

Comments

@mpirvu
Copy link
Contributor

mpirvu commented Mar 22, 2021

JLM (java lock monitor) gives us a summary of activity on monitors, whereas MonitorContended events are triggered for every monitor individually and they also can be used to compute the waiting time for a particular monitor operation (something that JLM does not do).
Ideally, we would first use the information from JLM to determine which monitors are expensive and then use the MonitorContended events to drill deeper and find stack traces for threads waiting on expensive monitors and for threads holding onto expensive monitors. To do that we need some common monitor information between the tho sources. There are two possibilities:

  1. Use the raw address of the OpenJ9 monitors. This is already printed by JLM and we need a way to find this address from the MonitorContended events.
  2. Use the hash value of the monitors (actually for the object we are synchronizing on). This is available in the current version of the code for the MonitorContended events, but is not available in JLM. We would need to modify OpenJ9 repo to make this change.
yathamravali added a commit to yathamravali/openj9-utils that referenced this issue May 21, 2021
Fixes: eclipse-openj9#56

Signed-Off-by: Ravali Yatham <[email protected]>
yathamravali added a commit to yathamravali/openj9-utils that referenced this issue May 21, 2021
Fixes: eclipse-openj9#56

Signed-off-by: Ravali Yatham <[email protected]>
@yathamravali yathamravali linked a pull request May 21, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant