RFR: 8226416: MonitorUsedDeflationThreshold can cause repeated async deflation requests [v3]

Daniel D.Daugherty dcubed at openjdk.java.net
Tue Jan 12 20:04:11 UTC 2021


> 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.

Daniel D. Daugherty has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 17 commits:

 - Merge branch 'master' into JDK-8226416
 - change NoAsyncDeflationProgressMax from 'int' to 'uintx' and _no_progress_cnt from 'intx' to 'uintx'.
 - Merge branch 'JDK-8259349' into JDK-8226416
 - change AvgMonitorsPerThreadEstimate from int to size_t and _in_use_list_ceiling from jint to size_t.
 - Merge branch 'JDK-8259349' into JDK-8226416
 - dholmes CR - delete a stale comment.
 - Merge branch 'JDK-8259349' into JDK-8226416
 - delete bad assert found by linux build.
 - change NoAsyncDeflationProgressMax from 'intx' to 'int'; clarify logic in monitors_used_above_threshold(); add MonitorUsedDeflationThresholdTest.java.
 - Merge branch 'JDK-8259349' into JDK-8226416
 - ... and 7 more: https://git.openjdk.java.net/jdk/compare/c338f116...2efe7b4b

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

Changes: https://git.openjdk.java.net/jdk/pull/1993/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1993&range=02
  Stats: 171 lines in 3 files changed: 164 ins; 0 del; 7 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1993.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1993/head:pull/1993

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


More information about the hotspot-runtime-dev mailing list