<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix"><br>
      Hi Vitaly,<br>
      <br>
      Thanks for looking at this!<br>
      <br>
      On 1/15/13 2:03 PM, Vitaly Davidovich wrote:<br>
    </div>
    <blockquote
cite="mid:CAHjP37GCUgS32KV5H5ZwUmTb_r61U0kKubwh5gY4UYkYu3GApg@mail.gmail.com"
      type="cite">
      <p dir="ltr">Hi Bengt,</p>
      <p dir="ltr">Looks good.  Do you need the constants for
        int/double/size_t? Would it be easier to have a static getter
        that returns "(T)-1" and then use that instead?</p>
    </blockquote>
    <br>
    It would work, but I'm not sure what is best. I prefer the
    constants, but I would be fine with a getter too. If you don't have
    strong objections I'll leave it as it is.<br>
    <br>
    Thanks,<br>
    Bengt<br>
    <br>
    <blockquote
cite="mid:CAHjP37GCUgS32KV5H5ZwUmTb_r61U0kKubwh5gY4UYkYu3GApg@mail.gmail.com"
      type="cite">
      <p dir="ltr">Thanks</p>
      <p dir="ltr">Sent from my phone</p>
      <div class="gmail_quote">On Jan 14, 2013 4:08 PM, "Bengt Rutisson"
        <<a moz-do-not-send="true"
          href="mailto:bengt.rutisson@oracle.com">bengt.rutisson@oracle.com</a>>
        wrote:<br type="attribution">
        <blockquote class="gmail_quote" style="margin:0 0 0
          .8ex;border-left:1px #ccc solid;padding-left:1ex">
          <div bgcolor="#FFFFFF" text="#000000"> <br>
            Hi all,<br>
            <br>
            Could I have a couple of reviews for this small change?<br>
            <a moz-do-not-send="true"
              href="http://cr.openjdk.java.net/%7Ebrutisso/8006242/webrev.00/"
              target="_blank">http://cr.openjdk.java.net/~brutisso/8006242/webrev.00/</a><br>
            <br>
            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>
          </div>
        </blockquote>
      </div>
    </blockquote>
    <br>
  </body>
</html>