[jdk8u-dev] RFR: 8230305: Cgroups v2: Container awareness [v5]
Jonathan Dowland
jdowland at openjdk.org
Mon Oct 31 09:44:03 UTC 2022
> This is a backport of 8230305 (Cgroups v2: Container awareness) for JDK8u, working from the jdk11u-dev backport as part of an effort to backport cgroups v2 to jdk8u-dev.
>
> The patch does not apply clean after unshuffling and path fixing:
>
> * mostly copyright line issues
> * different package name for jdk.test.lib.process.OutputAnalyzer
> * Most of hotspot/src/os/linux/vm/cgroupSubsystem_linux.cpp failed due to context changes, manually resolved (mostly deletions)
>
> A few other changes were made
>
> * Rework use of newer logging API (log_debug etc) to use guarded tty->print_cr
> * replace os::strerror with the libc strerror (which is thread unsafe, but that should be addressed by backporting 8148425 separately)
>
> patch touched hotspot/test/runtime/containers/docker/TestCPUAwareness.java which passes afterwards; I'm running further tests now (and awaiting GitHub CI doing the same)
Jonathan Dowland has updated the pull request incrementally with one additional commit since the last revision:
Remove C++11-style non-static member initialisation
This causes issues with pre-C++11 compilers (of which jdk8u-dev
ostensibly supports). Issue has materialised on ARM32.
I haven't introduced initialiser lists to the constructor because
all of these are assigned to in the body of the constructor.
-------------
Changes:
- all: https://git.openjdk.org/jdk8u-dev/pull/127/files
- new: https://git.openjdk.org/jdk8u-dev/pull/127/files/739788d2..e6fcc668
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=127&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=127&range=03-04
Stats: 4 lines in 1 file changed: 0 ins; 0 del; 4 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/127.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev pull/127/head:pull/127
PR: https://git.openjdk.org/jdk8u-dev/pull/127
More information about the jdk8u-dev
mailing list