RFR: 8140348: Convert TraceSafepoint to Unified Logging

[David Holmes]

On 24/10/2015 7:58 AM, Coleen Phillimore wrote:
>
> Copying Mattis,  Mattis, this is the second flag to be converted to use
> the Unified Logging framework.
>
> -XX:+TraceSafepoint was a develop mode flag but Unified Logging makes it
> available in product mode.  This may be useful to customers.  Your
> comments are appreciated, as well as comments from the openjdk community.

I'm concerned that a number of previously Develop-only trace flags are 
now being made product flags with the conversion to Unified-logging! Has 
anyone done the performance checks for that? Individually they may seem 
okay but what about in combination?

There has to be a balance between the perceived utility of a flag and 
the cost everyone has to pay whether they turn the flag on or not.

Thanks,
David

> Thanks,
> Coleen
>
>
> On 10/23/15 3:59 PM, Rachel Protacio wrote:
>> Thank you, Harold! I'll fix that.
>> Rachel
>>
>> On 10/23/2015 9:09 AM, harold seigel wrote:
>>> Hi Rachel,
>>>
>>> The change s look good.
>>>
>>> One nit, if SafepointTest.java is a new file then you can remove
>>> "2014,  " for its copyright.
>>>
>>> Thanks, Harold
>>>
>>> On 10/22/2015 1:33 PM, Rachel Protacio wrote:
>>>> Hello,
>>>>
>>>> Please review my next logging enhancement.
>>>>
>>>> Summary: The former -XX:+TraceSafepoint flag is updated to the
>>>> unified logging framework and is now replaced with -Xlog:safepoint
>>>> in product mode.
>>>>
>>>> open webrev: http://cr.openjdk.java.net/~rprotacio/8140348/
>>>> bug link: https://bugs.openjdk.java.net/browse/JDK-8140348
>>>> testing: Passes JPRT, RBT, and RefWorkload performance testing.
>>>>
>>>> A compatibility request has been accepted with regard to this change.
>>>>
>>>> Thank you,
>>>> Rachel
>>>
>>
>