Rob,
Could you help me with bug review. It stay in review pool too long.
Nobody like to look into the native code.
Here is the long problem & solution description:
https://jbs.oracle.com/bugs/browse/JDK-7147084?focusedCommentId=13322689&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13322689
Discussion thread:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-May/016753.html
Bug description:
https://jbs.oracle.com/bugs/browse/JDK-7147084
http://bugs.sun.com/view_bug.do?bug_id=7147084
Here is the suggested fix:
http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-7147084/webrev.01/
Summary for v1 changes:
-- The set of handles that need to restore the inherit flag was extended
by child process IOE handles. That was done to avoid "greedy sibling"
problem for the file handles. The file handles are closed outside the
synchronized block.
-- "Greedy sibling" problem is covered by the
[test/java/lang/ProcessBuilder/SiblingIOEHandle.java] test.
-- The fact that the set of current process standard IOE handles and the
set of child process IOE handles can intersect was taken into account.
-- The [test/java/lang/ProcessBuilder/InheritIOEHandle.java] was changed
in accordance with Martin's concern.
Regards,
-uta
On 7/5/2013 5:59 PM, Alan Bateman wrote:
On 05/07/2013 09:37, Alexey Utkin wrote:
Hi Alan,
I know that your stack is overflowed,
that is just ping.
Regards,
-uta
Sorry Alexey, I just have not had time to fully digest this one and
understand the performance/correctness trade-off. Also I've been
desperately trying to get through a list of things before going on
vacation this week (for 2 weeks).
Rob - since you are interested in ProcessBuilder and Runtime.exec then
could you take a pass over Alexey's change on core-libs-dev. This one
fixes a long standing issue on Windows. I will get make time to help
review when I get back.
-Alan