Request for review (s) 8025635: SoftReferences are not cleared before metaspace OOME are thrown
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Oct 17 08:32:35 UTC 2013
Hi,
On Wed, 2013-10-16 at 13:56 -0700, Jon Masamitsu wrote:
> I've fixed the CMS case and updated the webrev.
>
> http://cr.openjdk.java.net/~jmasa/8025635/webrev.01/
>
> With the fix an unneeded call to expand_and_allocate() may be
> made for CMS. I chose to keep the code simpler rather
> than to treat that CMS case.
>
Looks good. Could you also fix the additional spaces in line 214?
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list