rmeove remaining debugging code

[software/libev.git] / ev.c

diff --git a/ev.c b/ev.c
index 5df23b624723f75fdffa92fc13a8923a1130181e..aefc9dfc3da7c5b9453f9168721dfff56ccffafc 100644 (file)
--- a/ev.c
+++ b/ev.c
@@ -11,6 +11,8 @@
 #include <sys/time.h>
 #include <time.h>
 
 #include <sys/time.h>
 #include <time.h>
 

+#define HAVE_EPOLL 1
+

 #ifndef HAVE_MONOTONIC
 # ifdef CLOCK_MONOTONIC
 #  define HAVE_MONOTONIC 1
 #ifndef HAVE_MONOTONIC
 # ifdef CLOCK_MONOTONIC
 #  define HAVE_MONOTONIC 1


@@ -34,16 +36,9 @@
 
 #include "ev.h"
 
 
 #include "ev.h"
 

-struct ev_watcher {
-  EV_WATCHER (ev_watcher);
-};
-
-struct ev_watcher_list {
-  EV_WATCHER_LIST (ev_watcher_list);
-};
-

 typedef struct ev_watcher *W;
 typedef struct ev_watcher_list *WL;
 typedef struct ev_watcher *W;
 typedef struct ev_watcher_list *WL;

+typedef struct ev_watcher_time *WT;

 
 static ev_tstamp now, diff; /* monotonic clock */
 ev_tstamp ev_now;
 
 static ev_tstamp now, diff; /* monotonic clock */
 ev_tstamp ev_now;


