[jdk8u-dev] RFR: 8239785: Cgroups: Incorrect detection logic on old systems in hotspot [v5]
Jonathan Dowland
jdowland at openjdk.org
Fri Dec 9 15:36:56 UTC 2022
On Fri, 9 Dec 2022 11:02:29 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
>>> Did you run the newly added test hotspot/test/runtime/containers/cgroup/CgroupSubsystemFactory.java?
>>
>> I guess not. I began the patch series running all the touched tests, but since review wasn't going to start for a while, and also some tests are known-broken until later patches are applied, I decided to focus on the non-test hunks and watch the status of test pass/fail via GHA on the "tip" of the backport effort.
>>
>> Now that review has started I should switch to paying attention to them!
>>
>>> It didn't work for me and required a patch like this to actually work:
>>
>> Thanks for fixing this one. I'll look at the rest downstream of this PR.
>
> @jmtd Please add approval requests to the bug.
@jerboaa
> Please add approval requests to the bug.
Done — also another rebase as GitHub was showing the previous PR commit again. I'm going to try and get ahead of this and pre-emptively rebase the rest of the chain ahead of reviews. Shall I do all the fix-request labels at this point too? It looks like we've gone past the initial set of patches under consideration.
Thanks
-------------
PR: https://git.openjdk.org/jdk8u-dev/pull/135
More information about the jdk8u-dev
mailing list