Review request for 6824135
Xueming Shen
Xueming.Shen at Sun.COM
Wed Apr 1 19:56:46 UTC 2009
Alan Bateman wrote:
> 6824135: (ch) test/java/nio/channels/AsyncCloseAndInterrupt.java fails
> (lnx)
>
> test/java/nio/channels/AsyncCloseAndInterrupt.java has been failing
> since b50. The failure is a hang that arises when FileChannel methods
> are invoked with a pending interrupt. The pending interrupt causes the
> channel to be pre-closed and a wait until all other threads are
> finished with the file descriptor. I broke this in b50 with the nio2
> bits so that the current thread is added to the thread list before the
> begin method, which marks the beginning of an I/O operation, is
> invoked. The test caught the bug but I'm surprised that I didn't spot
> the failure before pushing the changes. In any case, the fix is simple:
> http://cr.openjdk.java.net/~alanb/6824135/webrev.00/
>
> (Sherman, if you cycles, I'd like to get this one into b54).
>
> Thanks,
> Alan.
looks good; approved.
Sherman
More information about the core-libs-dev
mailing list