Review request: JDK-7147084 (process) appA hangs when read output stream of appB which starts appC that runs forever (v.1)
rob.mckenna at oracle.com
Wed Jul 17 04:44:49 UTC 2013
I've just had a look at the webrev. Looks good. Unfortunately I'm not a
reviewer here so the only benefit was for my education I'm afraid. It
would be great to get Alan's input next week.
For the benefit of the alias: I had asked Alexey if an intermediate
launcher akin to the one described by MS and found in 5049299 would make
sense. Alexey mentioned the cost of launching processes on windows was
pretty high making this approach preferable.
On 11/07/13 10:03, Alexey Utkin wrote:
> 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:
> Discussion thread:
> Bug description:
> Here is the suggested fix:
> 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.
> 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.
>> 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.
More information about the core-libs-dev