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

[Leo Korinth]

On 29/10/2019 09:29, Thomas Schatzl wrote:
> 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.

filed: https://bugs.openjdk.java.net/browse/JDK-8233109
> 
>>
>>> 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.

filed: https://bugs.openjdk.java.net/browse/JDK-8233110

>>
>>> 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

Thanks Kim and Thomas! I will add you as reviewers.

Is it starting to look okay for the rest of you?

Thanks,
Leo