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

[Erik Österlund]

On Mon, 21 Feb 2022 15:11:30 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

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

Looks good.

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

Marked as reviewed by eosterlund (Reviewer).

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