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

Jon Masamitsu jon.masamitsu at oracle.com
Wed Jan 25 15:11:46 UTC 2012



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.

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
>



More information about the hotspot-gc-dev mailing list