avoid backend_modify call unless ev_io_set was used

[software/libev.git] / ev_epoll.c

diff --git a/ev_epoll.c b/ev_epoll.c
index b38011ee9e9a4304b7ef3774958849da9290d5d3..bb56218d54ec7bd2056edebbe70fb25e663c0b0c 100644 (file)
--- a/ev_epoll.c
+++ b/ev_epoll.c
@@ -42,9 +42,9 @@ epoll_modify (EV_P_ int fd, int oev, int nev)
       (nev & EV_READ ? EPOLLIN : 0)
       | (nev & EV_WRITE ? EPOLLOUT : 0);
 
       (nev & EV_READ ? EPOLLIN : 0)
       | (nev & EV_WRITE ? EPOLLOUT : 0);
 

-  if (epoll_ctl (epoll_fd, mode, fd, &ev))
+  if (expect_false (epoll_ctl (backend_fd, mode, fd, &ev)))

     if (errno != ENOENT /* on ENOENT the fd went away, so try to do the right thing */
     if (errno != ENOENT /* on ENOENT the fd went away, so try to do the right thing */

-        || (nev && epoll_ctl (epoll_fd, EPOLL_CTL_ADD, fd, &ev)))
+        || (nev && epoll_ctl (backend_fd, EPOLL_CTL_ADD, fd, &ev)))

       fd_kill (EV_A_ fd);
 }
 
       fd_kill (EV_A_ fd);
 }
 


@@ -52,9 +52,9 @@ static void
 epoll_poll (EV_P_ ev_tstamp timeout)
 {
   int i;
 epoll_poll (EV_P_ ev_tstamp timeout)
 {
   int i;

-  int eventcnt = epoll_wait (epoll_fd, epoll_events, epoll_eventmax, (int)ceil (timeout * 1000.));
+  int eventcnt = epoll_wait (backend_fd, epoll_events, epoll_eventmax, (int)ceil (timeout * 1000.));

 
 

-  if (eventcnt < 0)
+  if (expect_false (eventcnt < 0))

     {
       if (errno != EINTR)
         syserr ("(libev) epoll_wait");
     {
       if (errno != EINTR)
         syserr ("(libev) epoll_wait");


@@ -74,24 +74,24 @@ epoll_poll (EV_P_ ev_tstamp timeout)
   if (expect_false (eventcnt == epoll_eventmax))
     {
       ev_free (epoll_events);
   if (expect_false (eventcnt == epoll_eventmax))
     {
       ev_free (epoll_events);

-      epoll_eventmax = array_roundsize (epoll_events, epoll_eventmax << 1);
+      epoll_eventmax = array_nextsize (sizeof (struct epoll_event), epoll_eventmax, epoll_eventmax + 1);

       epoll_events = (struct epoll_event *)ev_malloc (sizeof (struct epoll_event) * epoll_eventmax);
     }
 }
 
       epoll_events = (struct epoll_event *)ev_malloc (sizeof (struct epoll_event) * epoll_eventmax);
     }
 }
 

-static int
+int inline_size

 epoll_init (EV_P_ int flags)
 {
 epoll_init (EV_P_ int flags)
 {

-  epoll_fd = epoll_create (256);
+  backend_fd = epoll_create (256);

 
 

-  if (epoll_fd < 0)
+  if (backend_fd < 0)

     return 0;
 
     return 0;
 

-  fcntl (epoll_fd, F_SETFD, FD_CLOEXEC);
+  fcntl (backend_fd, F_SETFD, FD_CLOEXEC);

 
 

-  method_fudge  = 1e-3; /* needed to compensate for epoll returning early */
-  method_modify = epoll_modify;
-  method_poll   = epoll_poll;
+  backend_fudge  = 0.; /* kernel sources seem to indicate this to be zero */
+  backend_modify = epoll_modify;
+  backend_poll   = epoll_poll;

 
   epoll_eventmax = 64; /* intiial number of events receivable per poll */
   epoll_events = (struct epoll_event *)ev_malloc (sizeof (struct epoll_event) * epoll_eventmax);
 
   epoll_eventmax = 64; /* intiial number of events receivable per poll */
   epoll_events = (struct epoll_event *)ev_malloc (sizeof (struct epoll_event) * epoll_eventmax);


@@ -99,23 +99,21 @@ epoll_init (EV_P_ int flags)
   return EVBACKEND_EPOLL;
 }
 
   return EVBACKEND_EPOLL;
 }
 

-static void
+void inline_size

 epoll_destroy (EV_P)
 {
 epoll_destroy (EV_P)
 {

-  close (epoll_fd);
-

   ev_free (epoll_events);
 }
 
   ev_free (epoll_events);
 }
 

-static void
+void inline_size

 epoll_fork (EV_P)
 {
 epoll_fork (EV_P)
 {

-  close (epoll_fd);
+  close (backend_fd);

 
 

-  while ((epoll_fd = epoll_create (256)) < 0)
+  while ((backend_fd = epoll_create (256)) < 0)

     syserr ("(libev) epoll_create");
 
     syserr ("(libev) epoll_create");
 

-  fcntl (epoll_fd, F_SETFD, FD_CLOEXEC);
+  fcntl (backend_fd, F_SETFD, FD_CLOEXEC);

 
   fd_rearm_all (EV_A);
 }
 
   fd_rearm_all (EV_A);
 }