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");

On 12/5/2014 9:47 AM, Kumar Srinivasan wrote:
Hang on, I have to increase the Heap size to pack200.exe, I will
repost the changes, when I am done testing with JPRT.

Kumar

On 12/5/2014 9:11 AM, Alan Bateman wrote:
On 05/12/2014 15:49, Kumar Srinivasan wrote:
Hi Alan,

Per your suggestion I have made this othervm.

Thanks
Kumar
Do you want to add -Xshare:off too? Just wondering how tight this is on 32-bit Windows.

-Alan.


Reply via email to