<html>

  <head>

    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <tt>> <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tschatzl/8129977/webrev/">http://cr.openjdk.java.net/~tschatzl/8129977/webrev/</a><br>

      <br>

      test/gc/g1/TestSummarizeRSetStats.java<br>

      test/gc/g1/TestSummarizeRSetStatsPerRegion.java<br>

          No comments on the @bug additions.<br>

      <br>

      test/gc/g1/TestSummarizeRSetStatsTools.java<br>

          '-Xms' addition looks fine.<br>

      <br>

      Thumbs up!<br>

      <br>

      Dan<br>

      <br>

      <br>

    </tt>

    <meta http-equiv="content-type" content="text/html;

      charset=ISO-8859-1">

    <pre>On 7/1/2015 8:18 AM, Thomas Schatzl wrote:

><i> Hi all,

</i>><i>

</i>><i>    can I have reviews for the following tiny change that fixes the test

</i>><i> TestSummarizeRSetStats on machines with a "small" amount of available

</i>><i> physical memory?

</i>><i>

</i>><i> The problem is that when run on machines with little memory, it issues

> two young-only garbage collections instead of one.

</i>><i>

</i>><i> The problem is that the test does not fix the initial heap size, which

</i>><i> is calculated according to available memory. This causes an additional

</i>><i> young GC after the first one induced by the test, messing up the

> expected amount of log messages.

</i>><i>

</i>><i> The reported behavior can be reproduced locally by either manually

</i>><i> setting -Xms to a significantly lower value than -Xmx (like 12m) or

</i>><i> setting MaxRAM to the same size as the machine's total physical memory

</i>><i> (512m - which results in a too small initial heap size of 10M, causing

> this exact error).

</i>><i>

</i>><i> After forcing -Xms to -Xmx in the test, it is good again regardless of

</i>><i> other options.

</i>><i>

</i>><i> Thanks to StefanJ for observing the issue with automatic heap sizing on

> the same machine for JDK-8129590 and getting me on the right track.

</i>><i>

</i>><i> CR:

</i>><i> <a href="https://bugs.openjdk.java.net/browse/JDK-8129977">https://bugs.openjdk.java.net/browse/JDK-8129977</a>

</i>><i>

</i>><i> Webrev:

</i>><i> <a href="http://cr.openjdk.java.net/%7Etschatzl/8129977/webrev/">http://cr.openjdk.java.net/~tschatzl/8129977/webrev/</a>

</i>><i>

</i>><i> Testing:

</i>><i> jprt

</i>><i>

</i>><i> Thanks,

</i>><i>    Thomas

</i>><i>

</i>></pre>

    <br>

  </body>

</html>