Per, The fix looks good for me.
Copyright year of these tests is not updated. GcCapacityTest.java GcCauseTest01.java GcTest02.java -Dmitry On 2016-04-06 12:32, Per Liden wrote: > Summary: This patch updates the tests in serviceability/tmtools/jstat/ > to use a fixed (and relatively small) heap size. Without this these > tests tend to run out of memory on machines with VM overcommit disabled. > This has happened in nightly testing. > > The patch also moves a misplaced @ignore tag and cleans out an unused > method. > > Testing: Without this patch I can easily get these tests to OOM on my > local workstation by just disabling VM overcommit. They pass with this > patch applied. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8152989 > Webrev: http://cr.openjdk.java.net/~pliden/8152989/webrev.0/ > > cheers, > Per > -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.