@@ -90,7 +85,6 @@ get_clock (void)
   if ((cnt) > cur)                                     \
     {                                                  \
       int newcnt = cur ? cur << 1 : 16;                        \
   if ((cnt) > cur)                                     \
     {                                                  \
       int newcnt = cur ? cur << 1 : 16;                        \

-      fprintf (stderr, "resize(" # base ") from %d to %d\n", cur, newcnt);\

       base = realloc (base, sizeof (*base) * (newcnt));        \
       init (base + cur, newcnt - cur);                 \
       cur = newcnt;                                    \
       base = realloc (base, sizeof (*base) * (newcnt));        \
       init (base + cur, newcnt - cur);                 \
       cur = newcnt;                                    \


@@ -165,16 +159,16 @@ queue_events (W *events, int eventcnt, int type)
 
 /*****************************************************************************/
 
 
 /*****************************************************************************/
 

-static struct ev_timer **atimers;
-static int atimermax, atimercnt;
+static struct ev_timer **timers;
+static int timermax, timercnt;

 
 

-static struct ev_timer **rtimers;
-static int rtimermax, rtimercnt;
+static struct ev_periodic **periodics;
+static int periodicmax, periodiccnt;

 
 static void
 
 static void

-upheap (struct ev_timer **timers, int k)
+upheap (WT *timers, int k)

 {
 {

-  struct ev_timer *w = timers [k];
+  WT w = timers [k];

 
   while (k && timers [k >> 1]->at > w->at)
     {
 
   while (k && timers [k >> 1]->at > w->at)
     {


@@ -189,9 +183,9 @@ upheap (struct ev_timer **timers, int k)
 }
 
 static void
 }
 
 static void

-downheap (struct ev_timer **timers, int N, int k)
+downheap (WT *timers, int N, int k)

 {
 {

-  struct ev_timer *w = timers [k];
+  WT w = timers [k];

 
   while (k < (N >> 1))
     {
 
   while (k < (N >> 1))
     {


@@ -327,7 +321,7 @@ int ev_init (int flags)
 
   if (ev_method)
     {
 
   if (ev_method)
     {

-      evw_init (&sigev, sigcb, 0);
+      evw_init (&sigev, sigcb);

       siginit ();
     }
 
       siginit ();
     }
 


@@ -408,7 +402,7 @@ call_pending ()
 }
 
 static void
 }
 
 static void

-timers_reify (struct ev_timer **timers, int timercnt, ev_tstamp now)
+timers_reify ()

 {
   while (timercnt && timers [0]->at <= now)
     {
 {
   while (timercnt && timers [0]->at <= now)
     {


@@ -417,37 +411,75 @@ timers_reify (struct ev_timer **timers, int timercnt, ev_tstamp now)
       /* first reschedule or stop timer */
       if (w->repeat)
         {
       /* first reschedule or stop timer */
       if (w->repeat)
         {

-          if (w->is_abs)
-            w->at += floor ((now - w->at) / w->repeat + 1.) * w->repeat;
-          else
-            w->at = now + w->repeat;
-
-          assert (w->at > now);
-
-          downheap (timers, timercnt, 0);
+          w->at = now + w->repeat;
+          assert (("timer timeout in the past, negative repeat?", w->at > now));
+          downheap ((WT *)timers, timercnt, 0);

         }
       else
         }
       else

+        evtimer_stop (w); /* nonrepeating: stop timer */
+
+      event ((W)w, EV_TIMEOUT);
+    }
+}
+
+static void
+periodics_reify ()
+{
+  while (periodiccnt && periodics [0]->at <= ev_now)
+    {
+      struct ev_periodic *w = periodics [0];
+
+      /* first reschedule or stop timer */
+      if (w->interval)

         {
         {

-          evtimer_stop (w); /* nonrepeating: stop timer */
-          --timercnt; /* maybe pass by reference instead? */
+          w->at += floor ((ev_now - w->at) / w->interval + 1.) * w->interval;
+          assert (("periodic timeout in the past, negative interval?", w->at > ev_now));
+          downheap ((WT *)periodics, periodiccnt, 0);

         }
         }

+      else
+        evperiodic_stop (w); /* nonrepeating: stop timer */

 
       event ((W)w, EV_TIMEOUT);
     }
 }
 
 
       event ((W)w, EV_TIMEOUT);
     }
 }
 

+static void
+periodics_reschedule (ev_tstamp diff)
+{
+  int i;
+
+  /* adjust periodics after time jump */
+  for (i = 0; i < periodiccnt; ++i)
+    {
+      struct ev_periodic *w = periodics [i];
+
+      if (w->interval)
+        {
+          ev_tstamp diff = ceil ((ev_now - w->at) / w->interval) * w->interval;
+
+          if (fabs (diff) >= 1e-4)
+            {
+              evperiodic_stop (w);
+              evperiodic_start (w);
+
+              i = 0; /* restart loop, inefficient, but time jumps should be rare */
+            }
+        }
+    }
+}
+

 static void
 time_update ()
 {
   int i;
 static void
 time_update ()
 {
   int i;

+

   ev_now = ev_time ();
 
   if (have_monotonic)
     {
       ev_tstamp odiff = diff;
 
   ev_now = ev_time ();
 
   if (have_monotonic)
     {
       ev_tstamp odiff = diff;
 

-      /* detecting time jumps is much more difficult */
-      for (i = 2; --i; ) /* loop a few times, before making important decisions */
+      for (i = 4; --i; ) /* loop a few times, before making important decisions */

         {
           now = get_clock ();
           diff = ev_now - now;
         {
           now = get_clock ();
           diff = ev_now - now;


@@ -458,19 +490,19 @@ time_update ()
           ev_now = ev_time ();
         }
 
           ev_now = ev_time ();
         }
 

-      /* time jump detected, reschedule atimers */
-      for (i = 0; i < atimercnt; ++i)
-        {
-          struct ev_timer *w = atimers [i];
-          w->at += ceil ((ev_now - w->at) / w->repeat + 1.) * w->repeat;
-        }
+      periodics_reschedule (diff - odiff);
+      /* no timer adjustment, as the monotonic clock doesn't jump */

     }
   else
     {
       if (now > ev_now || now < ev_now - MAX_BLOCKTIME - MIN_TIMEJUMP)
     }
   else
     {
       if (now > ev_now || now < ev_now - MAX_BLOCKTIME - MIN_TIMEJUMP)

-        /* time jump detected, adjust rtimers */
-        for (i = 0; i < rtimercnt; ++i)
-          rtimers [i]->at += ev_now - now;
+        {
+          periodics_reschedule (ev_now - now);
+
+          /* adjust timers. this is easy, as the offset is the same for all */
+          for (i = 0; i < timercnt; ++i)
+            timers [i]->at += diff;
+        }

 
       now = ev_now;
     }
 
       now = ev_now;
     }


@@ -481,7 +513,7 @@ int ev_loop_done;
 void ev_loop (int flags)
 {
   double block;
 void ev_loop (int flags)
 {
   double block;

-  ev_loop_done = flags & EVLOOP_ONESHOT;
+  ev_loop_done = flags & EVLOOP_ONESHOT ? 1 : 0;

 
   if (checkcnt)
     {
 
   if (checkcnt)
     {


@@ -495,21 +527,25 @@ void ev_loop (int flags)
       fd_reify ();
 
       /* calculate blocking time */
       fd_reify ();
 
       /* calculate blocking time */

+
+      /* we only need this for !monotonic clock, but as we always have timers, we just calculate it every time */
+      ev_now = ev_time ();
+

       if (flags & EVLOOP_NONBLOCK || idlecnt)
         block = 0.;
       else
         {
           block = MAX_BLOCKTIME;
 
       if (flags & EVLOOP_NONBLOCK || idlecnt)
         block = 0.;
       else
         {
           block = MAX_BLOCKTIME;
 

-          if (rtimercnt)
+          if (timercnt)

             {
             {

-              ev_tstamp to = rtimers [0]->at - get_clock () + method_fudge;
+              ev_tstamp to = timers [0]->at - (have_monotonic ? get_clock () : ev_now) + method_fudge;

               if (block > to) block = to;
             }
 
               if (block > to) block = to;
             }
 

-          if (atimercnt)
+          if (periodiccnt)

             {
             {

-              ev_tstamp to = atimers [0]->at - ev_time   () + method_fudge;
+              ev_tstamp to = periodics [0]->at - ev_now + method_fudge;

               if (block > to) block = to;
             }
 
               if (block > to) block = to;
             }
 


@@ -522,10 +558,8 @@ void ev_loop (int flags)
       time_update ();
 
       /* queue pending timers and reschedule them */
       time_update ();
 
       /* queue pending timers and reschedule them */

-      /* absolute timers first */
-      timers_reify (atimers, atimercnt, ev_now);
-      /* relative timers second */
-      timers_reify (rtimers, rtimercnt, now);
+      periodics_reify (); /* absolute timers first */
+      timers_reify (); /* relative timers second */

 
       /* queue idle watchers unless io or timers are pending */
       if (!pendingcnt)
 
       /* queue idle watchers unless io or timers are pending */
       if (!pendingcnt)


@@ -537,6 +571,9 @@ void ev_loop (int flags)
       call_pending ();
     }
   while (!ev_loop_done);
       call_pending ();
     }
   while (!ev_loop_done);

+
+  if (ev_loop_done != 2)
+    ev_loop_done = 0;

 }
 
 /*****************************************************************************/
 }
 
 /*****************************************************************************/


@@ -612,33 +649,21 @@ evio_stop (struct ev_io *w)
   fdchanges [fdchangecnt - 1] = w->fd;
 }
 
   fdchanges [fdchangecnt - 1] = w->fd;
 }
 

+

 void
 evtimer_start (struct ev_timer *w)
 {
   if (ev_is_active (w))
     return;
 
 void
 evtimer_start (struct ev_timer *w)
 {
   if (ev_is_active (w))
     return;
 

-  if (w->is_abs)
-    {
-      /* this formula differs from the one in timer_reify becuse we do not round up */
-      if (w->repeat)
-        w->at += ceil ((ev_now - w->at) / w->repeat) * w->repeat;
-
-      ev_start ((W)w, ++atimercnt);
-      array_needsize (atimers, atimermax, atimercnt, );
-      atimers [atimercnt - 1] = w;
-      upheap (atimers, atimercnt - 1);
-    }
-  else
-    {
-      w->at += now;
+  w->at += now;

 
 

-      ev_start ((W)w, ++rtimercnt);
-      array_needsize (rtimers, rtimermax, rtimercnt, );
-      rtimers [rtimercnt - 1] = w;
-      upheap (rtimers, rtimercnt - 1);
-    }
+  assert (("timer repeat value less than zero not allowed", w->repeat >= 0.));

 
 

+  ev_start ((W)w, ++timercnt);
+  array_needsize (timers, timermax, timercnt, );
+  timers [timercnt - 1] = w;
+  upheap ((WT *)timers, timercnt - 1);

 }
 
 void
 }
 
 void


@@ -647,21 +672,62 @@ evtimer_stop (struct ev_timer *w)
   if (!ev_is_active (w))
     return;
 
   if (!ev_is_active (w))
     return;
 

-  if (w->is_abs)
+  if (w->active < timercnt--)

     {
     {

-      if (w->active < atimercnt--)
-        {
-          atimers [w->active - 1] = atimers [atimercnt];
-          downheap (atimers, atimercnt, w->active - 1);
-        }
+      timers [w->active - 1] = timers [timercnt];
+      downheap ((WT *)timers, timercnt, w->active - 1);

     }
     }

-  else
+
+  w->at = w->repeat;
+
+  ev_stop ((W)w);
+}
+
+void
+evtimer_again (struct ev_timer *w)
+{
+  if (ev_is_active (w))

     {
     {

-      if (w->active < rtimercnt--)
+      if (w->repeat)

         {
         {

-          rtimers [w->active - 1] = rtimers [rtimercnt];
-          downheap (rtimers, rtimercnt, w->active - 1);
+          w->at = now + w->repeat;
+          downheap ((WT *)timers, timercnt, w->active - 1);

         }
         }

+      else
+        evtimer_stop (w);
+    }
+  else if (w->repeat)
+    evtimer_start (w);
+}
+
+void
+evperiodic_start (struct ev_periodic *w)
+{
+  if (ev_is_active (w))
+    return;
+
+  assert (("periodic interval value less than zero not allowed", w->interval >= 0.));
+
+  /* this formula differs from the one in periodic_reify because we do not always round up */
+  if (w->interval)
+    w->at += ceil ((ev_now - w->at) / w->interval) * w->interval;
+
+  ev_start ((W)w, ++periodiccnt);
+  array_needsize (periodics, periodicmax, periodiccnt, );
+  periodics [periodiccnt - 1] = w;
+  upheap ((WT *)periodics, periodiccnt - 1);
+}
+
+void
+evperiodic_stop (struct ev_periodic *w)
+{
+  if (!ev_is_active (w))
+    return;
+
+  if (w->active < periodiccnt--)
+    {
+      periodics [w->active - 1] = periodics [periodiccnt];
+      downheap ((WT *)periodics, periodiccnt, w->active - 1);

     }
 
   ev_stop ((W)w);
     }
 
   ev_stop ((W)w);


@@ -736,6 +802,8 @@ void evcheck_stop (struct ev_check *w)
 
 #if 0
 
 
 #if 0
 

+struct ev_io wio;
+

 static void
 sin_cb (struct ev_io *w, int revents)
 {
 static void
 sin_cb (struct ev_io *w, int revents)
 {


@@ -754,23 +822,23 @@ static void
 scb (struct ev_signal *w, int revents)
 {
   fprintf (stderr, "signal %x,%d\n", revents, w->signum);
 scb (struct ev_signal *w, int revents)
 {
   fprintf (stderr, "signal %x,%d\n", revents, w->signum);

+  evio_stop (&wio);
+  evio_start (&wio);

 }
 
 static void
 gcb (struct ev_signal *w, int revents)
 {
   fprintf (stderr, "generic %x\n", revents);
 }
 
 static void
 gcb (struct ev_signal *w, int revents)
 {
   fprintf (stderr, "generic %x\n", revents);

+

 }
 
 int main (void)
 {
 }
 
 int main (void)
 {

-  struct ev_io sin;
-

   ev_init (0);
 
   ev_init (0);
 

-  evw_init (&sin, sin_cb, 55);
-  evio_set (&sin, 0, EV_READ);
-  evio_start (&sin);
+  evio_init (&wio, sin_cb, 0, EV_READ);
+  evio_start (&wio);

 
   struct ev_timer t[10000];
 
 
   struct ev_timer t[10000];
 


@@ -780,7 +848,7 @@ int main (void)
     {
       struct ev_timer *w = t + i;
       evw_init (w, ocb, i);
     {
       struct ev_timer *w = t + i;
       evw_init (w, ocb, i);

-      evtimer_set_abs (w, drand48 (), 0.99775533);
+      evtimer_init_abs (w, ocb, drand48 (), 0.99775533);

       evtimer_start (w);
       if (drand48 () < 0.5)
         evtimer_stop (w);
       evtimer_start (w);
       if (drand48 () < 0.5)
         evtimer_stop (w);


@@ -788,21 +856,19 @@ int main (void)
 #endif
 
   struct ev_timer t1;
 #endif
 
   struct ev_timer t1;

-  evw_init (&t1, ocb, 0);
-  evtimer_set_abs (&t1, 5, 10);
+  evtimer_init (&t1, ocb, 5, 10);

   evtimer_start (&t1);
 
   struct ev_signal sig;
   evtimer_start (&t1);
 
   struct ev_signal sig;

-  evw_init (&sig, scb, 65535);
-  evsignal_set (&sig, SIGQUIT);
+  evsignal_init (&sig, scb, SIGQUIT);

   evsignal_start (&sig);
 
   struct ev_check cw;
   evsignal_start (&sig);
 
   struct ev_check cw;

-  evw_init (&cw, gcb, 0);
+  evcheck_init (&cw, gcb);

   evcheck_start (&cw);
 
   struct ev_idle iw;
   evcheck_start (&cw);
 
   struct ev_idle iw;

-  evw_init (&iw, gcb, 0);
+  evidle_init (&iw, gcb);

   evidle_start (&iw);
 
   ev_loop (0);
   evidle_start (&iw);
 
   ev_loop (0);