RFR (XXS): 8072688: Description of flag ExplicitGCInvokesConcurrent should mantion G1 as well [Was: Re: comments in globals.hpp]

Mikael Gerdin mikael.gerdin at oracle.com
Mon Feb 9 15:08:23 UTC 2015


On 2015-02-09 16:01, Thomas Schatzl wrote:
> Hi Jesper,
>
> On Fri, 2015-02-06 at 15:30 +0100, Jesper Wilhelmsson wrote:
>> Hi Kirk,
>>
>> Thanks for reporting this!
>> I filed a bug:
>> https://bugs.openjdk.java.net/browse/JDK-8072688
>>
>> and have a fix out for review:
>> http://cr.openjdk.java.net/~jwilhelm/8072688/webrev.00/
>>
>
>    looks good - not sure if the explicit mention of the two collectors is
> necessary. We do not do it for many other flags either.

I agree that it would be better to not mention any specific GC:
"effective only when a concurrent GC is enabled"
is my suggestion.

/Mikael

>
> Maybe if you waited a little more if somebody else feels strongly about
> this.
>
> You do not need to hide the review request too :)
>
> Thanks,
>    Thomas
>
>


More information about the hotspot-gc-dev mailing list