Rename RCU CB context to 'cannot block' context
[akaros.git] / kern / src / alarm.c
index cf1fd70..168ceb9 100644 (file)
@@ -2,10 +2,9 @@
  * 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
@@ -19,6 +18,7 @@
 #include <alarm.h>
 #include <stdio.h>
 #include <smp.h>
+#include <kmalloc.h>
 
 /* Helper, resets the earliest/latest times, based on the elements of the list.
  * If the list is empty, we set the times to be the 12345 poison time.  Since
@@ -37,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,
-                      void (*set_interrupt) (uint64_t, struct timer_chain *))
+                      void (*set_interrupt)(struct timer_chain *))
 {
        spinlock_init_irqsave(&tchain->lock);
        TAILQ_INIT(&tchain->waiters);
@@ -45,16 +45,16 @@ void init_timer_chain(struct timer_chain *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. */
 void init_awaiter(struct alarm_waiter *waiter,
                   void (*func) (struct alarm_waiter *awaiter))
 {
-       waiter->wake_up_time = ALARM_POISON_TIME;
+       assert(func);
        waiter->func = func;
-       if (!func)
-               sem_init_irqsave(&waiter->sem, 0);
-       waiter->has_fired = FALSE;      /* so we can check this before arming */
+       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
@@ -94,77 +94,98 @@ static void reset_tchain_interrupt(struct timer_chain *tchain)
        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);
-               tchain->set_interrupt(tchain->earliest_time, tchain);
+               tchain->set_interrupt(tchain);
        }
 }
 
-/* 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)
+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)
 {
-       waiter->has_fired = TRUE;
-       cmb();  /* enforce the has_fired write before the handlers */
-       if (waiter->func)
-               waiter->func(waiter);
-       else
-               sem_up(&waiter->sem); /* IRQs are disabled, can call sem_up directly */
+       struct alarm_waiter *waiter = (struct alarm_waiter*)a0;
+
+       waiter->func(waiter);
+       __finish_awaiter(waiter);
+}
+
+static void wake_awaiter(struct alarm_waiter *waiter,
+                         struct hw_trapframe *hw_tf)
+{
+       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
  * everyone whose time is up.  Called from IRQ context. */
-void trigger_tchain(struct timer_chain *tchain)
+void __trigger_tchain(struct timer_chain *tchain, struct hw_trapframe *hw_tf)
 {
        struct alarm_waiter *i, *temp;
        uint64_t now = read_tsc();
-       bool changed_list = FALSE;
-       assert(!irq_is_enabled());
-       spin_lock(&tchain->lock);
+       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 */
-               if (i->wake_up_time <= now) {
-                       changed_list = TRUE;
-                       TAILQ_REMOVE(&tchain->waiters, i, next);
-                       /* 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);
-               } else {
+               if (i->wake_up_time > now)
                        break;
-               }
+               /* 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);
        }
-       if (changed_list) {
-               reset_tchain_times(tchain);
-       }
-       /* Need to reset the interrupt no matter what */
+       reset_tchain_times(tchain);
        reset_tchain_interrupt(tchain);
-       spin_unlock(&tchain->lock);
+       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);
+       }
 }
 
-/* Sets the alarm.  If it is a kthread-style alarm (func == 0), sleep on it
- * later.  This version assumes you have the lock held.  That only makes sense
- * from alarm handlers, which are called with this lock held from IRQ context */
-void __set_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
+/* Helper, inserts the waiter into the tchain, returning TRUE if we still need
+ * to reset the tchain interrupt.  Caller holds the lock. */
+static bool __insert_awaiter(struct timer_chain *tchain,
+                             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);
-       /* has_fired tells us if it is on the tchain or not */
-       waiter->has_fired = FALSE;
+
+       waiter->on_tchain = TRUE;
        /* Either the list is empty, or not. */
        if (TAILQ_EMPTY(&tchain->waiters)) {
                tchain->earliest_time = waiter->wake_up_time;
                tchain->latest_time = waiter->wake_up_time;
                TAILQ_INSERT_HEAD(&tchain->waiters, waiter, next);
                /* Need to turn on the timer interrupt later */
-               goto reset_out;
+               return TRUE;
        }
        /* If not, either we're first, last, or in the middle.  Reset the interrupt
         * and adjust the tchain's times accordingly. */
@@ -172,7 +193,7 @@ void __set_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
                tchain->earliest_time = waiter->wake_up_time;
                TAILQ_INSERT_HEAD(&tchain->waiters, waiter, next);
                /* Changed the first entry; we'll need to reset the interrupt later */
-               goto reset_out;
+               return TRUE;
        }
        /* If there is a tie for last, the newer one will really go last.  We need
         * to handle equality here since the loop later won't catch it. */
@@ -180,7 +201,7 @@ void __set_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
                tchain->latest_time = waiter->wake_up_time;
                /* Proactively put it at the end if we know we're last */
                TAILQ_INSERT_TAIL(&tchain->waiters, waiter, next);
-               goto no_reset_out;
+               return FALSE;
        }
        /* Insert before the first one you are earlier than.  This won't scale well
         * (TODO) if we have a lot of inserts.  The proactive insert_tail up above
@@ -188,43 +209,41 @@ void __set_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
        TAILQ_FOREACH_SAFE(i, &tchain->waiters, next, temp) {
                if (waiter->wake_up_time < i->wake_up_time) {
                        TAILQ_INSERT_BEFORE(i, waiter, next);
-                       goto no_reset_out;
+                       return FALSE;
                }
        }
        panic("Could not find a spot for awaiter %p\n", waiter);
-reset_out:
-       reset_tchain_interrupt(tchain);
-no_reset_out:
-       ;
-       /* TODO: could put some debug stuff here */
 }
 
