On Sat, 17 Aug 2024 05:29:31 GMT, Jan Kratochvil <jkratoch...@openjdk.org> wrote:
>> The testcase requires root permissions. >> >> Fix by Severin Gehwolf. >> Testcase by Jan Kratochvil. > > Jan Kratochvil has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains 12 commits: > > - Functionality fixes after the refactorization > - Fix up compilation > - Merge remote-tracking branch 'origin/master' into master-cgroup > - Merge remote-tracking branch 'origin/master' into master-cgroup > - Testcase update upon review by Severin Gehwolf > - Testcase update upon review by Severin Gehwolf > - Inline adjust_controller() twice > - Revert "Unify 4 copies of adjust_controller()" > > This reverts commit 77a81d07d74c8ae9bf34bfd8df9bcaca451ede9a. > - Implement vm.cgroup.tools > - Use Metrics.systemMetrics().getProvider() > - ... and 2 more: https://git.openjdk.org/jdk/compare/07352c67...f06645f2 As we cannot find an agreement even on the comment in the testcase and this pull request will have soon an anniversary, proposing: - check-in the fix from a separate pull request as it is whole your fix anyway - I will just file this testcase into JBS, keep it downstream and close this pull request - or you can also check it in together with this testcase in any way, shape or form as long as there isn't stated my name ------------- PR Comment: https://git.openjdk.org/jdk/pull/17198#issuecomment-2295940427