[9] RFR(L): 8062854: move compiler jtreg test to corresponding subfolders and use those in TEST.groups

[Zoltán Majó]

Thank you, Vladimir and David, for the review!

Best regards,


Zoltan

On 11/14/2014 05:35 PM, Vladimir Kozlov wrote:
> Looks good.
>
> Thanks,
> Vladimir
>
> On 11/13/14 11:44 PM, Zoltán Majó wrote:
>> Hi Vladimir,
>>
>>
>> thank you for the feedback.
>>
>> On 11/13/2014 05:52 PM, Vladimir Kozlov wrote:
>>> On 11/13/14 8:45 AM, Zoltán Majó wrote:
>>>> Hi,
>>>>
>>>>
>>>> On 11/13/2014 05:38 PM, Vladimir Kozlov wrote:
>>>>> This is good approach. I see you removed all 5091921 tests when 
>>>>> before you ran most of them. What is the reason? Can
>>>>> you still run some?
>>>>
>>>> I removed 5091921 test because
>>>>
>>>> - 3 of them have a high execution time and
>>>> - I wanted to keep the exclude list simple (not exclude many 
>>>> individual tests).
>>>>
>>>> But if you like, we can include short-running 5091921 tests again 
>>>> (for example into hotspot_compiler_1). That would
>>>> result in around 1 min extra time and 3 tests excluded.
>>>
>>> Yes, please do that.
>>
>> I added the short-running 5091921 tests to hotspot_compiler_1. Here 
>> are the results (on solaris_sparcv9 cpus=6
>> parallelcount=6 cpufreqmhz=2848):
>>
>> before: 107 tests / work=15m
>> after: 121 tests / work=16m14s
>>
>> Here is the updated webrev: 
>> http://cr.openjdk.java.net/~zmajo/8062854/webrev.02/
>>
>> Thank you and best regards,
>>
>>
>> Zoltan
>>
>>>
>>> Thanks,
>>> Vladimir
>>>
>>>>
>>>> Thank you and best regards,
>>>>
>>>>
>>>> Zoltan
>>>>
>>>>
>>>>> Thanks,
>>>>> Vladimir
>>>>>
>>>>> On 11/13/14 8:29 AM, Zoltán Majó wrote:
>>>>>> Hi,
>>>>>>
>>>>>>
>>>>>> On 11/13/2014 03:12 PM, Zoltán Majó wrote:
>>>>>>> Here is only the "work" time for the same runs as before:
>>>>>>>
>>>>>>> hotspot_compiler_1 / work=15m6s
>>>>>>> hotspot_compiler_2 / work=13m55s
>>>>>>> hotspot_compiler_3 / work=12m43s
>>>>>>
>>>>>> I updated the test list. Here are the results:
>>>>>>
>>>>>> group / #tests / "work" time on solaris_sparcv9 cpus=6 
>>>>>> parallelcount=6 cpufreqmhz=2848
>>>>>>
>>>>>> hotspot_compiler_1 / 107 / work=15m
>>>>>> hotspot_compiler_2 / 79 / work=15m2s
>>>>>> hotspot_compiler_3 / 62 / work=14m14s
>>>>>>
>>>>>> Here is the updated webrev: 
>>>>>> http://cr.openjdk.java.net/~zmajo/8062854/webrev.01/
>>>>>>
>>>>>> Thank you and best regards,
>>>>>>
>>>>>>
>>>>>> Zoltan
>>>>>>
>>>>
>>