[ViewVC] Diff of: cvs/libev/ev

Comparing libev/ev_epoll.c (file contents):
Revision 1.63 by root, Sat Jun 4 05:33:29 2011 UTC vs.
Revision 1.76 by root, Wed Jun 26 00:01:46 2019 UTC

 /*
  * libev epoll fd activity backend
  *
- * Copyright (c) 2007,2008,2009,2010,2011 Marc Alexander Lehmann <libev@schmorp.de>
+ * Copyright (c) 2007,2008,2009,2010,2011,2016,2017,2019 Marc Alexander Lehmann <libev@schmorp.de>
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without modifica-
  * tion, are permitted provided that the following conditions are met:
  *
       anfds [fd].emask = EV_EMASK_EPERM;
       /* add fd to epoll_eperms, if not already inside */
       if (!(oldmask & EV_EMASK_EPERM))
         {
-          array_needsize (int, epoll_eperms, epoll_epermmax, epoll_epermcnt + 1, EMPTY2);
+          array_needsize (int, epoll_eperms, epoll_epermmax, epoll_epermcnt + 1, array_needsize_noinit);
           epoll_eperms [epoll_epermcnt++] = fd;
         }
       return;
     }
+  else
+    assert (("libev: I/O watcher with invalid fd found in epoll_ctl", errno != EBADF && errno != ELOOP && errno != EINVAL));
   fd_kill (EV_A_ fd);
 dec_egen:
   /* we didn't successfully call epoll_ctl, so decrement the generation counter again */
       int fd = (uint32_t)ev->data.u64; /* mask out the lower 32 bits */
       int want = anfds [fd].events;
       int got  = (ev->events & (EPOLLOUT | EPOLLERR | EPOLLHUP) ? EV_WRITE : 0)
                | (ev->events & (EPOLLIN  | EPOLLERR | EPOLLHUP) ? EV_READ  : 0);
+      /*
-      /* check for spurious notification */
+       * check for spurious notification.
+       * this only finds spurious notifications on egen updates
+       * other spurious notifications will be found by epoll_ctl, below
-      /* we assume that fd is always in range, as we never shrink the anfds array */
+       * we assume that fd is always in range, as we never shrink the anfds array
+       */
       if (expect_false ((uint32_t)anfds [fd].egen != (uint32_t)(ev->data.u64 >> 32)))
         {
           /* recreate kernel state */
-          postfork = 1;
+          postfork |= 2;
           continue;
         }
       if (expect_false (got & ~want))
         {
           anfds [fd].emask = want;
+          /*
-          /* we received an event but are not interested in it, try mod or del */
+           * we received an event but are not interested in it, try mod or del
-          /* I don't think we ever need MOD, but let's handle it anyways */
+           * this often happens because we optimistically do not unregister fds
+           * when we are no longer interested in them, but also when we get spurious
+           * notifications for fds from another process. this is partially handled
+           * above with the gencounter check (== our fd is not the event fd), and
+           * partially here, when epoll_ctl returns an error (== a child has the fd
+           * but we closed it).
+           */
           ev->events = (want & EV_READ  ? EPOLLIN  : 0)
                      | (want & EV_WRITE ? EPOLLOUT : 0);
           /* pre-2.6.9 kernels require a non-null pointer with EPOLL_CTL_DEL, */
           /* which is fortunately easy to do for us. */
           if (epoll_ctl (backend_fd, want ? EPOLL_CTL_MOD : EPOLL_CTL_DEL, fd, ev))
             {
-              postfork = 1; /* an error occurred, recreate kernel state */
+              postfork |= 2; /* an error occurred, recreate kernel state */
               continue;
             }
         }
       fd_event (EV_A_ fd, got);
       unsigned char events = anfds [fd].events & (EV_READ | EV_WRITE);
       if (anfds [fd].emask & EV_EMASK_EPERM && events)
         fd_event (EV_A_ fd, events);
       else
+        {
-        epoll_eperms [i] = epoll_eperms [--epoll_epermcnt];
+          epoll_eperms [i] = epoll_eperms [--epoll_epermcnt];
+          anfds [fd].emask = 0;
+        }
+    }
+}
+static int
+epoll_epoll_create (void)
+{
+  int fd;
+#if defined EPOLL_CLOEXEC && !defined __ANDROID__
+  fd = epoll_create1 (EPOLL_CLOEXEC);
+  if (fd < 0 && (errno == EINVAL || errno == ENOSYS))
+#endif
-    }
+    {
-}
+      fd = epoll_create (256);
+      if (fd >= 0)
+        fcntl (fd, F_SETFD, FD_CLOEXEC);
+    }
+  return fd;
+}
-int inline_size
+inline_size
+int
 epoll_init (EV_P_ int flags)
 {
-#ifdef EPOLL_CLOEXEC
+  if ((backend_fd = epoll_epoll_create ()) < 0)
-  backend_fd = epoll_create1 (EPOLL_CLOEXEC);
-  if (backend_fd <= 0)
-#endif
-    backend_fd = epoll_create (256);
-  if (backend_fd < 0)
     return 0;
-  fcntl (backend_fd, F_SETFD, FD_CLOEXEC);
-  backend_mintime = 1./1024.; /* epoll does sometimes return early, this is just to avoid the worst */
+  backend_mintime = 1e-3; /* epoll does sometimes return early, this is just to avoid the worst */
   backend_modify  = epoll_modify;
   backend_poll    = epoll_poll;
   epoll_eventmax = 64; /* initial number of events receivable per poll */
   epoll_events = (struct epoll_event *)ev_malloc (sizeof (struct epoll_event) * epoll_eventmax);
   return EVBACKEND_EPOLL;
 }
-void inline_size
+inline_size
+void
 epoll_destroy (EV_P)
 {
   ev_free (epoll_events);
   array_free (epoll_eperm, EMPTY);
 }
-void inline_size
+inline_size
+void
 epoll_fork (EV_P)
 {
   close (backend_fd);
-  while ((backend_fd = epoll_create (256)) < 0)
+  while ((backend_fd = epoll_epoll_create ()) < 0)
     ev_syserr ("(libev) epoll_create");
-  fcntl (backend_fd, F_SETFD, FD_CLOEXEC);
   fd_rearm_all (EV_A);
 }

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing libev/ev_epoll.c (file contents): Revision 1.63 by root, Sat Jun 4 05:33:29 2011 UTC vs. Revision 1.76 by root, Wed Jun 26 00:01:46 2019 UTC

Diff Legend

Comparing libev/ev_epoll.c (file contents):
Revision 1.63 by root, Sat Jun 4 05:33:29 2011 UTC vs.
Revision 1.76 by root, Wed Jun 26 00:01:46 2019 UTC