RFR (S): 8151614: Improve logging in concurrent mark code

sangheon sangheon.kim at oracle.com
Thu Mar 10 18:32:40 UTC 2016


Hi Thomas,

Looks nice especially for combining logging and trace event.
Thanks for cleaning up this.

One minor comment, could you update the copyright in logPrefix.hpp 
before pushing?

Sangheon


On 03/10/2016 09:02 AM, Thomas Schatzl wrote:
> Hi all,
>
>    can I have reviews for this change that improves the logging code for
> concurrent mark by:
>
> - removes the need to have two scoped objects, one for logging, one for
> JFR for every phase
> - tries to make the phase names correspond to method names
> - add a "marking" tag to marking related log messages
> - removes a duplicate log message ("Restart for overflow") at different
> levels
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8151614
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8151614/webrev/
> Testing:
> jprt
>
>
> Thanks,
>    Thomas




More information about the hotspot-gc-dev mailing list