RFR: 8271008: appcds/*/MethodHandlesAsCollectorTest.java tests time out because of excessive GC (CodeCache GC Threshold) in loom [v2]

Coleen Phillimore coleenp at openjdk.java.net
Mon Feb 21 15:11:30 UTC 2022


> In Loom there's a full heap walk when the sweeper is triggered.  Many of the triggers in this test case are for the adapters created by the test, which are not deallocated.  Since there is a fall back to other code cache heap areas for NonNMethod and for NMethodProfiled, made the function CodeCache::reverse_free_ratio() examine the total code cache available rather than the specific area that it is allocating into.  The compilation policy also uses this to increase the C1 compile threshold so also uses the entire free code cache size to calculate new threshold (ask @TobiHartmann about this).  Thanks to Tobias for the discussion for this fix.
> Tested with tier1-4.

Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:

  Fixed comment

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7514/files
  - new: https://git.openjdk.java.net/jdk/pull/7514/files/7a69dc43..97b7a59c

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7514&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7514&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7514.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7514/head:pull/7514

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


More information about the hotspot-dev mailing list