RFR: 8351458: (ch) Move preClose to UnixDispatcher

[Jaikiran Pai]

On Sun, 9 Mar 2025 09:10:15 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> Network channels in blocking mode, and the NIO based SocketImpl, have to deal with async close when there are threads blocked on the channel. Virtual threads blocked on the channel need to be unparked. On Unix systems, platform threads blocked on the channel require the file descriptor to be dup'ed to a special file descriptor and the threads signalled.
> 
> There is a bit of duplication in the implementation of the 5 channels, and in the SocketImpl. In addition, there is discussion on net-dev about an issue in AIX that will require allowing for signals to be queued (the current implementation does not require OS to support queuing of signals).
> 
> We refactor this code so that the "prepare for close" is in one place, UnixDispatcher.

src/java.base/share/classes/sun/nio/ch/NativeDispatcher.java line 82:

> 80:      */
> 81:     final void preClose(FileDescriptor fd, long reader, long writer) throws IOException {
> 82:         if (NativeThread.isVirtualThread(reader) || NativeThread.isVirtualThread(writer)) {

There used to be a `if (reader != 0 || writer != 0) {...}` check before doing any of this pre-close work, in `DatagramChannelImpl`. Is that no longer needed?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23956#discussion_r1988367022