Looks good.
Thanks!
Coleen
On 10/11/2013 2:02 PM, Stefan Karlsson wrote:
On 10/11/13 7:44 PM, Coleen Phillimore wrote:
Stefan,
I added a check about the count exceeding 1 000 000 because it causes
the test to create an invalid class file. With the shell script
changes, it won't happen but I would be happy if that check were
there so no one ever has to debug this thinking it has a corrupted
constant pool again.
OK. I though that was old debugging left-overs.
New webrev:
http://cr.openjdk.java.net/~stefank/7196801/webrev.01/
thanks,
StefanK
Thanks,
Coleen
On 10/11/2013 1:33 PM, Stefan Karlsson wrote:
Please, review this test fix.
http://cr.openjdk.java.net/~stefank/7196801/webrev.00/
Most of the fix was implemented by Coleen, I've only done some small
tweaks and updated some comments.
The fix depends on these changes:
JDK-8025996 Track metaspace usage when metaspace is expanded
JDK-8024547 MaxMetaspaceSize should limit the committed memory
used by the metaspaces
Both of these changesets have been pushed to hotspot-gc and are
currently undergoing testing. LowMemoryTest2 can only be removed
from the test/ProblemList.txt when both JDK-8025996 and JDK-8024547
have propagated through all necessary repositories.
thanks,
StefanK
