Request for review (s) - 8155948: Add message for CMS deprecation for some internal builds

[Kim Barrett]

> On Sep 27, 2016, at 11:16 AM, Jon Masamitsu <jon.masamitsu at oracle.com> wrote:
> 
> Kim,
> 
> I fixed
> 
> https://bugs.openjdk.java.net/browse/JDK-8166461
> 
> so that the fix version is 9.

Thanks.  That addresses my objection to the latest version (webrev.04) for 8155948.

> Jon
> 
> On 9/26/2016 2:18 PM, Jon Masamitsu wrote:
>> Kim,
>> 
>> I thought that upshot of the discussion earlier in the week was that
>> the UseAutoGCSelectPolicy flag should be deprecated itself and
>> so would circumvent the warning about CMS being deprecated if
>> UseAutoGCSelectPolicy picked CMS.  Dima's point was that GC
>> ergonomics should not pick a collector that was deprecated.
>> 
>> Not so?
>> 
>> Jon
>> 
>> On 9/23/2016 3:33 PM, Kim Barrett wrote:
>>>> On Sep 23, 2016, at 2:25 PM, Jon Masamitsu <jon.masamitsu at oracle.com> wrote:
>>>> 
>>>> After some discussion I've removed the handling of the
>>>> UseAutoGCSelectPolicy flag from this patch.  It will be handled
>>>> separately under
>>>> 
>>>> https://bugs.openjdk.java.net/browse/JDK-8166461
>>>> 
>>>> 
>>>> The new full webrev is
>>>> 
>>>> http://cr.openjdk.java.net/~jmasa/8155948/webrev.04/
>>>> 
>>>> The delta is
>>>> 
>>>> http://cr.openjdk.java.net/~jmasa/8155948/webrev_delta.03_04/index.html
>>>> 
>>>> Thanks.
>>>> 
>>>> Jon
>>> ------------------------------------------------------------------------------ 
>>> src/share/vm/runtime/arguments.cpp
>>> 
>>> I think this line should not be removed:
>>> 
>>> 1807       handle_extra_cms_flags("UseAutoGCSelectPolicy uses UseConcMarkSweepGC");
>>> 
>>> Removal of this line means that commercial builds that otherwise warn
>>> about CMS being deprecated when it is selected won't warn if it is
>>> selected via this mechanism.
>>> 
>>> Similarly for the test.
>>> 
>>> So I liked the previous version and not so much this one.
>>> 
>>> ------------------------------------------------------------------------------