[jdk11u-dev] RFR: 8239785: Cgroups: Incorrect detection logic on old systems in hotspot

[Jonathan Dowland]

On Fri, 25 Mar 2022 10:55:05 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> This is a backport of 8239785 for jdk11u. The dependent backports have now all been integrated.
>> 
>> This PR replaces #881 which was stuck with a dependency on a superseded dependent PR.
>> 
>> It's not clean, but the only conflict was the copyright line in the
>> comment header for src/hotspot/share/prims/whitebox.hpp.
>> 
>> I'm checking jtreg tests in test/hotspot/jtreg/containers on a mixture of cgroup v1 and v2 systems now.
>> 
>> ---------
>> ### Progress
>> - [x] Change must not contain extraneous whitespace
>> - [x] Commit message must refer to an issue
>> - [ ] Change must be properly reviewed
>> 
>> ### Error
>>  ⚠️ The pull request body must not be empty.
>> 
>> 
>> 
>> ### Reviewing
>> <details><summary>Using <code>git</code></summary>
>> 
>> Checkout this PR locally: \
>> `$ git fetch https://git.openjdk.java.net/jdk11u-dev pull/946/head:pull/946` \
>> `$ git checkout pull/946`
>> 
>> Update a local copy of the PR: \
>> `$ git checkout pull/946` \
>> `$ git pull https://git.openjdk.java.net/jdk11u-dev pull/946/head`
>> 
>> </details>
>> <details><summary>Using Skara CLI tools</summary>
>> 
>> Checkout this PR locally: \
>> `$ git pr checkout 946`
>> 
>> View PR using the GUI difftool: \
>> `$ git pr show -t 946`
>> 
>> </details>
>> <details><summary>Using diff file</summary>
>> 
>> Download this PR as a diff file: \
>> <a href="https://git.openjdk.java.net/jdk11u-dev/pull/946.diff">https://git.openjdk.java.net/jdk11u-dev/pull/946.diff</a>
>> 
>> </details>
>
> test/lib/sun/hotspot/WhiteBox.java line 2:
> 
>> 1: /*
>> 2:  * Copyright (c) 2012, 2020, Oracle and/or its affiliates. All rights reserved.
> 
> This change isn't in the original patch. Please remove.

Odd sorry, I picked the wrong whitebox file!

-------------

PR: https://git.openjdk.java.net/jdk11u-dev/pull/946