Drop disconnect_socket blocking in NioPeerHandler for 0.0.99
[ldk-java] / src / main / java / org / ldk / batteries / NioPeerHandler.java
index f93d1652f6ec723f4201b37a14e8fee02f0743bd..eac4d8058a898fd7a8af0a61ddb8e2be68627f92 100644 (file)
@@ -1,17 +1,13 @@
 package org.ldk.batteries;
 
-import org.ldk.impl.bindings;
 import org.ldk.structs.*;
 
 import java.io.IOException;
 import java.net.SocketAddress;
 import java.net.StandardSocketOptions;
+import java.nio.Buffer;
 import java.nio.ByteBuffer;
-import java.nio.channels.SelectionKey;
-import java.nio.channels.Selector;
-import java.nio.channels.ServerSocketChannel;
-import java.nio.channels.SocketChannel;
-import java.util.concurrent.Callable;
+import java.nio.channels.*;
 
 /**
  * A NioPeerHandler maps LDK's PeerHandler to Java's NIO I/O interface. It spawns a single background thread which
@@ -20,11 +16,6 @@ import java.util.concurrent.Callable;
 public class NioPeerHandler {
     private static class Peer {
         SocketDescriptor descriptor;
-        // When we are told by LDK to disconnect, we can't return to LDK until we are sure
-        // won't call any more read/write PeerManager functions with the same connection.
-        // This is set to true if we're in such a condition (with disconnect checked
-        // before with the Peer monitor lock held) and false when we can return.
-        boolean block_disconnect_socket = false;
         SelectionKey key;
     }
 
@@ -91,13 +82,6 @@ public class NioPeerHandler {
                         peer.key.channel().close();
                     });
                 } catch (IOException ignored) { }
-                synchronized (peer) {
-                    while (peer.block_disconnect_socket) {
-                        try {
-                            peer.wait();
-                        } catch (InterruptedException ignored) { }
-                    }
-                }
             }
             @Override public boolean eq(SocketDescriptor other_arg) { return other_arg.hash() == our_id; }
             @Override public long hash() { return our_id; }
@@ -146,67 +130,66 @@ public class NioPeerHandler {
                 if (shutdown) return;
                 if (Thread.interrupted()) return;
                 for (SelectionKey key : this.selector.selectedKeys()) {
-                    if ((key.interestOps() & SelectionKey.OP_ACCEPT) != 0) {
-                        if (key.isAcceptable()) {
-                            SocketChannel chan;
-                            try {
-                                chan = ((ServerSocketChannel) key.channel()).accept();
-                            } catch (IOException ignored) {
-                                key.cancel();
-                                continue;
-                            }
-                            if (chan == null) continue;
-                            try {
-                                Peer peer = setup_socket(chan);
-                                Result_NonePeerHandleErrorZ res = this.peer_manager.new_inbound_connection(peer.descriptor);
-                                if (res instanceof  Result_NonePeerHandleErrorZ.Result_NonePeerHandleErrorZ_OK) {
-                                    peer.key = chan.register(this.selector, SelectionKey.OP_READ, peer);
-                                }
-                            } catch (IOException ignored) { }
-                        }
-                        continue; // There is no attachment so the rest of the loop is useless
-                    }
-                    Peer peer = (Peer) key.attachment();
-                    synchronized (peer) {
-                        peer.block_disconnect_socket = true;
-                    }
                     try {
-                        if (key.isValid() && (key.interestOps() & SelectionKey.OP_WRITE) != 0 && key.isWritable()) {
-                            Result_NonePeerHandleErrorZ res = this.peer_manager.write_buffer_space_avail(peer.descriptor);
-                            if (res instanceof Result_NonePeerHandleErrorZ.Result_NonePeerHandleErrorZ_Err) {
-                                key.channel().close();
-                                key.cancel();
+                        if ((key.interestOps() & SelectionKey.OP_ACCEPT) != 0) {
+                            if (key.isAcceptable()) {
+                                SocketChannel chan;
+                                try {
+                                    chan = ((ServerSocketChannel) key.channel()).accept();
+                                } catch (IOException ignored) {
+                                    key.cancel();
+                                    continue;
+                                }
+                                if (chan == null) continue;
+                                try {
+                                    Peer peer = setup_socket(chan);
+                                    Result_NonePeerHandleErrorZ res = this.peer_manager.new_inbound_connection(peer.descriptor);
+                                    if (res instanceof Result_NonePeerHandleErrorZ.Result_NonePeerHandleErrorZ_OK) {
+                                        peer.key = chan.register(this.selector, SelectionKey.OP_READ, peer);
+                                    }
+                                } catch (IOException ignored) { }
                             }
+                            continue; // There is no attachment so the rest of the loop is useless
                         }
-                        if (key.isValid() && (key.interestOps() & SelectionKey.OP_READ) != 0 && key.isReadable()) {
-                            buf.clear();
-                            int read = ((SocketChannel) key.channel()).read(buf);
-                            if (read == -1) {
-                                this.peer_manager.socket_disconnected(peer.descriptor);
-                                key.cancel();
-                            } else if (read > 0) {
-                                buf.flip();
-                                byte[] read_bytes = new byte[read];
-                                buf.get(read_bytes, 0, read);
-                                Result_boolPeerHandleErrorZ res = this.peer_manager.read_event(peer.descriptor, read_bytes);
-                                if (res instanceof Result_boolPeerHandleErrorZ.Result_boolPeerHandleErrorZ_OK) {
-                                    if (((Result_boolPeerHandleErrorZ.Result_boolPeerHandleErrorZ_OK) res).res) {
-                                        key.interestOps(key.interestOps() & (~SelectionKey.OP_READ));
-                                    }
-                                } else {
+                        Peer peer = (Peer) key.attachment();
+                        try {
+                            if (key.isValid() && (key.interestOps() & SelectionKey.OP_WRITE) != 0 && key.isWritable()) {
+                                Result_NonePeerHandleErrorZ res = this.peer_manager.write_buffer_space_avail(peer.descriptor);
+                                if (res instanceof Result_NonePeerHandleErrorZ.Result_NonePeerHandleErrorZ_Err) {
                                     key.channel().close();
                                     key.cancel();
                                 }
                             }
+                            if (key.isValid() && (key.interestOps() & SelectionKey.OP_READ) != 0 && key.isReadable()) {
+                                ((Buffer)buf).clear();
+                                int read = ((SocketChannel) key.channel()).read(buf);
+                                if (read == -1) {
+                                    this.peer_manager.socket_disconnected(peer.descriptor);
+                                    key.cancel();
+                                } else if (read > 0) {
+                                    ((Buffer)buf).flip();
+                                    byte[] read_bytes = new byte[read];
+                                    buf.get(read_bytes, 0, read);
+                                    Result_boolPeerHandleErrorZ res = this.peer_manager.read_event(peer.descriptor, read_bytes);
+                                    if (res instanceof Result_boolPeerHandleErrorZ.Result_boolPeerHandleErrorZ_OK) {
+                                        if (((Result_boolPeerHandleErrorZ.Result_boolPeerHandleErrorZ_OK) res).res) {
+                                            key.interestOps(key.interestOps() & (~SelectionKey.OP_READ));
+                                        }
+                                    } else {
+                                        key.channel().close();
+                                        key.cancel();
+                                    }
+                                }
+                            }
+                        } catch (IOException ignored) {
+                            try { key.channel().close(); } catch (IOException ignored2) { }
+                            key.cancel();
+                            peer_manager.socket_disconnected(peer.descriptor);
                         }
-                    } catch (IOException ignored) {
-                        try { key.channel().close(); } catch (IOException ignored2) { }
-                        key.cancel();
-                        peer_manager.socket_disconnected(peer.descriptor);
-                    }
-                    synchronized (peer) {
-                        peer.block_disconnect_socket = false;
-                        peer.notifyAll();
+                    } catch (CancelledKeyException e) {
+                        try { key.channel().close(); } catch (IOException ignored) { }
+                        // The key is only cancelled when we have notified the PeerManager that the socket is closed, so
+                        // no need to do anything here with the PeerManager.
                     }
                 }
                 if (lastTimerTick < System.currentTimeMillis() - 30 * 1000) {
@@ -286,4 +269,4 @@ public class NioPeerHandler {
     public void check_events() {
         selector.wakeup();
     }
-}
\ No newline at end of file
+}