RFR: 8284900: Check InitialHeapSize and container memory limits before startup [v6]
Ioi Lam
iklam at openjdk.java.net
Wed Apr 20 06:53:26 UTC 2022
On Wed, 20 Apr 2022 02:17:19 GMT, xpbob <duke at openjdk.java.net> wrote:
>> Thanks for the update. I'll let those that understand the OSContainer APIs review if this is the correct usage of it.
>>
>> Small nit: Local variables in GC code tend to be written with underscores and not camelCase. Maybe just rename `memswBytes` to `limit`. A short name seems fine for such a small scope.
>
> @stefank
> Thanks for review.
> The code has been updated
I have updated the bug report [JDK-8284900](https://bugs.openjdk.java.net/browse/JDK-8284900), but there are still some details that are missing. I.e., how to set up the `foo` control group. @xpbob could you add the details into the bug report?
The bug report should have a concrete reproducible test case so anyone reading it in the future won't need to guess the reproduction steps.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8256
More information about the hotspot-gc-dev
mailing list