[9] RFR (S): 8150349: Reduce the execution time of the hotspot_compiler_3 group

[Nils Eliasson]

Hi Zoltan,

On 2016-02-26 13:29, Zoltán Majó wrote:
> Hi Vladimir,
>
>
> On 02/25/2016 06:21 PM, Vladimir Kozlov wrote:
>> I am fine with removal of adler32 and bmi testing from JPRT runs 
>> because we don't have a plan to touch them.
>> But code related to CRC32, CRC32c and string may be touched soon for 
>> some platforms.
>
> OK, so I've re-enabled testing of the CRC23, CRC32c and string 
> intrinsics (still within the hotspot_compiler_3 group). Running the 
> group still takes < 30 minutes on our slowest platforms, here are the 
> execution times:
>
> West:
>  linux_arm64_2.6-fastdebug-c2-hotspot_compiler_3     success(25m 12s)
>  solaris_sparcv9_5.11-fastdebug-c2-hotspot_compiler_3 success(11m 46s)
>
> Stockholm:
>   linux_arm64_2.6-fastdebug-c2-hotspot_compiler_3     success(24m 18s)
>   solaris_sparcv9_5.11-fastdebug-c2-hotspot_compiler_3 success(21m 01s)
>
> Here is the updated webrev:
> http://cr.openjdk.java.net/~zmajo/8150349/webrev.01/
>
>> What if we created hotspot_compiler_4 group for them? How much time 
>> it would take to run it on slow platforms?
>
> I think it's fine to leave all tests in hotspot_compiler_3 as all 
> execution times are still below 30 minutes.

Removing these test solves this issue in the JPRT run, but they will 
still run in nightlies. We should still file bugs for making these tests 
faster.

Regards,
Nils

>
> I filed INTJDK-7622854: "Track the execution time of JPRT tests 
> groups" so that we can better track the execution times of the 
> hotspot_compiler_[1-3] test groups.
>
> Thank you!
>
> Best regards,
>
>
> Zoltan
>
>>
>> Thanks,
>> Vladimir
>>
>> On 2/25/16 8:12 AM, Zoltán Majó wrote:
>>> Hi Nils,
>>>
>>>
>>> thank you for the feedback!
>>>
>>> On 02/25/2016 04:22 PM, Nils Eliasson wrote:
>>>> Hi Zoltan,
>>>>
>>>> Have you opened a bug for fixing the tests?
>>>
>>> No, I have not, because there is an existing issue (INTJDK-7620326) 
>>> that is supposed to take care of the high execution
>>> time of intrinsic-related tests on platforms where the relevant 
>>> intrinsics are not available. I've linked the two issues.
>>>
>>>>
>>>> Do we really want to remove the String intrinsics from the list? 
>>>> aren't they pretty new?
>>>
>>> You are right, they are in part new (in part old intrinsics have 
>>> been updated). I thought to remove them nevertheless,
>>> because they take a long time to execute and are anyway executed in 
>>> our nightlies.
>>>
>>> Best regards,
>>>
>>>
>>> Zoltan
>>>
>>>>
>>>> Thanks for fixing this,
>>>> Nils Eliasson
>>>>
>>>> On 2016-02-25 15:17, Zoltán Majó wrote:
>>>>> Hi,
>>>>>
>>>>>
>>>>> please review the patch for 8150349.
>>>>>
>>>>> https://bugs.openjdk.java.net/browse/JDK-8150349
>>>>>
>>>>> Problem: The hotspot_compiler_3 group of tests takes more than 30 
>>>>> minutes on our slowest platforms.
>>>>>
>>>>>
>>>>> Solution: Remove a number of long-running tests from the 
>>>>> hotspot_compiler_3 group. All removed tests are related to
>>>>> intrinsics, that is, they check the functionality of a reduced set 
>>>>> of the VM's functionality that is not often
>>>>> modified. Also, we execute all removed tests daily (in our nightly 
>>>>> testing).
>>>>>
>>>>> Here is the time spent in hotspot_compiler_3 on our slowest 
>>>>> platforms after removing the tests:
>>>>>
>>>>> West:
>>>>> solaris_sparcv9_5.11-fastdebug-c2-hotspot_compiler_3 success(15m 22s)
>>>>>     platform=solaris_sparc_5.11 osname=solaris osarch=sparc cpus=6 
>>>>> parallelcount=6 ram=31744MB cpufreqmhz=2848
>>>>> linux_arm64_2.6-fastdebug-c2-hotspot_compiler_3 success(19m 09s)
>>>>>     platform=linux_x64_3.8 osname=linux osarch=x64 cpus=12 
>>>>> parallelcount=12 ram=60690MB cpufreqmhz=2294
>>>>>
>>>>> Stockholm:
>>>>> solaris_sparcv9_5.11-fastdebug-c2-hotspot_compiler_3 success(15m 32s)
>>>>>     platform=solaris_sparc_5.11 osname=solaris osarch=sparc 
>>>>> cpus=12 parallelcount=12 ram=24576MB cpufreqmhz=2848
>>>>> linux_arm64_2.6-fastdebug-c2-hotspot_compiler_3 success(20m 24s)
>>>>>     platform=linux_arm64_3.13 osname=linux osarch=arm64 cpus=8 
>>>>> parallelcount=8 ram=16061MB
>>>>>
>>>>>
>>>>> Webrev:
>>>>> http://cr.openjdk.java.net/~zmajo/8150349/webrev.00/
>>>>>
>>>>> Thank you!
>>>>>
>>>>> Best regards,
>>>>>
>>>>>
>>>>> Zoltan
>>>>>
>>>>
>>>
>