Rename RCU CB context to 'cannot block' context
[akaros.git] / kern / src / alarm.c
index a10ebfc..168ceb9 100644 (file)
@@ -2,10 +2,9 @@
  * Barret Rhoden <brho@cs.berkeley.edu>
  * See LICENSE for details.
  *
  * Barret Rhoden <brho@cs.berkeley.edu>
  * See LICENSE for details.
  *
- * Alarms.  This includes various ways to sleep for a while or defer work on a
- * specific timer.  These can be per-core, global or whatever.  Like with most
- * systems, you won't wake up til after the time you specify. (for now, this
- * might change).
+ * Alarms.  This includes ways to defer work on a specific timer.  These can be
+ * per-core, global or whatever.  Like with most systems, you won't wake up til
+ * after the time you specify. (for now, this might change).
  *
  * TODO:
  *     - have a kernel sense of time, instead of just the TSC or whatever timer the
  *
  * TODO:
  *     - have a kernel sense of time, instead of just the TSC or whatever timer the
@@ -38,7 +37,7 @@ static void reset_tchain_times(struct timer_chain *tchain)
 
 /* One time set up of a tchain, currently called in per_cpu_init() */
 void init_timer_chain(struct timer_chain *tchain,
 
 /* One time set up of a tchain, currently called in per_cpu_init() */
 void init_timer_chain(struct timer_chain *tchain,
-                      void (*set_interrupt) (uint64_t, struct timer_chain *))
+                      void (*set_interrupt)(struct timer_chain *))
 {
        spinlock_init_irqsave(&tchain->lock);
        TAILQ_INIT(&tchain->waiters);
 {
        spinlock_init_irqsave(&tchain->lock);
        TAILQ_INIT(&tchain->waiters);
@@ -46,34 +45,16 @@ void init_timer_chain(struct timer_chain *tchain,
        reset_tchain_times(tchain);
 }
 
        reset_tchain_times(tchain);
 }
 
-/* Initializes a new awaiter.  Pass 0 for the function if you want it to be a
- * kthread-alarm, and sleep on it after you set the alarm later. */
-static void __init_awaiter(struct alarm_waiter *waiter)
-{
-       waiter->wake_up_time = ALARM_POISON_TIME;
-       waiter->on_tchain = FALSE;
-       waiter->holds_tchain_lock = FALSE;
-       if (!waiter->has_func)
-               sem_init_irqsave(&waiter->sem, 0);
-}
-
 void init_awaiter(struct alarm_waiter *waiter,
                   void (*func) (struct alarm_waiter *awaiter))
 {
 void init_awaiter(struct alarm_waiter *waiter,
                   void (*func) (struct alarm_waiter *awaiter))
 {
-       waiter->irq_ok = FALSE;
-       waiter->has_func = func ? TRUE : FALSE;
-       waiter->func = func;                    /* if !func, this is a harmless zeroing */
-       __init_awaiter(waiter);
-}
-
-void init_awaiter_irq(struct alarm_waiter *waiter,
-                      void (*func_irq) (struct alarm_waiter *awaiter,
-                                        struct hw_trapframe *hw_tf))
-{
-       waiter->irq_ok = TRUE;
-       waiter->has_func = func_irq ? TRUE : FALSE;
-       waiter->func_irq = func_irq;    /* if !func, this is a harmless zeroing */
-       __init_awaiter(waiter);
+       assert(func);
+       waiter->func = func;
+       waiter->wake_up_time = ALARM_POISON_TIME;
+       waiter->on_tchain = false;
+       waiter->is_running = false;
+       waiter->no_rearm = false;
+       cv_init_irqsave(&waiter->done_cv);
 }
 
 /* Give this the absolute time.  For now, abs_time is the TSC time that you want
 }
 
 /* Give this the absolute time.  For now, abs_time is the TSC time that you want
@@ -113,39 +94,46 @@ static void reset_tchain_interrupt(struct timer_chain *tchain)
        if (TAILQ_EMPTY(&tchain->waiters)) {
                /* Turn it off */
                printd("Turning alarm off\n");
        if (TAILQ_EMPTY(&tchain->waiters)) {
                /* Turn it off */
                printd("Turning alarm off\n");
-               tchain->set_interrupt(0, tchain);
+               tchain->set_interrupt(tchain);
        } else {
                /* Make sure it is on and set to the earliest time */
                assert(tchain->earliest_time != ALARM_POISON_TIME);
                /* TODO: check for times in the past or very close to now */
                printd("Turning alarm on for %llu\n", tchain->earliest_time);
        } else {
                /* Make sure it is on and set to the earliest time */
                assert(tchain->earliest_time != ALARM_POISON_TIME);
                /* TODO: check for times in the past or very close to now */
                printd("Turning alarm on for %llu\n", tchain->earliest_time);
-               tchain->set_interrupt(tchain->earliest_time, tchain);
+               tchain->set_interrupt(tchain);
        }
 }
 
        }
 }
 
