[jdk8u-dev] RFR: 8244500: jtreg test error in test/hotspot/jtreg/containers/docker/TestMemoryAwareness.java [v4]
Jonathan Dowland
jdowland at openjdk.org
Fri Nov 4 10:24:00 UTC 2022
> This is a backport of 8244500 to jdk8u-dev as part of cgroups v2 support.
>
> It's a test-only change.
>
> The patch needed adjusting for jdk8u, mostly to JDK-module-specific things.
Jonathan Dowland has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 29 commits:
- Don't pass --add-exports to jdk8u java inside docker
These are JDK9+ module specific parameters.
- 8244500: jtreg test error in test/hotspot/jtreg/containers/docker/TestMemoryAwareness.java
When the kernel doesn't support swap limits, expect host values instead.
Reviewed-by: sgehwolf
Backport-of: fb5132254d834ba01a4b65ce64143843e83c674e
- Move test to a more 8u-appropriate location
Container (cgroups, docker) tests in 8u reside in
hotspot/test/runtime/containers
- remove duplicate include of osContainer_linux
With the backport of 8189762, the include of osContainer_linux was
moved to a later #ifdef stanza, relative to the original. This caused
a context problem with this backport.
- 8239785: Cgroups: Incorrect detection logic on old systems in hotspot
Return NULL subsystem if no cgroup controllers are mounted.
Reviewed-by: sgehwolf
Backport-of: add18914fb1294999877b563c734a25b4c17b922
- 8237479: 8230305 causes slowdebug build failure
Declare methods as pure virtual.
Backport-of: 4ca06995855b5c974321d7b3622d661b8d27ba76
- 8253714: [cgroups v2] Soft memory limit incorrectly using memory.high
The early implementation of cgroups v2 support was done with
crun 0.8 and it contained a bug which set memory.high over
memory.low when --memory-reservation was being used as a CLI
option.
This bug has been fixed in later crun versions, starting with
crun 0.11. Use memory.low in OpenJDK as well.
Backport-of: ff6843ca4842498791061f924c545fa9469cc1dc
- Address style nit
- TestCgroupSubsystemController: rework use of Files.writeString
- CgroupSubsystemController: fix library paths
We need the testlibrary copy of FileUtils but the test.lib.util copy of
Utils (method createTempDirectory is missing from the testlib copy)
- ... and 19 more: https://git.openjdk.org/jdk8u-dev/compare/f04ad96c...0cfe1277
-------------
Changes: https://git.openjdk.org/jdk8u-dev/pull/142/files
Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=142&range=03
Stats: 7873 lines in 48 files changed: 5524 ins; 2179 del; 170 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/142.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev pull/142/head:pull/142
PR: https://git.openjdk.org/jdk8u-dev/pull/142
More information about the jdk8u-dev
mailing list