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

[harold seigel]

Thanks George!

Harold


On 7/12/2017 2:52 PM, George Triantafillou wrote:
> Hi Harold,
>
> Your changes look good.
>
> -George
>
> On 7/12/2017 2:19 PM, harold seigel wrote:
>> 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
>>
>