-/* Sets the alarm.  Don't call this from an alarm handler, since you already
- * have the lock held.  Call __set_alarm() instead. */
+/* 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)
 {
+       assert(waiter->wake_up_time != ALARM_POISON_TIME);
+       assert(!waiter->on_tchain);
+
        spin_lock_irqsave(&tchain->lock);
-       __set_alarm(tchain, waiter);
+       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);
+               return;
+       }
+       if (__insert_awaiter(tchain, waiter))
+               reset_tchain_interrupt(tchain);
        spin_unlock_irqsave(&tchain->lock);
 }
 
-/* Removes waiter from the tchain before it goes off.  Returns TRUE if we
- * disarmed before the alarm went off, FALSE if it already fired. */
-bool unset_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
+/* Helper, rips the waiter from the tchain, knowing that it is on the list.
+ * Returns TRUE if the tchain interrupt needs to be reset.  Callers hold the
+ * lock. */
+static bool __remove_awaiter(struct timer_chain *tchain,
+                             struct alarm_waiter *waiter)
 {
        struct alarm_waiter *temp;
        bool reset_int = FALSE;         /* whether or not to reset the interrupt */
 
-       spin_lock_irqsave(&tchain->lock);
-       if (waiter->has_fired) {
-               /* the alarm has already gone off.  its not even on this tchain's list,
-                * though the concurrent change to has_fired (specifically, the setting
-                * of it to TRUE), happens under the tchain's lock.  As a side note, the
-                * code that sets it to FALSE is called when the waiter is on no chain,
-                * so there is no race on that. */
-               spin_unlock_irqsave(&tchain->lock);
-               return FALSE;
-       }
        /* 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) {
@@ -237,59 +256,102 @@ bool unset_alarm(struct timer_chain *tchain, struct alarm_waiter *waiter)
                tchain->latest_time = (temp) ? temp->wake_up_time : ALARM_POISON_TIME;
        }
        TAILQ_REMOVE(&tchain->waiters, waiter, next);
-       if (reset_int)
-               reset_tchain_interrupt(tchain);
-       spin_unlock_irqsave(&tchain->lock);
-       return TRUE;
+       waiter->on_tchain = FALSE;
+       return reset_int;
 }
 
-/* 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)
+/* Removes waiter from the tchain before it goes off.  Returns TRUE if we
+ * 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)
 {
        int8_t irq_state = 0;
-       if (waiter->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;
+
+       spin_lock_irqsave(&tchain->lock);
+       if (waiter->on_tchain) {
+               if (__remove_awaiter(tchain, waiter))
+                       reset_tchain_interrupt(tchain);
+               spin_unlock_irqsave(&tchain->lock);
+               return true;
+       }
+
+       /* 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);
+
+       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);
+
+       waiter->no_rearm = false;
+       return false;
+}
+
+bool reset_alarm_abs(struct timer_chain *tchain, struct alarm_waiter *waiter,
+                     uint64_t abs_time)
+{
+       bool ret;
+
+       ret = unset_alarm(tchain, waiter);
+       set_awaiter_abs(waiter, abs_time);
+       set_alarm(tchain, waiter);
+       return ret;
+}
+
+bool reset_alarm_rel(struct timer_chain *tchain, struct alarm_waiter *waiter,
+                     uint64_t usleep)
+{
+       bool ret;
+
+       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)
- *     - Make sure you don't clobber an old tchain here (a bug) 
+ *     - Make sure the interrupt handler calls __trigger_tchain(tchain)
+ *     - 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). */
-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 timer_chain *pcpui_tchain = &per_cpu_info[pcoreid].tchain;
+       struct per_cpu_info *rem_pcpui, *pcpui = &per_cpu_info[pcoreid];
+       struct timer_chain *pcpui_tchain = &pcpui->tchain;
 
        if (pcpui_tchain != tchain) {
                /* cross-core call.  we can simply send an alarm IRQ.  the alarm handler
                 * will reset its pcpu timer, based on its current lists.  they take an
                 * extra IRQ, but it gets the job done. */
+               rem_pcpui = (struct per_cpu_info*)((uintptr_t)tchain -
+                                   offsetof(struct per_cpu_info, tchain));
                /* TODO: using the LAPIC vector is a bit ghetto, since that's x86.  But
                 * RISCV ignores the vector field, and we don't have a global IRQ vector
                 * namespace or anything. */
-               send_ipi(pcoreid + (pcpui_tchain - tchain), LAPIC_TIMER_DEFAULT_VECTOR);
+               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. */
@@ -319,11 +381,10 @@ void print_chain(struct timer_chain *tchain)
               tchain->earliest_time,
               tchain->latest_time);
        TAILQ_FOREACH(i, &tchain->waiters, next) {
-               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);
 }
@@ -334,7 +395,7 @@ void print_pcpu_chains(void)
        struct timer_chain *pcpu_chain;
        printk("PCPU Chains:  It is now %llu\n", read_tsc());
 
-       for (int i = 0; i < num_cpus; i++) {
+       for (int i = 0; i < num_cores; i++) {
                pcpu_chain = &per_cpu_info[i].tchain;
                print_chain(pcpu_chain);
        }