RFR (XS) 8152957: Improve specificity of safepoint logging to print safepoint typ

David Holmes david.holmes at oracle.com
Tue Dec 19 22:05:23 UTC 2017


Hi Coleen,

On 20/12/2017 6:12 AM, coleen.phillimore at oracle.com wrote:
> Summary: upgrade safepoint begin logs to Info logging, which has the 
> reason.

410   log_info(safepoint)("Entering safepoint region: %s", 
VMThread::vm_safepoint_description());
497/504       log_debug(safepoint)("Leaving safepoint region");

You need to change the "leaving" part to info as well.

Though I'm not clear how this simple change relates to the bug synopsis 
and the request to "print safepoint type" ??

Thanks,
David

> See bug for sample.
> 
> Tested with existing jtreg tests to make sure new logging line doesn't 
> break anything.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/8152957.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8152957
> 
> Thanks,
> Coleen


More information about the hotspot-runtime-dev mailing list