RFR 8184281: -Xlog:startuptime does not include call to initPhase2

[harold seigel]

Hi,

Please review this updated webrev that has the changed message:

http://cr.openjdk.java.net/~hseigel/bug_8184281.2/webrev/index.html

Thanks, Harold


On 7/12/2017 11:12 AM, Alan Bateman wrote:
> On 12/07/2017 15:15, harold seigel wrote:
>> Hi,
>>
>> Please review this small JDK-10 fix for JDK-8184281.  The fix drops 
>> the need to specify "+module" in order to get Phase2 initialization 
>> timer information.  It will now be included when just 
>> -Xlog:startuptime is specified.
>>
>> Open Webrev: 
>> http://cr.openjdk.java.net/~hseigel/bug_8184281/webrev/index.html
>>
>> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8184281
> This looks good, I just wonder if we should change the message while 
> in the area. When looking at -Xlog:startuptime output then I think it 
> would be more useful to see "Initialize module system" for example.
>
> At some point we might need a TraceTime in call_initPhase3 too but 
> it's mostly close to nothing so not important now.
>
> -Alan