RFR(m): 8150015: Integrate TraceTime with Unified Logging more seamlessly

David Holmes david.holmes at oracle.com
Wed Mar 16 05:14:20 UTC 2016


Hi Robbin,

Thanks for tackling this. :)

On 16/03/2016 6:31 AM, Robbin Ehn wrote:
> Hi, please review this enhancement.
>
> This adds support for multiple UL tags in TraceTime.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8150015/
> Webrev: http://cr.openjdk.java.net/~rehn/8150015/v1/webrev/

Initial query: why do so many .cpp files now include the timerTrace.hpp 
file yet have no other changes?

I'm still mulling over the verbosity of the new format. You may recall 
there was a lot of discussion about this last time round :)

Thanks,
David

> Tested with jprt hotspot and manually tested log output.
>
> Thanks!
>
> /Robbin


More information about the serviceability-dev mailing list