<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Hi Bengt,<br>
    <br>
    Changes look good to me. Minor nits:<br>
    <br>
    Copyrights need updating.<br>
    Use UINT32_FORMAT instead of %d in the error message.<br>
    Check the indentation of the for loop in
    G1GCPhaseTimes::note_gc_end().<br>
    <br>
    JohnC<br>
    <br>
    <div class="moz-cite-prefix">On 1/14/2013 1:06 PM, Bengt Rutisson
      wrote:<br>
    </div>
    <blockquote cite="mid:50F47361.7020306@oracle.com" type="cite">
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      <br>
      Hi all,<br>
      <br>
      Could I have a couple of reviews for this small change?<br>
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Ebrutisso/8006242/webrev.00/">http://cr.openjdk.java.net/~brutisso/8006242/webrev.00/</a><br>
      <br>
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      Thanks to John Cuthbertson for finding this bug and providing
      excellent data to track down the issue.<br>
      <br>
      From the bug report:<br>
      <br>
      In non-product builds the WorkerDataArrays in G1 are initialized
      to -1 in WorkerDataArray<T>::reset() when a GC starts. At
      the end of a GC WorkerDataArray<T>::verify() verifies that
      all entries in a WorkerDataArray has been set. Currently it does
      this by asserting that the entries are >= 0. This is fine in
      theory since the entries should contain counts or times that are
      all positive.<br>
      <br>
      The problem is that some WorkerDataArrays are of type double. And
      some of those are set up through calculations using doubles. If
      those calculations result in a value close to 0 we could end up
      with a value slightly less than 0 since double calculations don't
      have full precision.<br>
      <br>
      All we really want to verify is that all the entries were set. So,
      it should be enough to verify that entries do not contain the
      value set by the reset() method. <br>
      <br>
      Bengt<br>
    </blockquote>
    <br>
  </body>
</html>