[9-dev] Request for review: JDK-8146128: compiler/cpuflags/TestAESIntrinsicsOnSupportedConfig timeouts

[Alexander Vorobyev]

This test uses test/compiler/codegen/aes/TestAESMain.java. It runs it 
with different AES-related flags and then analyzes output.  But 
TestAESMain.java performs very intensive calculations and it takes more 
time when axelerations are disabled (testNoUseAES(), 
testNoUseAESIntrinsic()). It is the cause of timeout.

On 01.09.2016 21:13, Leonid Mesnik wrote:
> Hi
>
> The hotspot compiler changes should go to jdk9/hs-comp and not to 
> 9-dev. Also hotspot-compiler-dev at openjdk.java.net alias should be used 
> for compiler specific product and test changes.
>
> It is unclear from issue description/comment what is the  root cause 
> of failure and how it was fixed. Could you please add this information.
>
> Leonid
>
> On 01.09.2016 20:58, Alexander Vorobyev wrote:
>>
>> Hi All,
>>
>> I'd like review for JDK-8146128 
>> (https://bugs.openjdk.java.net/browse/JDK-8146128)
>>
>> Test passes with timeout increased. Looks like it times out in 
>> sub-tests where AESIntrinsics are disabled (testNoUseAES(), 
>> testNoUseAESIntrinsic()). The easiest way to fix this test is to 
>> increase timeout.
>>
>> Run parameter was added:
>> @run main/othervm/timeout=300
>>
>>
>> Here is webrev:
>> http://cr.openjdk.java.net/~avorobye/8146128/webrev.00/
>>
>>
>> Thanks,
>> Alexander
>>
>>
>>
>