RFR: 8179013: Deprecate the Concurrent Mark Sweep (CMS) Garbage Collector

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR: 8179013: Deprecate the Concurrent Mark Sweep (CMS) Garbage Collector

Erik Helin-2
Hi all,

since JEP 291 [0] has been targeted for JDK 9 [1], here is the patch
that implements the change. The patch itself is very small, it just adds
a "-XX:+UseConcMarkSweepGC" entry to the special_jvm_flags array and
updates a test. The test checks that a deprecation message is correctly
printed for the flags:
-XX:+UseConcMarkSweepGC
-Xconcgc
-Xnocongc

Webrev:
http://cr.openjdk.java.net/~ehelin/8179013/00/

Issue:
https://bugs.openjdk.java.net/browse/JDK-8179013

Testing:
- JPRT
- make run-test TEST=hotspot/test/gc/startup_warnings/TestCMS.java
- make run-test TEST=hotspot_jprt
(tested with both open and commercial builds)

Thanks,
Erik

[0]: http://openjdk.java.net/jeps/291
[1]: http://mail.openjdk.java.net/pipermail/jdk9-dev/2017-April/005766.html
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: 8179013: Deprecate the Concurrent Mark Sweep (CMS) Garbage Collector

Stefan Johansson
Hi Erik,

On 2017-04-20 17:55, Erik Helin wrote:

> Hi all,
>
> since JEP 291 [0] has been targeted for JDK 9 [1], here is the patch
> that implements the change. The patch itself is very small, it just
> adds a "-XX:+UseConcMarkSweepGC" entry to the special_jvm_flags array
> and updates a test. The test checks that a deprecation message is
> correctly printed for the flags:
> -XX:+UseConcMarkSweepGC
> -Xconcgc
> -Xnocongc
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8179013/00/
>
Looks good,
StefanJ

> Issue:
> https://bugs.openjdk.java.net/browse/JDK-8179013
>
> Testing:
> - JPRT
> - make run-test TEST=hotspot/test/gc/startup_warnings/TestCMS.java
> - make run-test TEST=hotspot_jprt
> (tested with both open and commercial builds)
>
> Thanks,
> Erik
>
> [0]: http://openjdk.java.net/jeps/291
> [1]:
> http://mail.openjdk.java.net/pipermail/jdk9-dev/2017-April/005766.html

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: 8179013: Deprecate the Concurrent Mark Sweep (CMS) Garbage Collector

Erik Helin-2
On 04/21/2017 11:28 AM, Stefan Johansson wrote:

> Hi Erik,
>
> On 2017-04-20 17:55, Erik Helin wrote:
>> Hi all,
>>
>> since JEP 291 [0] has been targeted for JDK 9 [1], here is the patch
>> that implements the change. The patch itself is very small, it just
>> adds a "-XX:+UseConcMarkSweepGC" entry to the special_jvm_flags array
>> and updates a test. The test checks that a deprecation message is
>> correctly printed for the flags:
>> -XX:+UseConcMarkSweepGC
>> -Xconcgc
>> -Xnocongc
>>
>> Webrev:
>> http://cr.openjdk.java.net/~ehelin/8179013/00/
>>
> Looks good,

Thanks Stefan for reviewing!
Erik

> StefanJ
>> Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8179013
>>
>> Testing:
>> - JPRT
>> - make run-test TEST=hotspot/test/gc/startup_warnings/TestCMS.java
>> - make run-test TEST=hotspot_jprt
>> (tested with both open and commercial builds)
>>
>> Thanks,
>> Erik
>>
>> [0]: http://openjdk.java.net/jeps/291
>> [1]:
>> http://mail.openjdk.java.net/pipermail/jdk9-dev/2017-April/005766.html
>
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: 8179013: Deprecate the Concurrent Mark Sweep (CMS) Garbage Collector

sangheon.kim@oracle.com
In reply to this post by Erik Helin-2
Hi Erik,

On 04/20/2017 08:55 AM, Erik Helin wrote:

> Hi all,
>
> since JEP 291 [0] has been targeted for JDK 9 [1], here is the patch
> that implements the change. The patch itself is very small, it just
> adds a "-XX:+UseConcMarkSweepGC" entry to the special_jvm_flags array
> and updates a test. The test checks that a deprecation message is
> correctly printed for the flags:
> -XX:+UseConcMarkSweepGC
> -Xconcgc
> -Xnocongc
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8179013/00/
Looks good to me too.

Thanks,
Sangheon


>
> Issue:
> https://bugs.openjdk.java.net/browse/JDK-8179013
>
> Testing:
> - JPRT
> - make run-test TEST=hotspot/test/gc/startup_warnings/TestCMS.java
> - make run-test TEST=hotspot_jprt
> (tested with both open and commercial builds)
>
> Thanks,
> Erik
>
> [0]: http://openjdk.java.net/jeps/291
> [1]:
> http://mail.openjdk.java.net/pipermail/jdk9-dev/2017-April/005766.html

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: 8179013: Deprecate the Concurrent Mark Sweep (CMS) Garbage Collector

Erik Helin-2
On 04/21/2017 04:04 PM, sangheon wrote:

> Hi Erik,
>
> On 04/20/2017 08:55 AM, Erik Helin wrote:
>> Hi all,
>>
>> since JEP 291 [0] has been targeted for JDK 9 [1], here is the patch
>> that implements the change. The patch itself is very small, it just
>> adds a "-XX:+UseConcMarkSweepGC" entry to the special_jvm_flags array
>> and updates a test. The test checks that a deprecation message is
>> correctly printed for the flags:
>> -XX:+UseConcMarkSweepGC
>> -Xconcgc
>> -Xnocongc
>>
>> Webrev:
>> http://cr.openjdk.java.net/~ehelin/8179013/00/
> Looks good to me too.

Thanks for reviewing Sangheon!
Erik

> Thanks,
> Sangheon
>
>
>>
>> Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8179013
>>
>> Testing:
>> - JPRT
>> - make run-test TEST=hotspot/test/gc/startup_warnings/TestCMS.java
>> - make run-test TEST=hotspot_jprt
>> (tested with both open and commercial builds)
>>
>> Thanks,
>> Erik
>>
>> [0]: http://openjdk.java.net/jeps/291
>> [1]:
>> http://mail.openjdk.java.net/pipermail/jdk9-dev/2017-April/005766.html
>
Loading...