RFR [8024521] (process) Async close issues with Process InputStream
Ivan Gerasimov
ivan.gerasimov at oracle.com
Thu Oct 17 19:13:44 UTC 2013
Thank you Alan!
Yes, I missed that fact that reading from the recycled file descriptor
is actually a problem by itself.
Would you please take a look at another updated webrev?
It contains another implementation suggested by Paul Sandoz with some
minor changes.
http://cr.openjdk.java.net/~igerasim/8024521/2/webrev/
<http://cr.openjdk.java.net/%7Eigerasim/8024521/2/webrev/>
Here we synchronize close() with calls to available() and read() and
check for asynchronous close() that could have happened in between.
Sincerely yours,
Ivan
On 17.10.2013 16:34, Alan Bateman wrote:
> On 15/10/2013 16:31, Ivan Gerasimov wrote:
>> Here's the updated webrev with the solution by Paul:
>> http://cr.openjdk.java.net/~igerasim/8024521/1/webrev/
>>
>> I've also excluded the test from it.
>> Instead, I'm going to write a manual testing instruction for SQE.
>>
>> I've tested the fix and it works as expected.
> I've looked at the updated webrev but there still appears to be an
> issue if the application closes the stream at just around the time
> that the process exits. So I think you will need to go back to a
> solution along the lines of the first patch so there is coordination
> between the process exit and the close.
>
> -Alan.
>
>
More information about the core-libs-dev
mailing list