RFR(S) 8007270: Make IsMethodCompilable test work with tiered

Nils Eliasson nils.eliasson at oracle.com
Tue Feb 11 05:15:53 PST 2014


Hi,

Thanks for your contribution Igor.

After some additional testing I noticed some inconsistencies and filed 
https://bugs.openjdk.java.net/browse/JDK-8034188

This test now runs faster, and targets c2-compiles only, in server and 
with tiered.

Webrev: http://cr.openjdk.java.net/~neliasso/8007270/webrev.02/

Thanks,
//Nils


On 2014-01-29 11:11, Igor Ignatyev wrote:
> Hi Nils,
>
> you can skip running on client by yourself, see attached diff.
> Igor
>
> On 01/28/2014 07:02 PM, Nils Eliasson wrote:
>> Hi all,
>>
>> I need a review for this change.
>>
>> http://cr.openjdk.java.net/~neliasso/8007270/webrev.01/
>>
>> This test was disabled since it didn't work very well with tiered (or
>> client). It tests the PerMethodRecompilationCutoff that was introduced
>> to disable c2-compilations of a method when it has been deoptimized too
>> many times.  The bug report suggested we should disable c1 compilations
>> as well but I don't think that was the intent of the cutoff feature.
>>
>> I have changed the following in the test
>> * skip test when running client only (not supported by jtreg at the 
>> moment)
>> * check what compilation level was used when compiling so that it can
>> keep track of the number of c2 compiles (and deopts) correctly in tiered
>> mode
>> * compile and deopt up to the cutoff limit only once
>> * added PerMethodRecompilationCutoff=4 flag to commandline to reduce
>> wasted time in test (default 400)
>>
>> Now the test works and the running time has been reduced to seconds
>> instead of minutes.
>>
>> Kind regards,
>> Nils Eliasson
>>
>>



More information about the hotspot-compiler-dev mailing list