<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    <div class="moz-cite-prefix">On 4/11/14, 12:00 PM, Keith McGuigan
      wrote:<br>
    </div>
    <blockquote
cite="mid:CADNkbOFdTUb8-1=qcTk6=TQPOF5sWiQmg=vVP7OvTiPGDOoDKw@mail.gmail.com"
      type="cite">
      <div dir="ltr">Looks good, but why are you not using a newer
        version of webrev with "next" links??</div>
    </blockquote>
    <br>
    Thanks Keith.  The private copy I had of the "next" link webrev that
    I had broke for me for some mysterious reason.  I filed a bug to see
    if the "official" version of webrev could have "next" links instead
    of debugging my own.<br>
    Thanks!<br>
    Coleen<br>
    <br>
    <blockquote
cite="mid:CADNkbOFdTUb8-1=qcTk6=TQPOF5sWiQmg=vVP7OvTiPGDOoDKw@mail.gmail.com"
      type="cite">
      <div dir="ltr">
        <div><br>
        </div>
        <div>--</div>
        <div>- Keith</div>
      </div>
      <div class="gmail_extra"><br>
        <br>
        <div class="gmail_quote">On Fri, Apr 11, 2014 at 10:48 AM,
          Coleen Phillimore <span dir="ltr"><<a
              moz-do-not-send="true"
              href="mailto:coleen.phillimore@oracle.com" target="_blank">coleen.phillimore@oracle.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">Summary:
            Dtrace monitoring uses size before mirror size is set.<br>
            <br>
            The refactoring I did for bug <a moz-do-not-send="true"
              href="https://bugs.openjdk.java.net/browse/JDK-8028497"
              target="_blank">https://bugs.openjdk.java.net/browse/JDK-8028497</a>
            caused this bug. The size of the mirror is filled in by the
            InstanceMirrorKlass allocation but was used for dtrace
            probes before it in the normal allocation pass.  Pass the
            allocated size to dtrace function instead.<br>
            <br>
            Tested by dtrace tests on solaris sparcv9, testbase
            vm.quick.testlist also.<br>
            <br>
            open webrev at <a moz-do-not-send="true"
              href="http://cr.openjdk.java.net/%7Ecoleenp/8039904/"
              target="_blank">http://cr.openjdk.java.net/~coleenp/8039904/</a><br>
            bug link <a moz-do-not-send="true"
              href="https://bugs.openjdk.java.net/browse/JDK-8039904"
              target="_blank">https://bugs.openjdk.java.net/browse/JDK-8039904</a><br>
            <br>
            Thanks,<br>
            Coleen<br>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </body>
</html>