RFR: 8234382: Test tools/javac/processing/model/testgetallmembers/Main.java using too small heap

Per Liden per.liden at oracle.com
Tue Nov 19 07:32:10 UTC 2019


Hi,

Please review this one-liner test fix.

The test tools/javac/processing/model/testgetallmembers/Main.java is 
assuming that a 256M heap is enough to hold its live-set, but this is 
only true under some conditions. There are a number of JVM flags that, 
when used, can break this assumption. For example, choice of GC, 
compressed oop, object alignment, and other options affecting the heap 
layout or allocation strategy. Under ideal conditions, the test is 
already fairly close to using the whole heap, so it doesn't take that 
much to push it over the edge. For example, the following combinations 
all fail:

-XX:+UseSerialGC -XX:-UseCompressedOops
-XX:+UseParallelGC -XX:-UseCompressedOops
-XX:+UseZGC (always has -XX:-UseCompressedOops)
-XX:+UseG1GC -XX:-UseCompressedOops -XX:ObjectAlignmentInBytes=16

I suggest we bump the max heap size to something like 512M, to give the 
test more headroom and make it less sensitive to exact choice of JVM flags.

Bug: https://bugs.openjdk.java.net/browse/JDK-8234382
Webrev: http://cr.openjdk.java.net/~pliden/8234382/webrev.0

/Per


More information about the compiler-dev mailing list