Request for review (s) - 7198873

[Mikael Gerdin]

Jon,

On 2012-09-24 23:46, Jon Masamitsu wrote:
> NPG: VM Does not unload classes with UseConcMarkSweepGC
>
> If CMS is not doing class unloading, don't start a concurrent
> collection for classloader (and metadata) collection (since
> it won't happen without class unloading).

It looks like you still unconditionally call expand_and_allocate when 
running with CMS, no matter the value of CMSClassUnloadingEnabled.
I think that the code:

  213       // For CMS expand since the collection is going to be 
concurrent.
  214       _result =
  215 
_loader_data->metaspace_non_null()->expand_and_allocate(_size, _mdtype);

Should be inside the "if (CMSClassUnloadingEnabled)" and if running 
without it set then CMS users will have to take the hit of a stw full gc 
when running into the metadata threshold.

/Mikael

>
> http://cr.openjdk.java.net/~jmasa/7198873/webrev.00/
>
> Also, refactored the code for readability and guarded extra
> output with Verbose.
>
> Thanks.
>
> Jon

-- 
Mikael Gerdin
Java SE VM SQE Stockholm