request for review (s) - 8005452: Create new flags for Metaspace resizing policy

[Jon Masamitsu]

Coleen,

Thanks for the review.

I delete the print at 1013 (instead of moving it) and  reverted 
get_new_chunk().  I left in the
print at 1014.

I have 2 webrevs for  these changes now.  Your suggested changes are in

http://cr.openjdk.java.net/~jmasa/8006815/webrev.00/

Webrev with the Min/MaxMetaspaceFreeRatio changes is

http://cr.openjdk.java.net/~jmasa/8005452/webrev.02/

Jon

On 1/23/2013 9:01 AM, Coleen Phillimore wrote:
>
> It looks okay except I think passing SpaceManager* to get_new_chunk() 
> is really gross just to do a print out.  I'd rather see the printing 
> at line 1014 moved to 2170 whether you get a new virtual space or 
> not.  There's already a ton of output from 
> TraceMetadataChunkAllocation && Verbose, but you can leave the 
> printing at 1013 so you know which one created a new virtual space.
>
> Coleen
>
>
> On 01/14/2013 10:30 AM, Jon Masamitsu wrote:
>> 8005452: Create new flags for Metaspace resizing policy
>>
>> Previously the calculation of the metadata capacity at which
>> to do a GC (high water mark, HWM) to recover
>> unloaded classes used the MinHeapFreeRatio
>> and MaxHeapFreeRatio to decide on the next HWM.  That
>> generally left an excessive amount of unused capacity for
>> metadata.  This change adds specific flags for metadata
>> capacity with defaults more conservative in terms of
>> unused capacity.
>>
>> Added an additional check for doing a GC before expanding
>> the metadata capacity.  Required adding a new parameter to
>> get_new_chunk().
>>
>> Added some additional diagnostic prints.
>>
>> http://cr.openjdk.java.net/~jmasa/8005452/webrev.00/
>>
>> Thanks.
>