[8u-dev]: Request for Review and Approval: 8075484: SocketInputStream.socketRead0 can hang even with soTimeout set

[Vyom Tewari]

Hi Christoph,

Code change looks good to me, but i am not an official reviewer.

Thanks,

Vyom


On Monday 09 January 2017 11:26 AM, Langer, Christoph 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/ 
> <http://cr.openjdk.java.net/%7Eclanger/webrevs/8075484.8udev/>
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8075484 
> <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.
>
> Thanks & Best regards
>
> Christoph
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/net-dev/attachments/20170110/9a6c6c75/attachment.html>