RFR (XXS): 8008684 - CMS: concurrent phase start markers should always be printed

Mikael Gerdin mikael.gerdin at oracle.com
Tue Mar 12 07:17:54 PDT 2013


Thomas,

On 2013-03-12 14:39, Thomas Schatzl wrote:
> Hi,
>
> On Tue, 2013-03-12 at 13:45 +0100, Mikael Gerdin wrote:
>> Thomas,
>>
>> On 2013-03-07 07:56, Thomas Schatzl wrote:
>>> Hi,
>>>
>>>     new webrev at
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~tschatzl/8008684/webrev.1/
>>>
>>> Passes jprt.
>>
>> Looks good to me.
>> It's good that this gets cleaned up.
>>
>> /Mikael
>
> Thanks. Could you (or jon) also shepherd this change into the
> repository? :)

I'll take care of it.

/Mikael

>
> Here's an example openjdk commit text for this change (please adjust if
> needed):
>
> 8008684: CMS: concurrent phase start markers should always be printed
> Summary: Print the concurrent phase start markers for CMS when
> PrintGCDetails is enabled, not only if both PrintGCDetails and
> PrintGCTimeStamps are.
> Reviewed-by: mgerdin, jmasa
> Contributed-by: tschatzl
>
>
> Thanks,
> Thomas
>
>


More information about the hotspot-gc-dev mailing list