8370572: Cgroups hierarchical memory limit is not honored after JDK-8322420

[jerboaa]

Unclean backport of JDK-8370572 since JDK-8292984 is not in 25u (a cleanup patch). That change basically renames phys_mem to upper_bound. Not really critical to get JDK-8292984 into 25u.

Please review!

Testing:

• GHA
• Container tests on Linux x86_64 on cgroups v1 including the test from JDK-8370966 (see dependent PR). Also checked container tests on cgroups v2 for sanity. This patch is a no-op there.

---

Progress

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

Issue

• JDK-8370572: Cgroups hierarchical memory limit is not honored after JDK-8322420 (Bug - P4 - Approved)

Reviewers

• Paul Hohensee (@phohensee - Reviewer) Review applies to 36340d51
• Aleksey Shipilev (@shipilev - Reviewer)

Reviewing

Using git

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

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

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 76

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

Using diff file

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

Using Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back sgehwolf! 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.

[openjdk]

@jerboaa 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:

8370572: Cgroups hierarchical memory limit is not honored after JDK-8322420

Reviewed-by: shade, phh
Backport-of: c49a94bf89876c4d6c777a9452618afa564c5c23

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 18 new commits pushed to the master branch:

• 43f752f: 8293484: AArch64: TestUseSHA512IntrinsicsOptionOnSupportedCPU.java fails on CPU with SHA512 feature support
• 04bd246: 8366938: Test runtime/handshake/HandshakeTimeoutTest.java crashed
• b4041e3: 8358801: javac produces class that does not pass verifier.
• ... and 15 more: https://git.openjdk.org/jdk25u-dev/compare/687c6a0b0871f68a017b6a81782f9b28e5d9ddf9...master

As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the master branch, type /integrate in a new comment.

[openjdk]

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

[mlbridge]

Webrevs

• 01: Full - Incremental (ee19fd11)
• 00: Full (36340d51)

[jerboaa]

GHA failure of serviceability/jvmti/RedefineClasses/ClassVersionAfterRedefine on MacOSX seems unrelated (linux only patch).

[openjdk]

⚠️ @jerboaa 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.

[shipilev]

Thanks for doing this! I do have a comment:

[shipilev]

This looks great, thank you!

[jerboaa]

Thanks for the review!

[jerboaa]

/integrate

[openjdk]

Going to push as commit 5a08f94.
Since your change was applied there have been 20 commits pushed to the master branch:

• ced6e95: 8369911: Test sun/java2d/marlin/ClipShapeTest.java#CubicDoDash, #Cubic and #Poly fail intermittent
• 44ab436: 8371967: Add Visual Studio 2026 to build toolchain for Windows
• 43f752f: 8293484: AArch64: TestUseSHA512IntrinsicsOptionOnSupportedCPU.java fails on CPU with SHA512 feature support
• ... and 17 more: https://git.openjdk.org/jdk25u-dev/compare/687c6a0b0871f68a017b6a81782f9b28e5d9ddf9...master

Your commit was automatically rebased without conflicts.

[openjdk]

@jerboaa Pushed as commit 5a08f94.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.