<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Thank you, Igor.<br>
    <br>
    Best,<br>
    Tobias<br>
    <br>
    <div class="moz-cite-prefix">On 21.06.2014 05:50, Igor Veresov
      wrote:<br>
    </div>
    <blockquote
      cite="mid:F37DED7C-44A2-4F15-8F2C-AFA0353D8116@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=ISO-8859-1">
      Looks good.
      <div><br>
      </div>
      <div>igor</div>
      <div><br>
        <div style="">
          <div>On Jun 19, 2014, at 12:15 AM, Tobias Hartmann <<a
              moz-do-not-send="true"
              href="mailto:tobias.hartmann@oracle.com">tobias.hartmann@oracle.com</a>>
            wrote:</div>
          <br class="Apple-interchange-newline">
          <blockquote type="cite">
            <meta http-equiv="content-type" content="text/html;
              charset=ISO-8859-1">
            <div bgcolor="#FFFFFF" text="#000000"> Hi,<br>
              <br>
              please review the following patch for 8035328.<br>
              <br>
              Bug: <a moz-do-not-send="true"
                class="moz-txt-link-freetext"
                href="https://bugs.openjdk.java.net/browse/JDK-8035328">https://bugs.openjdk.java.net/browse/JDK-8035328</a><br>
              Webrev: <a moz-do-not-send="true"
                class="moz-txt-link-freetext"
                href="http://cr.openjdk.java.net/%7Ethartmann/8035328/webrev.00">http://cr.openjdk.java.net/~thartmann/8035328/webrev.00</a><br>
              <br>
              <b>Problem</b><b><br>
              </b>This bug is not a test bug. The test executes hundreds
              of small methods consisting of only a return statement
              multiple times. This was intended to test for an old
              (closed) bug in the C2 compiler. The timeout arises when
              the tiered compilation policy decides to recompile the
              caller method and therefore marks it as non-entrant. The
              method is still in use but call sites are not patched. See
              line 1179 of sharedRuntime.cpp:<br>
              <br>
              // We do not patch the call site if the caller nmethod has
              been made non-entrant.<br>
              <br>
              As a result, all call sites have to be resolved on every
              call which is costly and leads to the timeout (4mins vs.
              17s).<br>
              <br>
              <b>Solution<br>
              </b>The call site patching code in sharedRuntime is
              changed to patch not entrant call sites as well.<br>
              <b><br>
              </b><b>Testing<br>
              </b>- JPRT<br>
              - SPECjbb2005, SPECjbb2013<br>
              - SPECjvm2008-Compiler<b><br>
              </b>- Nashorn + Octane<b><br>
              </b>- UTE full run with Kitchensink<br>
               <br>
              Thanks,<br>
              Tobias<br>
            </div>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </body>
</html>