[ViewVC] Diff of: cvs/libev/ev

Comparing libev/ev_linuxaio.c (file contents):
Revision 1.48 by root, Mon Jul 8 03:00:09 2019 UTC vs.
Revision 1.52 by root, Wed Aug 28 10:05:23 2019 UTC

        * this is not well documented, so we better do it.
        */
       memset (iocb, 0, sizeof (*iocb));
       iocb->io.aio_lio_opcode = IOCB_CMD_POLL;
-      iocb->io.aio_data       = offset;
       iocb->io.aio_fildes     = offset;
       base [offset++] = iocb;
     }
 }
 static void
 linuxaio_modify (EV_P_ int fd, int oev, int nev)
 {
   array_needsize (ANIOCBP, linuxaio_iocbps, linuxaio_iocbpmax, fd + 1, linuxaio_array_needsize_iocbp);
   ANIOCBP iocb = linuxaio_iocbps [fd];
+  ANFD *anfd = &anfds [fd];
   if (ecb_expect_false (iocb->io.aio_reqprio < 0))
     {
       /* we handed this fd over to epoll, so undo this first */
       /* we do it manually because the optimisations on epoll_modify won't do us any good */
       epoll_ctl (backend_fd, EPOLL_CTL_DEL, fd, 0);
-      anfds [fd].emask = 0;
+      anfd->emask = 0;
       iocb->io.aio_reqprio = 0;
     }
   else if (ecb_expect_false (iocb->io.aio_buf))
     {
       /* iocb active, so cancel it first before resubmit */
+      /* this assumes we only ever get one call per fd per loop iteration */
       for (;;)
         {
           /* on all relevant kernels, io_cancel fails with EINPROGRESS on "success" */
           if (ecb_expect_false (evsys_io_cancel (linuxaio_ctx, &iocb->io, (struct io_event *)0) == 0))
             break;
             {
               assert (("libev: linuxaio unexpected io_cancel failed", errno != EINTR && errno != EINPROGRESS));
               break;
             }
        }
+      /* increment generation counter to avoid handling old events */
+      ++anfd->egen;
     }
   iocb->io.aio_buf =
       (nev & EV_READ ? POLLIN : 0)
       | (nev & EV_WRITE ? POLLOUT : 0);
   if (nev)
     {
+      iocb->io.aio_data = (uint32_t)fd | ((__u64)(uint32_t)anfd->egen << 32);
       /* queue iocb up for io_submit */
       /* this assumes we only ever get one call per fd per loop iteration */
       ++linuxaio_submitcnt;
       array_needsize (struct iocb *, linuxaio_submits, linuxaio_submitmax, linuxaio_submitcnt, array_needsize_noinit);
       linuxaio_submits [linuxaio_submitcnt - 1] = &iocb->io;
 static void
 linuxaio_parse_events (EV_P_ struct io_event *ev, int nr)
 {
   while (nr)
     {
-      int fd  = ev->data;
+      int fd       = ev->data & 0xffffffff;
+      uint32_t gen = ev->data >> 32;
-      int res = ev->res;
+      int res      = ev->res;
       assert (("libev: iocb fd must be in-bounds", fd >= 0 && fd < anfdmax));
+      /* only accept events if generation counter matches */
+      if (ecb_expect_true (gen == (uint32_t)anfds [fd].egen))
+        {
-      /* feed events, we do not expect or handle POLLNVAL */
+          /* feed events, we do not expect or handle POLLNVAL */
-      fd_event (
+          fd_event (
-        EV_A_
+            EV_A_
-        fd,
+            fd,
-        (res & (POLLOUT | POLLERR | POLLHUP) ? EV_WRITE : 0)
+            (res & (POLLOUT | POLLERR | POLLHUP) ? EV_WRITE : 0)
-        | (res & (POLLIN | POLLERR | POLLHUP) ? EV_READ : 0)
+            | (res & (POLLIN | POLLERR | POLLHUP) ? EV_READ : 0)
-      );
+          );
-      /* linux aio is oneshot: rearm fd. TODO: this does more work than strictly needed */
+          /* linux aio is oneshot: rearm fd. TODO: this does more work than strictly needed */
-      linuxaio_fd_rearm (EV_A_ fd);
+          linuxaio_fd_rearm (EV_A_ fd);
+        }
       --nr;
       ++ev;
     }
 }
             break;
         }
       else
         break; /* no events from the kernel, we are done */
-      timeout = 0; /* only wait in the first iteration */
+      timeout = EV_TS_CONST (0.); /* only wait in the first iteration */
     }
 }
 inline_size
 int
                 backend        = EVBACKEND_EPOLL;
                 backend_modify = epoll_modify;
                 backend_poll   = epoll_poll;
               }
-            timeout = 0;
+            timeout = EV_TS_CONST (0.);
             /* it's easiest to handle this mess in another iteration */
             return;
           }
         else if (errno == EBADF)
           {
     {
       epoll_destroy (EV_A);
       return 0;
     }
-  ev_io_init  (EV_A_ &linuxaio_epoll_w, linuxaio_epoll_cb, backend_fd, EV_READ);
+  ev_io_init  (&linuxaio_epoll_w, linuxaio_epoll_cb, backend_fd, EV_READ);
   ev_set_priority (&linuxaio_epoll_w, EV_MAXPRI);
   ev_io_start (EV_A_ &linuxaio_epoll_w);
   ev_unref (EV_A); /* watcher should not keep loop alive */
   backend_modify = linuxaio_modify;

Diff Legend

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

Comparing libev/ev_linuxaio.c (file contents): Revision 1.48 by root, Mon Jul 8 03:00:09 2019 UTC vs. Revision 1.52 by root, Wed Aug 28 10:05:23 2019 UTC

Diff Legend

Comparing libev/ev_linuxaio.c (file contents):
Revision 1.48 by root, Mon Jul 8 03:00:09 2019 UTC vs.
Revision 1.52 by root, Wed Aug 28 10:05:23 2019 UTC