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