Alan,
I have adjust both the tests Pack200Props as well as Pack200Test which uses
rt.jar to have increased heaps and -Xshare:off.
http://cr.openjdk.java.net/~ksrini/8066745/webrev.00/
Thanks
Kumar
On 12/6/2014 2:36 AM, Alan Bateman wrote:
On 06/12/2014 01:36, Kumar Srinivasan wrote:
Alan,
The heap needs to be increased (1.2GB) for this test, I think the
default provided
by the launcher ergonomics is not sufficient on certain platforms. I've
tested the patch with JPRT.
Thanks
Kumar
diff --git a/test/tools/pack200/Pack200Props.java
b/test/tools/pack200/Pack200Props.java
--- a/test/tools/pack200/Pack200Props.java
+++ b/test/tools/pack200/Pack200Props.java
@@ -62,6 +62,7 @@
List<String> cmdsList = new ArrayList<>();
cmdsList.add(Utils.getPack200Cmd());
+ cmdsList.add("-J-Xmx1280m");
cmdsList.add("--effort=1");
cmdsList.add("--verbose");
cmdsList.add("--no-gzip");
This looks okay to me although 1.2GB makes me wonder if you might
still need -Xshare:off on Windows 32-bit.
-Alan
