-
Notifications
You must be signed in to change notification settings - Fork 149
8030957: AIX: Implement OperatingSystemMXBean.getSystemCpuLoad() and .getProcessCpuLoad() on AIX #2403
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
base: master
Are you sure you want to change the base?
8030957: AIX: Implement OperatingSystemMXBean.getSystemCpuLoad() and .getProcessCpuLoad() on AIX #2403
Conversation
|
👋 Welcome back sroy! A progress list of the required criteria for merging this PR into |
|
@suchismith1993 This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be: You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been no new commits pushed to the As you do not have Committer status in this project an existing Committer must agree to sponsor your change. ➡️ To flag this PR as ready for integration with the above commit message, type |
|
This backport pull request has now been updated with issue from the original commit. |
|
|
Webrevs
|
|
/approval request "This feature is needed for customer builds" |
|
@suchismith1993 |
|
Backport to jdk25u needs to be done before jdk21u-dev. |
|
Can you please add a complete approval request text as described here? https://wiki.openjdk.org/display/JDKUpdates/How+to+contribute+or+backport+a+fix |
|
/approval request Critical fix as customers are unable to fetch cpu usage for AIX. |
|
@suchismith1993 |
|
Hi @suchismith1993 , please label again for approval once it is approved for 25. Also, please state risk and testing in your approval request. |
|
/approval request AIX CPU load support using libperfstat (needed for MXBean metrics); low risk, verified on AIX with jtreg and topas. |
|
@suchismith1993 |
|
@suchismith1993 This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a |
|
/approval request AIX CPU load support using libperfstat (needed for MXBean metrics); low risk, verified on AIX with jtreg and topas. |
|
@suchismith1993 |
|
Hi @GoeLin Changes made it to jdk25u. Can we get this integrated now ? |
|
Hi, please merge head and await the GHA tests before pushing. |
Hi all,
This pull request contains a backport of commit d45e65ba from the openjdk/jdk repository.
The commit being backported was authored by Suchismith Roy on 8 Oct 2025 and was reviewed by Joachim Kern, Martin Doerr and Matthias Baesken.
Thanks!
Progress
Issue
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk21u-dev.git pull/2403/head:pull/2403$ git checkout pull/2403Update a local copy of the PR:
$ git checkout pull/2403$ git pull https://git.openjdk.org/jdk21u-dev.git pull/2403/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 2403View PR using the GUI difftool:
$ git pr show -t 2403Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk21u-dev/pull/2403.diff
Using Webrev
Link to Webrev Comment