+static void __finish_awaiter(struct alarm_waiter *waiter)
+{
+       int8_t irq_state = 0;
+
+       /* Syncing with unset_alarm.  They are waiting for us to tell them the
+        * waiter is not running.
+        *
+        * 'is_running' is set true under the tchain lock.  It's checked and cleared
+        * under the cv_lock, but not necessarily with the tchain lock. */
+       cv_lock_irqsave(&waiter->done_cv, &irq_state);
+       waiter->is_running = false;
+       /* broadcast, instead of signal.  This allows us to have multiple unsetters
+        * concurrently.  (only one of which will succeed, so YMMV.) */
+       __cv_broadcast(&waiter->done_cv);
+       cv_unlock_irqsave(&waiter->done_cv, &irq_state);
+}
+
 static void __run_awaiter(uint32_t srcid, long a0, long a1, long a2)
 {
        struct alarm_waiter *waiter = (struct alarm_waiter*)a0;
 static void __run_awaiter(uint32_t srcid, long a0, long a1, long a2)
 {
        struct alarm_waiter *waiter = (struct alarm_waiter*)a0;
+
        waiter->func(waiter);
        waiter->func(waiter);
+       __finish_awaiter(waiter);
 }
 
 }
 
-/* When an awaiter's time has come, this gets called.  If it was a kthread, it
- * will wake up.  o/w, it will call the func ptr stored in the awaiter. */
 static void wake_awaiter(struct alarm_waiter *waiter,
                          struct hw_trapframe *hw_tf)
 {
 static void wake_awaiter(struct alarm_waiter *waiter,
                          struct hw_trapframe *hw_tf)
 {
-       if (waiter->has_func) {
-               if (waiter->irq_ok) {
-                       waiter->holds_tchain_lock = TRUE;
-                       waiter->func_irq(waiter, hw_tf);
-                       waiter->holds_tchain_lock = FALSE;
-               } else {
-                       send_kernel_message(core_id(), __run_awaiter, (long)waiter,
-                                           0, 0, KMSG_ROUTINE);
-               }
-       } else {
-               sem_up(&waiter->sem); /* IRQs are disabled, can call sem_up directly */
-       }
+       send_kernel_message(core_id(), __run_awaiter, (long)waiter,
+                           0, 0, KMSG_ROUTINE);
 }
 
 /* This is called when an interrupt triggers a tchain, and needs to wake up
 }
 
 /* This is called when an interrupt triggers a tchain, and needs to wake up
@@ -154,36 +142,33 @@ void __trigger_tchain(struct timer_chain *tchain, struct hw_trapframe *hw_tf)
 {
        struct alarm_waiter *i, *temp;
        uint64_t now = read_tsc();
 {
        struct alarm_waiter *i, *temp;
        uint64_t now = read_tsc();
-       bool changed_list = FALSE;
-       /* why do we disable irqs here?  the lock is irqsave, but we (think we) know
-        * the timer IRQ for this tchain won't fire again.  disabling irqs is nice
-        * for the lock debugger.  i don't want to disable the debugger completely,
-        * and we can't make the debugger ignore irq context code either in the
-        * general case.  it might be nice for handlers to have IRQs disabled too.*/
+       struct awaiters_tailq to_wake = TAILQ_HEAD_INITIALIZER(to_wake);
+
        spin_lock_irqsave(&tchain->lock);
        TAILQ_FOREACH_SAFE(i, &tchain->waiters, next, temp) {
                printd("Trying to wake up %p who is due at %llu and now is %llu\n",
                       i, i->wake_up_time, now);
                /* TODO: Could also do something in cases where we're close to now */
        spin_lock_irqsave(&tchain->lock);
        TAILQ_FOREACH_SAFE(i, &tchain->waiters, next, temp) {
                printd("Trying to wake up %p who is due at %llu and now is %llu\n",
                       i, i->wake_up_time, now);
                /* TODO: Could also do something in cases where we're close to now */
-               if (i->wake_up_time <= now) {
-                       changed_list = TRUE;
-                       i->on_tchain = FALSE;
-                       TAILQ_REMOVE(&tchain->waiters, i, next);
-                       cmb();  /* enforce waking after removal */
-                       /* Don't touch the waiter after waking it, since it could be in use
-                        * on another core (and the waiter can be clobbered as the kthread
-                        * unwinds its stack).  Or it could be kfreed */
-                       wake_awaiter(i, hw_tf);
-               } else {
+               if (i->wake_up_time > now)
                        break;
                        break;
-               }
-       }
-       if (changed_list) {
-               reset_tchain_times(tchain);
+               /* At this point, unset must wait until it has finished */
+               i->on_tchain = false;
+               i->is_running = true;
+               TAILQ_REMOVE(&tchain->waiters, i, next);
+               TAILQ_INSERT_TAIL(&to_wake, i, next);
        }
        }
-       /* Need to reset the interrupt no matter what */
+       reset_tchain_times(tchain);
        reset_tchain_interrupt(tchain);
        spin_unlock_irqsave(&tchain->lock);
        reset_tchain_interrupt(tchain);
        spin_unlock_irqsave(&tchain->lock);
+
+       TAILQ_FOREACH_SAFE(i, &to_wake, next, temp) {
+               /* Don't touch the waiter after waking it, since it could be in use on
+                * another core (and the waiter can be clobbered as the kthread unwinds
+                * its stack).  Or it could be kfreed.  Technically, the waiter hasn't
+                * finished until we cleared is_running and unlocked the cv lock. */
+               TAILQ_REMOVE(&to_wake, i, next);
+               wake_awaiter(i, hw_tf);
+       }
 }
 
 /* Helper, inserts the waiter into the tchain, returning TRUE if we still need
 }
 
 /* Helper, inserts the waiter into the tchain, returning TRUE if we still need
@@ -192,9 +177,7 @@ static bool __insert_awaiter(struct timer_chain *tchain,
                              struct alarm_waiter *waiter)
 {
        struct alarm_waiter *i, *temp;
                              struct alarm_waiter *waiter)
 {
        struct alarm_waiter *i, *temp;
-       /* This will fail if you don't set a time */
-       assert(waiter->wake_up_time != ALARM_POISON_TIME);
-       assert(!waiter->on_tchain);
+
        waiter->on_tchain = TRUE;
        /* Either the list is empty, or not. */
        if (TAILQ_EMPTY(&tchain->waiters)) {
        waiter->on_tchain = TRUE;
        /* Either the list is empty, or not. */
        if (TAILQ_EMPTY(&tchain->waiters)) {
@@ -232,23 +215,24 @@ static bool __insert_awaiter(struct timer_chain *tchain,
        panic("Could not find a spot for awaiter %p\n", waiter);
 }
 
        panic("Could not find a spot for awaiter %p\n", waiter);
 }
 
-static void __set_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
-{
-       if (__insert_awaiter(tchain, waiter))
-               reset_tchain_interrupt(tchain);
-}
-
 /* Sets the alarm.  If it is a kthread-style alarm (func == 0), sleep on it
  * later. */
 void set_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
 {
 /* Sets the alarm.  If it is a kthread-style alarm (func == 0), sleep on it
  * later. */
 void set_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
 {
-       if (waiter->holds_tchain_lock) {
-               __set_alarm(tchain, waiter);
-       } else {
-               spin_lock_irqsave(&tchain->lock);
-               __set_alarm(tchain, waiter);
+       assert(waiter->wake_up_time != ALARM_POISON_TIME);
+       assert(!waiter->on_tchain);
+
+       spin_lock_irqsave(&tchain->lock);
+       if (waiter->no_rearm) {
+               /* no_rearm exists to prevent alarm handlers from perpetually rearming
+                * when another thread is trying to unset the alarm.  We could return an
+                * error / false, but I don't have a use for that yet. */
                spin_unlock_irqsave(&tchain->lock);
                spin_unlock_irqsave(&tchain->lock);
+               return;
        }
        }
+       if (__insert_awaiter(tchain, waiter))
+               reset_tchain_interrupt(tchain);
+       spin_unlock_irqsave(&tchain->lock);
 }
 
 /* Helper, rips the waiter from the tchain, knowing that it is on the list.
 }
 
 /* Helper, rips the waiter from the tchain, knowing that it is on the list.
@@ -259,6 +243,7 @@ static bool __remove_awaiter(struct timer_chain *tchain,
 {
        struct alarm_waiter *temp;
        bool reset_int = FALSE;         /* whether or not to reset the interrupt */
 {
        struct alarm_waiter *temp;
        bool reset_int = FALSE;         /* whether or not to reset the interrupt */
+
        /* Need to make sure earliest and latest are set, in case we're mucking with
         * the first and/or last element of the chain. */
        if (TAILQ_FIRST(&tchain->waiters) == waiter) {
        /* Need to make sure earliest and latest are set, in case we're mucking with
         * the first and/or last element of the chain. */
        if (TAILQ_FIRST(&tchain->waiters) == waiter) {
@@ -276,63 +261,44 @@ static bool __remove_awaiter(struct timer_chain *tchain,
 }
 
 /* Removes waiter from the tchain before it goes off.  Returns TRUE if we
 }
 
 /* Removes waiter from the tchain before it goes off.  Returns TRUE if we
- * disarmed before the alarm went off, FALSE if it already fired. */
+ * disarmed before the alarm went off, FALSE if it already fired.  May block,
+ * since the handler may be running asynchronously. */
 bool unset_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
 {
 bool unset_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
 {
-       assert(!waiter->holds_tchain_lock);     /* Don't call from within a handler */
+       int8_t irq_state = 0;
+
        spin_lock_irqsave(&tchain->lock);
        spin_lock_irqsave(&tchain->lock);
-       bool ret = waiter->on_tchain;
-       if (ret && __remove_awaiter(tchain, waiter))
-               reset_tchain_interrupt(tchain);
+       if (waiter->on_tchain) {
+               if (__remove_awaiter(tchain, waiter))
+                       reset_tchain_interrupt(tchain);
+               spin_unlock_irqsave(&tchain->lock);
+               return true;
+       }
 
 
-       /* if alarm had already gone off then its not on this tchain's list, though
-        * the concurrent change to on_tchain (specifically, the setting of it to
-        * FALSE), happens under the tchain's lock. */
+       /* no_rearm is set and checked under the tchain lock.  It is cleared when
+        * unset completes, outside the lock.  That is safe since we know the alarm
+        * service is no longer aware of waiter (either the handler ran or we
+        * stopped it). */
+       waiter->no_rearm = true;
        spin_unlock_irqsave(&tchain->lock);
        spin_unlock_irqsave(&tchain->lock);
-       return ret;
-}
 
 
-/* waiter may be on the tchain, or it might have fired already and be off the
- * tchain.  Either way, this will put the waiter on the list, set to go off at
- * abs_time.  If you know the alarm has fired, don't call this.  Just set the
- * awaiter, and then set_alarm() */
-static bool __reset_alarm_abs(struct timer_chain *tchain,
-                              struct alarm_waiter *waiter, uint64_t abs_time)
-{
-       /* The tchain's lock is held */
-       bool ret = waiter->on_tchain;
-       /* We only need to remove/unset when the alarm has not fired yet (is still
-        * on the tchain).  If it has fired, it's like a fresh insert. We must also
-        * check if we need to reset the interrupt. */
-       bool reset_int = ret && __remove_awaiter(tchain, waiter);
-       set_awaiter_abs(waiter, abs_time);
-       /* regardless, we need to be reinserted */
-       if (__insert_awaiter(tchain, waiter) || reset_int)
-               reset_tchain_interrupt(tchain);
-       return ret;
-}
+       cv_lock_irqsave(&waiter->done_cv, &irq_state);
+       while (waiter->is_running)
+               cv_wait(&waiter->done_cv);
+       cv_unlock_irqsave(&waiter->done_cv, &irq_state);
 
 
-static bool __reset_alarm_rel(struct timer_chain *tchain,
-                              struct alarm_waiter *waiter, uint64_t usleep)
-{
-       uint64_t now, then;
-       now = read_tsc();
-       then = now + usec2tsc(usleep);
-       assert(now <= then);
-       return __reset_alarm_abs(tchain, waiter, then);
+       waiter->no_rearm = false;
+       return false;
 }
 
 bool reset_alarm_abs(struct timer_chain *tchain, struct alarm_waiter *waiter,
                      uint64_t abs_time)
 {
        bool ret;
 }
 
 bool reset_alarm_abs(struct timer_chain *tchain, struct alarm_waiter *waiter,
                      uint64_t abs_time)
 {
        bool ret;
-       if (waiter->holds_tchain_lock) {
-               ret = __reset_alarm_abs(tchain, waiter, abs_time);
-       } else {
-               spin_lock_irqsave(&tchain->lock);
-               ret = __reset_alarm_abs(tchain, waiter, abs_time);
-               spin_unlock_irqsave(&tchain->lock);
-       }
+
+       ret = unset_alarm(tchain, waiter);
+       set_awaiter_abs(waiter, abs_time);
+       set_alarm(tchain, waiter);
        return ret;
 }
 
        return ret;
 }
 
@@ -340,50 +306,35 @@ bool reset_alarm_rel(struct timer_chain *tchain, struct alarm_waiter *waiter,
                      uint64_t usleep)
 {
        bool ret;
                      uint64_t usleep)
 {
        bool ret;
-       if (waiter->holds_tchain_lock) {
-               ret =__reset_alarm_rel(tchain, waiter, usleep);
-       } else {
-               spin_lock_irqsave(&tchain->lock);
-               ret =__reset_alarm_rel(tchain, waiter, usleep);
-               spin_unlock_irqsave(&tchain->lock);
-       }
-       return ret;
-}
 
 
-/* Attempts to sleep on the alarm.  Could fail if you aren't allowed to kthread
- * (process limit, etc).  Don't call it on a waiter that is an event-handler. */
-int sleep_on_awaiter(struct alarm_waiter *waiter)
-{
-       int8_t irq_state = 0;
-       if (waiter->has_func)
-               panic("Tried blocking on a waiter %p with a func %p!", waiter,
-                     waiter->func);
-       /* Put the kthread to sleep.  TODO: This can fail (or at least it will be
-        * able to in the future) and we'll need to handle that. */
-       sem_down_irqsave(&waiter->sem, &irq_state);
-       return 0;
+       ret = unset_alarm(tchain, waiter);
+       set_awaiter_rel(waiter, usleep);
+       set_alarm(tchain, waiter);
+       return ret;
 }
 
 }
 
-/* Sets the Alarm interrupt, per-core style.  Also is an example of what any
- * similar function needs to do (this is the func ptr in the tchain). 
- * Note the tchain is our per-core one, and we don't need tchain passed to us to
- * figure that out.  It's kept around in case other tchain-usage wants it -
- * might not be necessary in the future.
+/* Sets the timer interrupt for the timer chain passed as parameter.
+ * The next interrupt will be scheduled at the nearest timer available in the
+ * chain.
+ * This function can be called either for the local CPU, or for a remote CPU.
+ * If called for the local CPU, it proceeds in setting up the local timer,
+ * otherwise it will trigger an IPI, and will let the remote CPU IRQ handler
+ * to setup the timer according to the active information on its timer chain.
  *
  * Needs to set the interrupt to trigger tchain at the given time, or disarm it
  * if time is 0.   Any function like this needs to do a few things:
  *     - Make sure the interrupt is on and will go off when we want
  *     - Make sure the interrupt source can find tchain
  *     - Make sure the interrupt handler calls __trigger_tchain(tchain)
  *
  * Needs to set the interrupt to trigger tchain at the given time, or disarm it
  * if time is 0.   Any function like this needs to do a few things:
  *     - Make sure the interrupt is on and will go off when we want
  *     - Make sure the interrupt source can find tchain
  *     - Make sure the interrupt handler calls __trigger_tchain(tchain)
- *     - Make sure you don't clobber an old tchain here (a bug) 
+ *     - Make sure you don't clobber an old tchain here (a bug)
  * This implies the function knows how to find its timer source/void
  *
  * Called with the tchain lock held, and IRQs disabled.  However, we could be
  * calling this cross-core, and we cannot disable those IRQs (hence the
  * locking). */
  * This implies the function knows how to find its timer source/void
  *
  * Called with the tchain lock held, and IRQs disabled.  However, we could be
  * calling this cross-core, and we cannot disable those IRQs (hence the
  * locking). */
-void set_pcpu_alarm_interrupt(uint64_t time, struct timer_chain *tchain)
+void set_pcpu_alarm_interrupt(struct timer_chain *tchain)
 {
 {
-       uint64_t rel_usec, now;
+       uint64_t time, rel_usec, now;
        int pcoreid = core_id();
        struct per_cpu_info *rem_pcpui, *pcpui = &per_cpu_info[pcoreid];
        struct timer_chain *pcpui_tchain = &pcpui->tchain;
        int pcoreid = core_id();
        struct per_cpu_info *rem_pcpui, *pcpui = &per_cpu_info[pcoreid];
        struct timer_chain *pcpui_tchain = &pcpui->tchain;
@@ -400,6 +351,7 @@ void set_pcpu_alarm_interrupt(uint64_t time, struct timer_chain *tchain)
                send_ipi(rem_pcpui - &per_cpu_info[0], IdtLAPIC_TIMER);
                return;
        }
                send_ipi(rem_pcpui - &per_cpu_info[0], IdtLAPIC_TIMER);
                return;
        }
+       time = TAILQ_EMPTY(&tchain->waiters) ? 0 : tchain->earliest_time;
        if (time) {
                /* Arm the alarm.  For times in the past, we just need to make sure it
                 * goes off. */
        if (time) {
                /* Arm the alarm.  For times in the past, we just need to make sure it
                 * goes off. */
@@ -429,23 +381,10 @@ void print_chain(struct timer_chain *tchain)
               tchain->earliest_time,
               tchain->latest_time);
        TAILQ_FOREACH(i, &tchain->waiters, next) {
               tchain->earliest_time,
               tchain->latest_time);
        TAILQ_FOREACH(i, &tchain->waiters, next) {
-               if (i->has_func) {
-                       uintptr_t f;
-                       if (i->irq_ok)
-                               f = (uintptr_t)i->func_irq;
-                       else
-                               f = (uintptr_t)i->func;
-                       char *f_name = get_fn_name(f);
-                       printk("\tWaiter %p, time %llu, func %p (%s)\n", i,
-                              i->wake_up_time, f, f_name);
-                       kfree(f_name);
-                       continue;
-               }
-               struct kthread *kthread = TAILQ_FIRST(&i->sem.waiters);
-               printk("\tWaiter %p, time: %llu, kthread: %p (%p) %s\n", i,
-                      i->wake_up_time, kthread, (kthread ? kthread->proc : 0),
-                      (kthread ? kthread->name : 0));
+               uintptr_t f = (uintptr_t)i->func;
 
 
+               printk("\tWaiter %p, time %llu, func %p (%s)\n", i,
+                      i->wake_up_time, f, get_fn_name(f));
        }
        spin_unlock_irqsave(&tchain->lock);
 }
        }
        spin_unlock_irqsave(&tchain->lock);
 }