[8u-dev]: Request for Review and Approval: 8075484: SocketInputStream.socketRead0 can hang even with soTimeout set
Chris Hegarty
chris.hegarty at oracle.com
Wed Jan 11 10:47:15 UTC 2017
Hi Christoph,
> On 9 Jan 2017, at 05:56, Langer, Christoph <christoph.langer at sap.com> wrote:
>
> Ping: Please review this backport to JDK8.
>
> From: Langer, Christoph
> Sent: Donnerstag, 29. Dezember 2016 10:37
> To: net-dev at openjdk.java.net; jdk8u-dev at openjdk.java.net
> Subject: [8u-dev]: Request for Review and Approval: 8075484: SocketInputStream.socketRead0 can hang even with soTimeout set
>
> Hi,
>
> please review (and eventually approve) the change for downporting 8075484.
>
> Webrev for 8u-dev: http://cr.openjdk.java.net/~clanger/webrevs/8075484.8udev/
The changes look ok to me.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8075484
> JDK9 Change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/af17b6bc08dd
> JDK9 Review Thread(s):
> http://mail.openjdk.java.net/pipermail/net-dev/2016-August/010171.html
> http://mail.openjdk.java.net/pipermail/net-dev/2016-September/010201.html
>
> We had customer reports who ran into that issue with Java 8. So this should be downported.
>
> The problem is, that the fix does not apply to Solaris as Solaris needs some calls into hotspot. This is because in JDK 8 the flag for interruptible IO is still supported (though deprecated). But I think it is still worthwile to bring this down for the other platforms which I’m proposing with my changeset. So I extracted the new code manually from the JDK9 changeset and made it fit into JDK8 coding.
Ok, I see the complication. On Solaris these calls still go through
the VM to support Interruptible IO ( through the JVM_XXX interface).
Thankfully, this is no longer the case in 9.
-Chris.
More information about the net-dev
mailing list