2002-11-29 Michael Koch <konqueror@gmx.de>
authorMichael Koch <konqueror@gmx.de>
Fri, 29 Nov 2002 09:57:05 +0000 (09:57 +0000)
committerMichael Koch <mkoch@gcc.gnu.org>
Fri, 29 Nov 2002 09:57:05 +0000 (09:57 +0000)
* gnu/java/nio/DatagramChannelImpl:
(fd): New member variable to store file descriptor of socket.
* gnu/java/nio/SelectionKeyImpl.java:
(ops): Removed.
(readyOps): New member variable.
(interestOps): New member variable.
(readyOps): Implemented.
(readyOps): New method to set member variable readyOps.
(interestOps): Replaced ops by interestOps.
* gnu/java/nio/SelectorImpl.java:
(SelectorImpl): Initialize key sets.
(select): Call select with -1 instead of Long.MAX_VALUE).
(java_do_select): Make it a native method.
(getFDsAsArray): New helper method.
(select): Remove canceled keys, give only interested file discriptors
to java_do_select, set ready ops.
(add): No need to initialize keys set here.
(add_selected): No need to initialize selected set here.
(deregisterCanceledKeys): New helper method.
(register): Set interest ops, set attachments, added handling of datagram
channels.
* gnu/java/nio/ServerSocketChannelImpl:
(SocketAccept): Renamed from NioSocketAccept.
(implConfigureBlocking): Implemented.
(accept): Use SocketAccept instead of NioSocketAccept.
* gnu/java/nio/SocketChannelImpl:
Reactivate native methods.

From-SVN: r59632

libjava/ChangeLog
libjava/gnu/java/nio/DatagramChannelImpl.java
libjava/gnu/java/nio/SelectionKeyImpl.java
libjava/gnu/java/nio/SelectorImpl.java
libjava/gnu/java/nio/ServerSocketChannelImpl.java
libjava/gnu/java/nio/SocketChannelImpl.java

index 9d80071ee62ba91a820a7e6c10e374db07ecdfaa..72d1abd8e2f9fe343d3fdb7d7010d5e5d6f72f68 100644 (file)
@@ -1,3 +1,33 @@
+2002-11-29  Michael Koch <konqueror@gmx.de>
+
+       * gnu/java/nio/DatagramChannelImpl:
+       (fd): New member variable to store file descriptor of socket.
+       * gnu/java/nio/SelectionKeyImpl.java:
+       (ops): Removed.
+       (readyOps): New member variable.
+       (interestOps): New member variable.
+       (readyOps): Implemented.
+       (readyOps): New method to set member variable readyOps.
+       (interestOps): Replaced ops by interestOps.
+       * gnu/java/nio/SelectorImpl.java:
+       (SelectorImpl): Initialize key sets.
+       (select): Call select with -1 instead of Long.MAX_VALUE).
+       (java_do_select): Make it a native method.
+       (getFDsAsArray): New helper method.
+       (select): Remove canceled keys, give only interested file discriptors
+       to java_do_select, set ready ops.
+       (add): No need to initialize keys set here.
+       (add_selected): No need to initialize selected set here.
+       (deregisterCanceledKeys): New helper method.
+       (register): Set interest ops, set attachments, added handling of datagram
+       channels.
+       * gnu/java/nio/ServerSocketChannelImpl:
+       (SocketAccept): Renamed from NioSocketAccept.
+       (implConfigureBlocking): Implemented.
+       (accept): Use SocketAccept instead of NioSocketAccept.  
+       * gnu/java/nio/SocketChannelImpl:
+       Reactivate native methods.
+       
 2002-11-29  Michael Koch <konqueror@gmx.de>
 
        * gnu/java/nio/natByteBufferImpl.cc,
