[jdk11u-dev] RFR: 8254001: [Metrics] Enhance parsing of cgroup interface files for version detection

[Andrew Azores]

On Tue, 12 Apr 2022 10:19:49 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Backport commit [a50725db2ab621e1a17cb5505f78e4bc73972a27](https://github.com/openjdk/jdk/commit/a50725db) for [JDK-8254001](https://bugs.openjdk.java.net/browse/JDK-8254001), enhancement to cgroups interface parsing and version detection.
>> 
>> The patch applied mostly cleanly with only two minor conflicts in `CgroupV1Subsystem.java`. These conflicts were simply hunks that failed to apply because the surrounding context did not match due to later changes in the internal implementation that are not present in JDK11. Since the patch being backported wholly replaces or removes these implementations anyway, my resolution was to simply remove the old implementation and accept the new implementation from the backport patch.
>
> src/java.base/linux/classes/jdk/internal/platform/CgroupSubsystemFactory.java line 197:
> 
>> 195:                 action = (tokens -> setCgroupV2Path(infos, tokens));
>> 196:             }
>> 197:             selfCgroupLines.map(line -> line.split(":"))
> 
> This will have to also include the fix from JDK-8272124 so that not regress. See: https://github.com/openjdk/jdk/commit/4d6593ce0243457e7431a5990957a8f880e0a3fb
> 
> It'll be covered by `TestCgroupSubsystemFactory` once [JDK-8284102](https://bugs.openjdk.java.net/browse/JDK-8284102), a 11u-only bug, gets fixed.

Okay, so rather than backporting the whole 8272124 I should just cherry-pick out that single fix to this file for inclusion in this PR?

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

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