Thank you for review!

Still waiting for official 'R'eview.

Leonid
On 04.10.2016 16:36, Dmitry Fazunenko wrote:
Hi Leonid,

The fix looks good to me.

Thanks,
Dima

On 04.10.2016 11:23, Leonid Mesnik wrote:

Hi

Could you please review following fix:

Webrev: http://cr.openjdk.java.net/~lmesnik/8155570/webrev.00/ <http://cr.openjdk.java.net/%7Elmesnik/8155570/webrev.00/>

Bug: https://bugs.openjdk.java.net/browse/JDK-8155570

Test filled java heap up to 70% and then failed with OOME in the java heap while filling metaspace. I updated test to fill metaspace first and then to fill heap. Also I added more info about unexpected OOME.

I verified locally that OOME doesn't happen now on my local host with all GC. Unfortunately I haven't run it yet in the lab because of infra outage. Also test still might fail because of jcmd/jstat crash caused by
<https://bugs.openjdk.java.net/browse/JDK-8166364>

JDK-8166364 <https://bugs.openjdk.java.net/browse/JDK-8166364> fatal error: acquiring lock DirtyCardQ_CBL_mon/16 out of order with lock Module_lock/6 -- possible deadlock


Leonid




Reply via email to