jdk17u-dev icon indicating copy to clipboard operation
jdk17u-dev copied to clipboard

8030957: AIX: Implement OperatingSystemMXBean.getSystemCpuLoad() and .getProcessCpuLoad() on AIX

Open suchismith1993 opened this issue 3 weeks ago • 9 comments
trafficstars

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

  • [x] Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • [ ] JDK-8030957 needs maintainer approval
  • [x] Change must not contain extraneous whitespace
  • [x] Commit message must refer to an issue

Issue

  • JDK-8030957: AIX: Implement OperatingSystemMXBean.getSystemCpuLoad() and .getProcessCpuLoad() on AIX (Enhancement - P4 - Requested)

Reviewers

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/4113/head:pull/4113
$ git checkout pull/4113

Update a local copy of the PR:
$ git checkout pull/4113
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/4113/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 4113

View PR using the GUI difftool:
$ git pr show -t 4113

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/4113.diff

Using Webrev

Link to Webrev Comment

suchismith1993 avatar Oct 27 '25 09:10 suchismith1993

:wave: Welcome back sroy! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

bridgekeeper[bot] avatar Oct 27 '25 09:10 bridgekeeper[bot]

❗ This change is not yet ready to be integrated. See the Progress checklist in the description for automated requirements.

openjdk[bot] avatar Oct 27 '25 09:10 openjdk[bot]

This backport pull request has now been updated with issue from the original commit.

openjdk[bot] avatar Oct 27 '25 09:10 openjdk[bot]

Webrevs

mlbridge[bot] avatar Oct 27 '25 09:10 mlbridge[bot]

Hi @MBaesken , can you review the backport PR

suchismith1993 avatar Oct 27 '25 15:10 suchismith1993

Looks mostly okay. But the original commit
https://github.com/openjdk/jdk/commit/d45e65bab45f78f9f378cdc53837fe33190b7801 has no COPYRIGHT year change in test/jdk/ProblemList.txt , so I think you should not start introducing this in the backport .

MBaesken avatar Oct 28 '25 08:10 MBaesken

⚠️ @suchismith1993 This change is now ready for you to apply for maintainer approval. This can be done directly in each associated issue or by using the /approval command.

openjdk[bot] avatar Oct 30 '25 08:10 openjdk[bot]

/approval request AIX CPU load support using libperfstat (needed for MXBean metrics); low risk, verified on AIX with jtreg and topas.

suchismith1993 avatar Nov 01 '25 13:11 suchismith1993

@suchismith1993 8030957: The approval request has been created successfully.

openjdk[bot] avatar Nov 01 '25 13:11 openjdk[bot]