RFR: 8232365: Implementation for JEP 363: Remove the Concurrent Mark Sweep (CMS) Garbage Collector

[Thomas Schatzl]

Hi,

On 28.10.19 21:34, Kim Barrett wrote:
>> On Oct 28, 2019, at 12:30 PM, Leo Korinth <leo.korinth at oracle.com> wrote:
>>
>> On 24/10/2019 23:16, Thomas Schatzl wrote:
>>> Other minor removals that could be done here:
>>> - GCStats::kind() / GCStats::Name enum / GCStats::average_promoted_in_bytes() / GCStats::padded_average_promoted_in_bytes() are unused.
>>
>> Thanks, removed.
> 
> With these removals, GCStats seems pretty vacuous.  Maybe another
> followup to look at further cleanup there?

+1 for follow-up.

> 
>> Also unrelated to this CMS removal, should I file a bug for obsolete/removal of PrintGC/PrintGCDetails (deprecated now), or do we want to keep it deprecated?
> 
> Don't forget -Xloggc. I don't have an informed opinion as to whether
> it's time to remove them. One would want some idea of how prevelent
> these formerly very widely used options still are. (For reference, the
> change that added them back as deprecated options (after having been
> removed by the conversion of GC to use UL) was JDK-8145180. See also
> JDK-8170636.)  I guess it doesn’t hurt to have the RFE, even if we
> decide not to act on it immediately.
> 
>> Fixes after suggestions from Kim and Thomas:
>>
>> http://cr.openjdk.java.net/~lkorinth/8232365/01_02/ (incremental)
>> http://cr.openjdk.java.net/~lkorinth/8232365/02/ (full)
> 
> Looks good.
> 

Looks good.

Thanks,
   Thomas