Code review request 7132889: Possible race conditions in AbstractSelectableChannel

Alan Bateman Alan.Bateman at oracle.com
Mon Aug 20 12:21:25 PDT 2012


On 20/08/2012 15:20, Shirish Kuncolienkar wrote:
>
> Alan,
>
> I created a new patch http://cr.openjdk.java.net/~luchsh/webrev.000/ 
> based on your suggestions and tested the same against 
> java/nio/channels tests
>
> I see no new failures as compared to
> java version "1.8.0-ea"
> Java(TM) SE Runtime Environment (build 1.8.0-ea-b49)
> Java HotSpot(TM) 64-Bit Server VM (build 24.0-b16, mixed mode)
>
> Detailed Test results
> passed: 159; failed: 1; error: 1
> Error:  java/nio/channels/DatagramChannel/AdaptDatagramSocket.java
> FAILED: java/nio/channels/DatagramChannel/MulticastSendReceiveTests.java
>
> The timing window was seen during fixing of the select loop issue and 
> the changes are being held as a patch.
Formatting aside, the updated patch looks good. I will sponsor this patch.

On the tests then I'm not sure why MulticastSendReceiveTests.java is 
failing for you. Can you forward the .jtr (output) file? It it possible 
that firewall or VPN settings are causing muticast packets to be dropped?

-Alan.


More information about the nio-dev mailing list