[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