[jdk8u-dev] RFR: 8253714: [cgroups v2] Soft memory limit incorrectly using memory.high [v4]

Jonathan Dowland jdowland at openjdk.org
Mon Dec 5 04:20:46 UTC 2022


On Mon, 5 Dec 2022 04:01:34 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:

>> Jonathan Dowland has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains one additional commit since the last revision:
>> 
>>   8253714: [cgroups v2] Soft memory limit incorrectly using memory.high
>>   
>>   The early implementation of cgroups v2 support was done with
>>   crun 0.8 and it contained a bug which set memory.high over
>>   memory.low when --memory-reservation was being used as a CLI
>>   option.
>>   
>>   This bug has been fixed in later crun versions, starting with
>>   crun 0.11. Use memory.low in OpenJDK as well.
>>   
>>   Backport-of: ff6843ca4842498791061f924c545fa9469cc1dc
>
> Confirmed clean and approved.
> 
> Not sure why it needed 23 commits in this case...  :/

@gnu-andrew  the 23 commits are likely a result of the PR branch and the target branch diverging too much so copies of already merged stuff is showing up. I’ll have to do a manual rebase of the PR before it can be integrated.

-------------

PR: https://git.openjdk.org/jdk8u-dev/pull/130


More information about the jdk8u-dev mailing list