[jdk21u-dev] RFR: 8272364: Parallel GC adaptive size policy may shrink the heap below MinHeapSize

[Goetz Lindenmaier]

On Thu, 13 Nov 2025 01:36:22 GMT, Daniel Hu <duke at openjdk.org> wrote:

>> Hi @cost0much 
>> Why do you merge this into one PR?  If it is clean, it would be much better to have three PRs.
>> You can do dependent PRs, i.e. for the second make a branch on top of the first, and in the PR 
>> use the git branch of the first RP to compare to.
>> 
>> Also, please give a reason why you want to backport [JDK-8272364](https://bugs.openjdk.org/browse/JDK-8272364)
>
> @GoeLin Ah, that's my bad. I assumed that given the subsequent test failures of [JDK-8272364](https://bugs.openjdk.org/browse/JDK-8272364), an atomic merge would be preferred. I can re-open with multiple PRs if that's better.
> 
> The justification for [JDK-8272364](https://bugs.openjdk.org/browse/JDK-8272364) is that the current behavior is incorrect, and the issue was significant enough to warrant a backport to 22. Moreover, the risk is comparatively low for hotspot backports, given the lack of bugtail (besides some failing tests). The only major risk I see is if 21 users are already reliant on the current incorrect behavior; if that's a main consideration, then this backport can be abandoned.

Hi @cost0much,
yes, the major concern of a backport is that it harms more use cases than it helps. This includes regressions, and as you argue these are unlikely here. But this also includes changes in behavior which is usually ignored in the risk assessment.

So is it possible that an application that runs with a smaller heap than configured goes OOM with your fix after updating?

And, for the positive side, do you have a specific use case where this fix is needed?

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/2419#issuecomment-3532635277