RFR: 8287741: Fix of JDK-8287107 (unused cgv1 freezer controller) was incomplete [v2]

Severin Gehwolf sgehwolf at openjdk.java.net
Tue Jun 7 12:28:13 UTC 2022


On Tue, 7 Jun 2022 08:49:58 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Please review this hotspot fix follow-up of JDK-8287107. The test which exercised this code was already added with  JDK-8287107. I've now added an assert ensuring that we don't set the cgroup path multiple times. The fix is essentially the same as on the JDK side, skip setting the cgroup path when the hierarchy is non-zero which is equivalent to the empty string match that was done in the Java code.
>> 
>> Testing:
>> - [x] `test/hotspot/jtreg/containers/cgroup/CgroupSubsystemFactory.java` with fastdebug on x86_64. Asserts before the code fix, passes after.
>> - [x] GHA
>
> Severin Gehwolf has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   8287741: Fix of JDK-8287107 (unused cgv1 freezer controller) was incomplete
>   
>   Add a comment to re-trigger GHA tests.

It doesn't look like tier1 failures in GHA on x86 (32 bit) Linux are related to this patch.

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

PR: https://git.openjdk.java.net/jdk/pull/9001


More information about the hotspot-runtime-dev mailing list