[jdk11u-dev] RFR: 8239785: Cgroups: Incorrect detection logic on old systems in hotspot [v2]
Jonathan Dowland
jdowland at openjdk.java.net
Wed Mar 23 13:34:29 UTC 2022
> This is a backport of 8239785 for jdk11u. It depends upon
> pr/865: 8253714 (hopefully the bots will figure this out automatically)
>
> It's not clean, but the only conflict was the copyright line in the
> comment header for src/hotspot/share/prims/whitebox.hpp.
>
> I tried these tests, and they pass:
>
> test/hotspot/jtreg/applications/ctw/modules/java_compiler.java
> test/hotspot/jtreg/compiler/c2/Test8004741.java
> test/hotspot/jtreg/containers/cgroup/CgroupSubsystemFactory.java
>
> Awaiting tier tests via GA.
>
> ---------
> ### Progress
> - [x] Change must not contain extraneous whitespace
> - [x] Commit message must refer to an issue
> - [ ] Change must be properly reviewed
>
> ### Integration blocker
> ⚠️ Dependency #865 must be integrated first
>
>
>
> ### Reviewing
> <details><summary>Using <code>git</code></summary>
>
> Checkout this PR locally: \
> `$ git fetch https://git.openjdk.java.net/jdk11u-dev pull/881/head:pull/881` \
> `$ git checkout pull/881`
>
> Update a local copy of the PR: \
> `$ git checkout pull/881` \
> `$ git pull https://git.openjdk.java.net/jdk11u-dev pull/881/head`
>
> </details>
> <details><summary>Using Skara CLI tools</summary>
>
> Checkout this PR locally: \
> `$ git pr checkout 881`
>
> View PR using the GUI difftool: \
> `$ git pr show -t 881`
>
> </details>
> <details><summary>Using diff file</summary>
>
> Download this PR as a diff file: \
> <a href="https://git.openjdk.java.net/jdk11u-dev/pull/881.diff">https://git.openjdk.java.net/jdk11u-dev/pull/881.diff</a>
>
> </details>
Jonathan Dowland has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase.
-------------
Changes:
- all: https://git.openjdk.java.net/jdk11u-dev/pull/881/files
- new: https://git.openjdk.java.net/jdk11u-dev/pull/881/files/43de2010..43de2010
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk11u-dev&pr=881&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk11u-dev&pr=881&range=00-01
Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jdk11u-dev/pull/881.diff
Fetch: git fetch https://git.openjdk.java.net/jdk11u-dev pull/881/head:pull/881
PR: https://git.openjdk.java.net/jdk11u-dev/pull/881
More information about the jdk-updates-dev
mailing list