RFR: 8287741: Fix of JDK-8287107 (unused cgv1 freezer controller) was incomplete
Severin Gehwolf
sgehwolf at openjdk.java.net
Thu Jun 2 18:38:00 UTC 2022
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.
- [ ] GHA - still running.
-------------
Commit messages:
- 8287741: Fix of JDK-8287107 was incomplete
Changes: https://git.openjdk.java.net/jdk/pull/9001/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=9001&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8287741
Stats: 6 lines in 1 file changed: 6 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/9001.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/9001/head:pull/9001
PR: https://git.openjdk.java.net/jdk/pull/9001
More information about the hotspot-runtime-dev
mailing list