<div dir="ltr">Comments reworked as suggested and webrev refreshed.<div><a href="http://cr.openjdk.java.net/~martin/webrevs/openjdk9/O_CLOEXEC/" target="_blank" style="font-family:arial,sans-serif;font-size:13px">http://cr.openjdk.java.net/~martin/webrevs/openjdk9/O_CLOEXEC/</a><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 22, 2014 at 7:48 PM, David Holmes <span dir="ltr"><<a href="mailto:david.holmes@oracle.com" target="_blank">david.holmes@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Martin,<span class=""><br>
<br>
On 23/10/2014 7:34 AM, Martin Buchholz wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Re-ping.<br>
<br>
I have a simple change, reviewed by hotspot engineer, and I just need a<br>
jprt run.<br>
<br>
Optionally, we could also use a simple port to other platforms that might<br>
support O_CLOEXEC, but that can be done later.<br>
</blockquote>
<br></span>
Functionally this seems okay to me. The only suggestion I have is to move the large comment block about close-on-exec to the point where you set O_CLOEXEC and explain the attempt and fallback.<br>
<br>
If you prepare the final changeset I will sponsor it for you.<br>
<br>
Cheers,<br>
David<div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, Jul 1, 2014 at 2:49 PM, Martin Buchholz <<a href="mailto:martinrb@google.com" target="_blank">martinrb@google.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Ping. Please review this change.<br>
<br>
It is in production use locally on Linux.  Please either accept this<br>
change as is (Linux only), or work with this patch to create a more<br>
ambitious patch that addresses other Unix variants.<br>
<br>
Adding porters-dev.<br>
<br>
<br>
On Thu, May 22, 2014 at 10:14 AM, Martin Buchholz <<a href="mailto:martinrb@google.com" target="_blank">martinrb@google.com</a>><br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
I'd like to contribute a patch.<br>
<br>
Webrev:<br>
   <a href="http://cr.openjdk.java.net/~martin/webrevs/openjdk9/O_CLOEXEC/" target="_blank">http://cr.openjdk.java.net/~<u></u>martin/webrevs/openjdk9/O_<u></u>CLOEXEC/</a><br>
<br>
Bug:<br>
   <a href="https://bugs.openjdk.java.net/browse/JDK-8043780" target="_blank">https://bugs.openjdk.java.net/<u></u>browse/JDK-8043780</a><br>
<br>
If accepted, corresponding changes should be made to the other Unix<br>
variants, but I can't do any testing there.<br>
<br>
</blockquote>
<br>
<br>
</blockquote></blockquote>
</div></div></blockquote></div><br></div>