<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Adding runtime alias...</p>
    <p>comments inlined below.</p>
    <p><br>
    </p>
    <p>On 11/13/17 9:10 PM, jamsheed wrote:<br>
    </p>
    <blockquote type="cite"
      cite="mid:d4167e21-5754-bc2a-1717-8dfe864fe6a8@oracle.com">
      <meta http-equiv="content-type" content="text/html; charset=utf-8">
      <span class="new" style="font-weight: normal;"></span>
      <pre style="font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration-style: initial; text-decoration-color: initial;"><span class="new" style="font-weight: normal;">Hi,

request for review,
jbs: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-6415680" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-6415680</a>

webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Ejcm/6415680/webrev.00/" moz-do-not-send="true">http://cr.openjdk.java.net/~jcm/6415680/webrev.00/</a>

Description:

1) changes equivalent to JDK-4454115 is done for windows.
</span></pre>
    </blockquote>
    <br>
    It looks like "nm" can be uninitialized if "in_java" is false.<br>
    <br>
    <blockquote type="cite"
      cite="mid:d4167e21-5754-bc2a-1717-8dfe864fe6a8@oracle.com">
      <pre style="font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration-style: initial; text-decoration-color: initial;"><span class="new" style="font-weight: normal;">
2) added guard to multiple value access sites, Unsafe_CopyMemory0, Unsafe_SetMemory0 and Unsafe_CopySwapMemory0.
</span></pre>
    </blockquote>
    <br>
    Can you narrow the scope of the unsafe access using something like
    GuardUnsafeAccess, instead of marking the whole function as doing
    unsafe access?<br>
    <br>
    There are some risks with trying to  abort a copy function.<br>
    <br>
    First, won't we get multiple exceptions until we finally hit the end
    of the range?  What if the bad range is very large?<br>
    <br>
    Second, what if the loop is using auto-increment instructions? 
    Skipping to the next instruction would mean we loop forever if the
    increment never happens.<br>
    <br>
    I think if we are going to safely abort copy functions then we need
    to register them as a kind of CodeBlob that has a special abort
    entry point or exception handler we can redirect to, or maybe pop
    the whole frame and return.<br>
    <br>
    Is there really a problem with these copy functions?  I'm wondering
    why Mikael did not identify these as a problem in 8154592.<br>
    <br>
    <blockquote type="cite"
      cite="mid:d4167e21-5754-bc2a-1717-8dfe864fe6a8@oracle.com">
      <pre style="font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration-style: initial; text-decoration-color: initial;"><span class="new" style="font-weight: normal;">
3) Unsafe_CopySwapMemory0 is JVM_LEAF so removed thread->thread_state() == _thread_in_vm checks from signal handler</span></pre>
    </blockquote>
    <br>
    How about adding a check for _thread_in_native instead of removing
    the check entirely?<br>
    <br>
    dl<br>
    <br>
    <blockquote type="cite"
      cite="mid:d4167e21-5754-bc2a-1717-8dfe864fe6a8@oracle.com">
      <pre style="font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration-style: initial; text-decoration-color: initial;"><span class="new" style="font-weight: normal;">

Best regards,
Jamsheed



 
</span><span class="new" style="font-weight: normal;"></span><span class="new" style="font-weight: normal;"></span></pre>
      <p> </p>
    </blockquote>
    <br>
  </body>
</html>