RFR: JDK-8144144 - ORB destroy() leaks filedescriptors after unsuccessful connection
Seán Coffey
sean.coffey at oracle.com
Wed Jan 20 18:17:24 UTC 2016
Hi Mark,
SelectorImpl.java:
line 125, could you use a 2 arg method call to dprint. It'll print the
stacktrace instead :
dprint(".close: selector.close: " + t); --> dprint(".close:
selector.close", t);
The "while (!isClosed()) " change introduces a new hot lock on closed
variable. Hopefully, it won't impact performance too much.
Looks good to me otherwise.
Regards,
Sean.
On 20/01/16 16:16, Mark Sheppard wrote:
> Hi,
> an update has been made to the webrev
>
> http://cr.openjdk.java.net/~msheppar/8144144/webrev.03/
>
> an anomaly was found in the select loop of the SelectorImpl.run() method
> some defensive programming, for selector null references, have been
> added, also.
>
> regards
> Mark
>
> On 08/01/2016 17:49, Mark Sheppard wrote:
>> Hi
>> please oblige and review the following changes
>> http://cr.openjdk.java.net/~msheppar/8144144/webrev/
>>
>> which addresses the issue
>> https://bugs.openjdk.java.net/browse/JDK-8144144
>>
>> the changes ensure that an Acceptor and its associated
>> ServerSocket/ServerSocketChannel
>> are closed.
>>
>> regards
>> Mark
>
More information about the core-libs-dev
mailing list