index b192ef2367d54b7f09316876ffd425de41148505..a1e0be15912ee8fd6bfa68c5860ad6e32bd94a04 100644 (file)
@@ -45,6 +45,8 @@ import java.nio.channels.spi.SelectorProvider;
 
 public class DatagramChannelImpl extends DatagramChannel
 {
+  int fd;
+
   protected DatagramChannelImpl (SelectorProvider provider)
   {
     super (provider);
index 8ef6f7f27cb82bbb90935326d26933f530419693..5763d6880d3a6c5951e9e9382b25854a1e0a2856 100644 (file)
@@ -44,7 +44,9 @@ import java.nio.channels.spi.AbstractSelectionKey;
 
 public class SelectionKeyImpl extends AbstractSelectionKey
 {
-  int fd, ops;
+  int fd;
+  int readyOps;
+  int interestOps;
   SelectorImpl impl;
   SelectableChannel ch;
 
@@ -62,17 +64,23 @@ public class SelectionKeyImpl extends AbstractSelectionKey
 
   public int readyOps ()
   {
-    return 0;
+    return readyOps;
+  }
+
+  public SelectionKey readyOps (int ops)
+  {
+    readyOps = ops;
+    return this;
   }
 
   public int interestOps ()
   {
-    return ops;    
+    return interestOps;    
   }
 
   public SelectionKey interestOps (int ops)
   {
-    this.ops = ops;
+    interestOps = ops;
     return this;
   }
     
index 7d3d88c75f45387350d7c18e36183d6fe38597a3..0b513f9da5216ba276bc409975269030282d9d4f 100644 (file)
@@ -56,6 +56,10 @@ public class SelectorImpl extends AbstractSelector
   public SelectorImpl (SelectorProvider provider)
   {
     super (provider);
+    
+    keys = new HashSet ();
+    selected = new HashSet ();
+    canceled = new HashSet ();
   }
 
   public Set keys ()
@@ -70,16 +74,48 @@ public class SelectorImpl extends AbstractSelector
 
   public int select ()
   {
-    return select (Long.MAX_VALUE);
+    return select (-1);
   }
 
-//   private static native int java_do_select(int[] read, int[] write,
-//                                            int[] except, long timeout);
+  // A timeout value of -1 means block forever.
+  private static native int java_do_select (int[] read, int[] write,
+                                            int[] except, long timeout);
 
-  private static int java_do_select(int[] read, int[] write,
-                                    int[] except, long timeout)
+  private int[] getFDsAsArray (int ops)
   {
-    return 0;
+    int[] result;
+    int counter = 0;
+    Iterator it = keys.iterator ();
+
+    // Count the number of file descriptors needed
+    while (it.hasNext ())
+      {
+        SelectionKeyImpl key = (SelectionKeyImpl) it.next ();
+
+        if ((key.interestOps () & ops) != 0)
+          {
+            counter++;
+          }
+      }
+
+    result = new int[counter];
+
+    counter = 0;
+    it = keys.iterator ();
+
+    // Fill the array with the file descriptors
+    while (it.hasNext ())
+      {
+        SelectionKeyImpl key = (SelectionKeyImpl) it.next ();
+
+        if ((key.interestOps () & ops) != 0)
+          {
+            result[counter] = key.fd;
+            counter++;
+          }
+      }
+
+    return result;
   }
 
   public int select (long timeout)
@@ -94,40 +130,79 @@ public class SelectorImpl extends AbstractSelector
         return 0;
            }
 
-    int[] read = new int[keys.size ()];
-    int[] write = new int[keys.size ()];
-    int[] except = new int[keys.size ()];
-    int i = 0;
+    int ret = 0;
+
+    deregisterCanceledKeys ();
+
+    // Set only keys with the needed interest ops into the arrays.
+    int[] read = getFDsAsArray (SelectionKey.OP_READ | SelectionKey.OP_ACCEPT);
+    int[] write = getFDsAsArray (SelectionKey.OP_WRITE | SelectionKey.OP_CONNECT);
+    int[] except = new int [0]; // FIXME: We dont need to check this yet
+
+    // Call the native select () on all file descriptors.
+    int anzahl = read.length + write.length + except.length;
+    ret = java_do_select (read, write, except, timeout);
+
     Iterator it = keys.iterator ();
 
     while (it.hasNext ())
-           {
-        SelectionKeyImpl k = (SelectionKeyImpl) it.next ();
-        read[i] = k.fd;
-        write[i] = k.fd;
-        except[i] = k.fd;
-        i++;
-           }
+      {
+        int ops = 0;
+        SelectionKeyImpl key = (SelectionKeyImpl) it.next ();
 
-    int ret = java_do_select (read, write, except, timeout);
+        // If key is already selected retrieve old ready ops.
+        if (selected.contains (key))
+          {
+            ops = key.readyOps ();
+          }
 
-    i = 0;
-    it = keys.iterator ();
+        // Set new ready read/accept ops
+        for (int i = 0; i < read.length; i++)
+          {
+            if (key.fd == read[i])
+              {
+                if (key.channel () instanceof ServerSocketChannelImpl)
+                  {
+                    ops = ops | SelectionKey.OP_ACCEPT;
+                  }
+                else
+                  {
+                    ops = ops | SelectionKey.OP_READ;
+                  }
+              }
+          }
 
-    while (it.hasNext ())
-           {
-        SelectionKeyImpl k = (SelectionKeyImpl) it.next ();
+        // Set new ready write ops
+        for (int i = 0; i < write.length; i++)
+          {
+            if (key.fd == write[i])
+              {
+                ops = ops | SelectionKey.OP_WRITE;
+                
+//                 if (key.channel ().isConnected ())
+//                   {
+//                     ops = ops | SelectionKey.OP_WRITE;
+//                   }
+//                 else
+//                   {
+//                     ops = ops | SelectionKey.OP_CONNECT;
+//                   }
+             }
+          }
+
+        // FIXME: We dont handle exceptional file descriptors yet.
 
-        if (read[i] != -1 ||
-            write[i] != -1 ||
-            except[i] != -1)
+        // If key is not yet selected add it.
+        if (!selected.contains (key))
           {
-            add_selected (k);
+            add_selected (key);
           }
 
-        i++;
-           }
+        // Set new ready ops
+        key.readyOps (key.interestOps () & ops);
+      }
 
+    deregisterCanceledKeys ();
     return ret;
   }
     
@@ -143,25 +218,30 @@ public class SelectorImpl extends AbstractSelector
 
   public void add (SelectionKeyImpl k)
   {
-    if (keys == null)
-           keys = new HashSet ();
-
     keys.add (k);
   }
 
   void add_selected (SelectionKeyImpl k)
   {
-    if (selected == null)
-           selected = new HashSet ();
-
-    selected.add(k);
+    selected.add (k);
   }
 
   protected void implCloseSelector ()
   {
     closed = true;
   }
-    
+
+  private void deregisterCanceledKeys ()
+  {
+    Iterator it = canceled.iterator ();
+
+    while (it.hasNext ())
+      {
+        keys.remove ((SelectionKeyImpl) it.next ());
+        it.remove ();
+      }
+  }
+
   protected SelectionKey register (SelectableChannel ch, int ops, Object att)
   {
     return register ((AbstractSelectableChannel) ch, ops, att);
@@ -176,6 +256,8 @@ public class SelectorImpl extends AbstractSelector
 //         FileChannelImpl fc = (FileChannelImpl) ch;
 //         SelectionKeyImpl impl = new SelectionKeyImpl (ch, this, fc.fd);
 //         keys.add (impl);
+//         impl.interestOps (ops);
+//         impl.attach (att);
 //         return impl;
 //       }
 //     else
@@ -185,13 +267,26 @@ public class SelectorImpl extends AbstractSelector
         SocketChannelImpl sc = (SocketChannelImpl) ch;
         SelectionKeyImpl impl = new SelectionKeyImpl (ch, this, sc.fd);
         add (impl);
+        impl.interestOps (ops);
+        impl.attach (att);
         return impl;
            }
+    else if (ch instanceof DatagramChannelImpl)
+      {
+        DatagramChannelImpl dc = (DatagramChannelImpl) ch;
+        SelectionKeyImpl impl = new SelectionKeyImpl (ch, this, dc.fd);
+        add (impl);
+        impl.interestOps (ops);
+        impl.attach (att);
+        return impl;
+      }
     else if (ch instanceof ServerSocketChannelImpl)
       {
         ServerSocketChannelImpl ssc = (ServerSocketChannelImpl) ch;
         SelectionKeyImpl impl = new SelectionKeyImpl (ch, this, ssc.fd);
         add (impl);
+        impl.interestOps (ops);
+        impl.attach (att);
         return impl;
       }
     else
index e75c68d81668083b538601faf0f43d8163171f40..4a51f2a4b61b2d7aeb9d362964185efc7c33a3f5 100644 (file)
@@ -54,14 +54,8 @@ class ServerSocketChannelImpl extends ServerSocketChannel
   boolean connected = false;
 //   InetSocketAddress sa;
 
-//   private static native int NioSocketAccept (ServerSocketChannelImpl server, 
-//                                              SocketChannelImpl s);
-
-  private static int NioSocketAccept (ServerSocketChannelImpl server, 
-                                      SocketChannelImpl s)
-  {
-    return 0;
-  }
+  private static native int SocketAccept (ServerSocketChannelImpl server,
+                                          SocketChannelImpl s);
 
   protected ServerSocketChannelImpl (SelectorProvider provider)
   {
@@ -101,13 +95,14 @@ class ServerSocketChannelImpl extends ServerSocketChannel
 
   protected void implConfigureBlocking (boolean block)
   {
+    blocking = block;
   }
 
   public SocketChannel accept ()
   {
     SocketChannelImpl result = new SocketChannelImpl (provider ());
     result.sa = new InetSocketAddress (0);
-    int res = NioSocketAccept (this, result);
+    int res = SocketAccept (this,result);
     return result;
   }
 
index b43108ba4739e5736187bcdb60d922b35a739be1..9f2de278d811a26314ca0b9662786c880e06859c 100644 (file)
@@ -57,25 +57,14 @@ public class SocketChannelImpl extends SocketChannel
   boolean connected = false;
   InetSocketAddress sa;
 
-/*    
   static native int SocketCreate();
-  static native int SocketConnect(int fd, InetAddress a, int port);
-  static native int SocketBind(int fd, InetAddress host, int port);
+  static native int SocketConnect(int fd, InetAddress addr, int port);
+  static native int SocketBind(int fd, InetAddress addr, int port);
   static native int SocketListen(int fd, int backlog);
   static native int SocketAvailable(int fd);
   static native int SocketClose(int fd);
   static native int SocketRead(int fd, byte b[], int off, int len);
   static native int SocketWrite(int fd, byte b[], int off, int len);
-*/
-
-  static int SocketCreate() { return 0; };
-  static int SocketConnect(int fd, InetAddress a, int port) { return 0; };
-  static int SocketBind(int fd, InetAddress host, int port) { return 0; };
-  static int SocketListen(int fd, int backlog) { return 0; };
-  static int SocketAvailable(int fd) { return 0; };
-  static int SocketClose(int fd) { return 0; };
-  static int SocketRead(int fd, byte b[], int off, int len) { return 0; };
-  static int SocketWrite(int fd, byte b[], int off, int len) { return 0; };
 
   public SocketChannelImpl(SelectorProvider provider)                
   {
@@ -87,7 +76,7 @@ public class SocketChannelImpl extends SocketChannel
         System.err.println("failed to create socket:"+fd);
            }
   }
-    
+
   public void finalizer()
   {
     if (connected)