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
Add support for thread local allocation stats #18202
Conversation
8cddc42
to
7d91123
Compare
ad804dd
to
3245fea
Compare
Signed-off-by: Tobi Ajila <atobia@ca.ibm.com>
No description provided. |
jenkins test sanity zlinux jdk21 |
jenkins compile win jdk8 |
if (mmfns->j9gc_get_cumulative_bytes_allocated_by_thread(currentThread, &allocatedBytes)) { | ||
result = (jlong) allocatedBytes; | ||
} | ||
goto done; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This leads to a second call to internalExitVMToJNI()
; perhaps the done
label should be after that second call?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or remove the exit call above - the GC code is called with VM access below, so it must not care.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think removing the call above would be more consistent (then both places where `j9gc_get_cumulative_bytes_allocated_by_thread() are called would be in the VM).
@TobiAjila the new feature should be documented in the What's New. Please create a docs issue https://github.com/eclipse-openj9/openj9-docs/issues/ |
No description provided.