RFR: 8252545: runtime/ErrorHandling/TestHeapDumpOnOutOfMemoryError.java timed out

[Thomas Stuefe]

On Mon, 25 Jan 2021 17:57:33 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

> Please review this change to fix JDK-8252545.  The fix uses -Xm128m to reduce the size of the heap, lessening the amount  of time needed to get an OutOfMemoryError exception.  The fix also simplifies things by including test TestHeapDumpOnOutOfMemoryErrorInMetaspace.java with this test.  (Hopefully, this won't increase the likelihood of timeouts.)
> 
> The fix was tested by running Mach5 tier 3 on Linux, Mac OS, and Windows, and running the test 100 times on Linxu x64 with the same JVM options that were used when the test last timed out: "-XX:MaxRAMPercentage=6 -XX:+UseParallelGC -XX:+UseNUMA".
> 
> Thanks, Harold

Hi Harold,

looks good and makes sense. I even believe we could drop the Metaspace test to the default timeout; the longest runtimes I found since JDK-8212218 are about 12-15 seconds on our slow platforms (leave that up to you though).

-------------

Marked as reviewed by stuefe (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2226