RFR: 8226416: MonitorUsedDeflationThreshold can cause repeated async deflation requests

David Holmes david.holmes at oracle.com
Fri Jan 8 04:37:32 UTC 2021


Hi Dan,

On 8/01/2021 9:20 am, Daniel D.Daugherty wrote:
> On Thu, 7 Jan 2021 23:14:17 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
> 
>> Adding support for a diagnostic option called NoAsyncDeflationProgressMax
>> with a default value of 3. If we have three async monitor deflation cycles in a
>> row with zero monitors deflated, then we adjust the in_use_list_ceiling up.
>>
>> I've locally built and tested this fix on my MBP13 using the
>> MonitorUsedDeflationThresholdTest.java test that I wrote when this issue
>> first came up in June of 2019. I will be including this fix in my next Mach5
>> Tier[1-3] testing batch.
> 
> Note: I'm testing out stacking patches in Git so there are commits for
> https://bugs.openjdk.java.net/browse/JDK-8259397 and
> https://bugs.openjdk.java.net/browse/JDK-8259349 that show up in this PR.
> 
> This is the correct commit for this PR review: https://github.com/openjdk/jdk/commit/af5c3fec96d1d45af5881947d366e22cb8962dc3

I'm not quite sure what you are trying to test out here. All three 
commits are part of this PR and will be integrated when you integrate 
this PR. If you wanted this to be handled separately then they should be 
in distinct branches and PRs. If you want all three reviewed together 
that is fine too.

Cheers,
David

> -------------
> 
> PR: https://git.openjdk.java.net/jdk/pull/1993
> 


More information about the hotspot-runtime-dev mailing list