RFR(S): 8202169: Reduce ctw_2 duration by parallelizing CtwRunner invocations

Mikael Vidstedt mikael.vidstedt at oracle.com
Wed Apr 25 01:54:25 UTC 2018


I decided to add some additional argument checking, please review:

Webrev: http://cr.openjdk.java.net/~mikael/webrevs/8202169/webrev.01/open/webrev/ <http://cr.openjdk.java.net/~mikael/webrevs/8202169/webrev.01/open/webrev/>
Incremental webrev: http://cr.openjdk.java.net/~mikael/webrevs/8202169/webrev.01.incr/open/webrev/ <http://cr.openjdk.java.net/~mikael/webrevs/8202169/webrev.01.incr/open/webrev/>

Cheers,
Mikael

> On Apr 24, 2018, at 4:45 PM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> 
> Hi Mikael,
> 
> looks good to me.
> 
> -- Igor
> 
>> On Apr 23, 2018, at 7:07 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>> 
>> Looks good.
>> 
>> Ask Igor Ignatyev to look as author of code.
>> 
>> Thanks,
>> Vladimir
>> 
>> On 4/23/18 6:58 PM, Mikael Vidstedt wrote:
>>> Please review this change which parallelizes the CompileTheWorld/CtwRunner invocations to reduce the (wall clock) duration of running the ctw_2 jtreg test group.
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8202169
>>> webrev: http://cr.openjdk.java.net/~mikael/webrevs/8202169/webrev.00/open/webrev/
>>> The CompileTheWorld class already has support for specifying the start and end class (index), so this change just enables passing the relevant arguments in to the CtwRunner wrapper class. For flexibility I made it possible both to specify the start/stop values both as absolute index, or as percentage values. This change only uses the latter format.
>>> Cheers,
>>> Mikael
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180424/99cf4a1e/attachment.html>


More information about the hotspot-compiler-dev mailing list