review request (XS) - 7112413: disable AdaptiveSizePolicy w/CMS

[Bengt Rutisson]

Jon,

Thanks for your feedback!

On 2012-01-25 16:11, Jon Masamitsu wrote:
>
>
> On 1/25/2012 12:01 AM, Bengt Rutisson wrote:
>>
>> ...
>>
>> Finally, a question that is not directly related to your change now. 
>> But what is the plan for CMS and UseAdaptiveSizePolicy? Do we plan on 
>> fixing it or should we just remove it? If the latter is the case, is 
>> there a CR to remove it? With your change there is quite a few lines 
>> of code that are essentially dead now.
>
> If you do a CR to remove the UseAdaptiveSizePolicy for CMS, please do 
> no remove the code
> from ParNew and SerialGC.  I think client side GC ergo is in our 
> future.  I'd also suggest
> making the CR dependent on G1 replacing CMS.  Just on the decision 
> that we will be
> removing CMS, not waiting for the actual removal of the CMS code.

Good point. My intent was not to file a CR, but to find out what the 
plans are. Let's do what John Coomes suggested and include this topic in 
our discussions around the future path for the GC code.

Bengt

>
> Jon
>
>>
>> Bengt
>>
>> On 2012-01-25 06:26, John Coomes wrote:
>>> I'd appreciate reviews of a simple change to disable
>>> AdaptiveSizePolicy with CMS and/or ParNew, even if it has been enabled
>>> on the command line.
>>>
>>> http://cr.openjdk.java.net/~jcoomes/7112413-cms-adaptive
>>>
>>> -John
>>