<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Seems fine.<div><br></div><div>igor</div><div><br><div><div>On Sep 17, 2013, at 10:44 AM, Albert Noll &lt;<a href="mailto:albert.noll@oracle.com">albert.noll@oracle.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
  

    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
  
  <div text="#000000" bgcolor="#FFFFFF">
    Hi,<br>
    <br>
    could I have reviews for this small patch?<br>
    <br>
    bug: <a href="https://bugs.openjdk.java.net/browse/JDK-7009641">https://bugs.openjdk.java.net/browse/JDK-7009641</a><br>
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1">
    webrev:
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1">
    <a href="http://cr.openjdk.java.net/%7Eanoll/7009641/webrev.00/">http://cr.openjdk.java.net/~anoll/7009641/webrev.00/</a><br>
    <br>
    <br>
    <br>
    Problem: <br>
    If the code cache is full and we try to allocate vtable stubs, we
    have a crash although we could continue execution.<br>
    <br>
    Solution: <br>
    Since memory from the code cache can be allocated/released by
    multiple threads, the cleanest solution is to let the allocation<br>
    in the code cache return NULL instead of failing the entire VM (for
    vtable stubs). If there is no memory left, no vtable stubs will<br>
    be created and the IC is cleaned. <br>
    <br>
    Many thanks in advance,<br>
    Albert<br>
  </div>

</blockquote></div><br></div></body></html>