Changeset 7ca5d3e

Timestamp:

Oct 27, 2009, 3:08:19 PM (15 years ago)

Author:

Alejandro R. Sedeño <asedeno@mit.edu>

Branches:

master, release-1.10, release-1.5, release-1.6, release-1.7, release-1.8, release-1.9

Children:

ad0dedd

Parents:

a01ed7c

git-author:

Alejandro R. Sedeño <asedeno@mit.edu> (10/27/09 15:02:36)

git-committer:

Alejandro R. Sedeño <asedeno@mit.edu> (10/27/09 15:08:19)

Message:

Remove obsolete dispatch simplifications

This should have been part of the I/O Dispatch API changeset that was
just merged, had I noticed it when ripping out the old API.

Signed-off-by: Alejandro R. Sedeño <asedeno@mit.edu>

File:

• select.c (modified) (1 diff)

select.c

@@ -458 +458 @@
 
   if(ret > 0) {
-    /* Merge fd_sets and clear AIM FDs. */
-    for(i = 0; i <= max_fd; i++) {
-      /* Merge all interesting FDs into one set, since we have a
-         single dispatch per FD. */
+    /* AIM HACK: process all AIM events at once. */
+    for(i = 0; !aim_done && i <= max_fd; i++) {
       if (FD_ISSET(i, &r) || FD_ISSET(i, &w) || FD_ISSET(i, &e)) {
-        /* AIM HACK: no separate dispatch, just process here if
-           needed, and only once per run through. */
-        if (!aim_done && (FD_ISSET(i, &aim_rfds) || FD_ISSET(i, &aim_wfds))) {
+        if (FD_ISSET(i, &aim_rfds) || FD_ISSET(i, &aim_wfds)) {
           owl_process_aim();
           aim_done = 1;
         }
-        else {
-          FD_SET(i, &r);
-        }
-      }
-    }
-    /* NOTE: the same dispatch function is called for both exceptional
-       and read ready FDs. */
+      }
+    }
     owl_select_io_dispatch(&r, &w, &e, max_fd);
   }