Add JFR CPULoad and ThreadCPU event support #20186
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@tajila requesting your review |
|
Im seeing negative numbers for Thread user mode CPU load |
tajila
reviewed
Oct 4, 2024
tajila
reviewed
Oct 4, 2024
tajila
reviewed
Oct 4, 2024
tajila
reviewed
Oct 4, 2024
tajila
reviewed
Oct 4, 2024
tajila
reviewed
Oct 4, 2024
thallium
force-pushed
the
cpuload
branch
2 times, most recently
from
9399d91 to
9f17f2e
Compare
tajila
reviewed
Oct 7, 2024
thallium
force-pushed
the
cpuload
branch
3 times, most recently
from
42e1d9d to
94218be
Compare
tajila
reviewed
Oct 10, 2024
tajila
reviewed
Oct 10, 2024
tajila
reviewed
Oct 10, 2024
tajila
reviewed
Oct 10, 2024
thallium
added a commit
to thallium/omr
that referenced
this pull request
Oct 10, 2024
This function returns the user and system cpu time of the calling thread. Related: eclipse-openj9/openj9#20186 Signed-off-by: Gengchen Tuo <[email protected]>
|
Opend OMR PR and addressed previous state. |
thallium
added a commit
to thallium/omr
that referenced
this pull request
Oct 10, 2024
This function returns the user and system cpu time of the calling thread. Related: eclipse-openj9/openj9#20186 Signed-off-by: Gengchen Tuo <[email protected]>
thallium
added a commit
to thallium/omr
that referenced
this pull request
Oct 11, 2024
This function returns the user and system cpu time of the calling thread. Related: eclipse-openj9/openj9#20186 Signed-off-by: Gengchen Tuo <[email protected]>
thallium
added a commit
to thallium/omr
that referenced
this pull request
Oct 11, 2024
This function returns the user and system cpu time of the calling thread. Related: eclipse-openj9/openj9#20186 Signed-off-by: Gengchen Tuo <[email protected]>
keithc-ca
requested changes
Oct 11, 2024
thallium
added a commit
to thallium/omr
that referenced
this pull request
Oct 11, 2024
This function returns the user and system cpu time of the calling thread. This new function is needed because the only way that we're aware of to get the user CPU time of a thread is to use getrusage(RUSAGE_THREAD, ...) which can only be called on the current thread. Related: eclipse-openj9/openj9#20186 Signed-off-by: Gengchen Tuo <[email protected]>
thallium
added a commit
to thallium/omr
that referenced
this pull request
Oct 11, 2024
This function returns the user and system cpu time of the calling thread. This new function is needed because the only way that we're aware of to get the user CPU time of a thread is to use getrusage(RUSAGE_THREAD, ...) which can only be called on the current thread. Related: eclipse-openj9/openj9#20186 Signed-off-by: Gengchen Tuo <[email protected]>
thallium
added a commit
to thallium/omr
that referenced
this pull request
Oct 11, 2024
This function returns the user and system cpu time of the calling thread. This new function is needed because the only way that we're aware of to get the user CPU time of a thread is to use getrusage(RUSAGE_THREAD, ...) which can only be called on the current thread. Related: eclipse-openj9/openj9#20186 Signed-off-by: Gengchen Tuo <[email protected]>
|
Addressed previous feedbacks |
|
@thallium Please rebase your changes |
|
Rebased |
keithc-ca
requested changes
Oct 31, 2024
|
@thallium Please fix merge conflicts |
|
Resolved merge conflict. |
|
I had to add |
tajila
approved these changes
Nov 7, 2024
tajila
reviewed
Nov 7, 2024
|
@keithc-ca Please take another look at this |
|
Opened a PR to fix the missing export eclipse-omr/omr#7535 |
|
jenkins test sanity xlinux jdk17 |
|
@keithc-ca Do you have any more comments? |
|
I'll try to find time today to have another look. |
keithc-ca
requested changes
Nov 11, 2024
|
Addressed feedbacks. |
keithc-ca
requested changes
Nov 11, 2024
Depends on: eclipse-omr/omr#7491 Signed-off-by: Gengchen Tuo <[email protected]>
|
@keithc-ca Please take another look at this |
|
Jenkins test sanity xlinux jdk17 |
keithc-ca
approved these changes
Nov 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Depends on: eclipse-omr/omr#7491