Removed event overflow handling
[akaros.git] / user / pthread / pthread.c
index 94c754b..224cee9 100644 (file)
@@ -5,7 +5,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <assert.h>
-#include <rstdio.h>
+#include <stdio.h>
 #include <errno.h>
 #include <parlib.h>
 #include <ros/event.h>
@@ -23,6 +23,10 @@ pthread_once_t init_once = PTHREAD_ONCE_INIT;
 int threads_ready = 0;
 int threads_active = 0;
 
+/* Array of per-vcore structs to manage waiting on syscalls and handling
+ * overflow.  Init'd in pth_init(). */
+struct sysc_mgmt *sysc_mgmt = 0;
+
 /* Helper / local functions */
 static int get_next_pid(void);
 static inline void spin_to_sleep(unsigned int spins, unsigned int *spun);
@@ -34,11 +38,13 @@ struct uthread *pth_thread_create(void (*func)(void), void *udata);
 void pth_thread_runnable(struct uthread *uthread);
 void pth_thread_yield(struct uthread *uthread);
 void pth_thread_exit(struct uthread *uthread);
-unsigned int pth_vcores_wanted(void);
 void pth_preempt_pending(void);
 void pth_spawn_thread(uintptr_t pc_start, void *data);
 void pth_blockon_sysc(struct syscall *sysc);
 
+/* Event Handlers */
+static void pth_handle_syscall(struct event_msg *ev_msg, unsigned int ev_type);
+
 struct schedule_ops pthread_sched_ops = {
        pth_init,
        pth_sched_entry,
@@ -47,7 +53,6 @@ struct schedule_ops pthread_sched_ops = {
        pth_thread_yield,
        pth_thread_exit,
        pth_blockon_sysc,
-       pth_vcores_wanted,
        0, /* pth_preempt_pending, */
        0, /* pth_spawn_thread, */
 };
@@ -72,11 +77,24 @@ struct uthread *pth_init(void)
         * to use parts of event.c to do what you want. */
        enable_kevent(EV_USER_IPI, 0, EVENT_IPI);
 
+       /* Handle syscall events.  Using small ev_qs, with no internal ev_mbox. */
+       ev_handlers[EV_SYSCALL] = pth_handle_syscall;
+       /* Set up the per-vcore structs to track outstanding syscalls */
+       sysc_mgmt = malloc(sizeof(struct sysc_mgmt) * max_vcores());
+       assert(sysc_mgmt);
+       for (int i = 0; i < max_vcores(); i++) {
+               /* Set up each of the per-vcore syscall event queues so that they point
+                * to the VCPD/default vcore mailbox (for now)  Note you'll need the
+                * vcore to be online to get the events (for now). */
+               sysc_mgmt[i].ev_q.ev_mbox =  &__procdata.vcore_preempt_data[i].ev_mbox;
+               sysc_mgmt[i].ev_q.ev_flags = EVENT_IPI;         /* totally up to you */
+               sysc_mgmt[i].ev_q.ev_vcore = i;
+       }
        /* Create a pthread_tcb for the main thread */
        pthread_t t = (pthread_t)calloc(1, sizeof(struct pthread_tcb));
+       assert(t);
        t->id = get_next_pid();
        assert(t->id == 0);
-
        /* Put the new pthread on the active queue */
        mcs_lock_notifsafe(&queue_lock, &local_qn);
        threads_active++;
@@ -90,6 +108,7 @@ struct uthread *pth_init(void)
  * event.c (table of function pointers, stuff like that). */
 void __attribute__((noreturn)) pth_sched_entry(void)
 {
+       uint32_t vcoreid = vcore_id();
        if (current_uthread) {
                run_current_uthread();
                assert(0);
@@ -97,24 +116,35 @@ void __attribute__((noreturn)) pth_sched_entry(void)
        /* no one currently running, so lets get someone from the ready queue */
        struct pthread_tcb *new_thread = NULL;
        struct mcs_lock_qnode local_qn = {0};
-       mcs_lock_notifsafe(&queue_lock, &local_qn);
-       new_thread = TAILQ_FIRST(&ready_queue);
-       if (new_thread) {
-               TAILQ_REMOVE(&ready_queue, new_thread, next);
-               TAILQ_INSERT_TAIL(&active_queue, new_thread, next);
-               threads_active++;
-               threads_ready--;
+       /* For now, let's spin and handle events til we get a thread to run.  This
+        * will help catch races, instead of only having one core ever run a thread
+        * (if there is just one, etc).  Also, we don't need the EVENT_IPIs for this
+        * to work (since we poll handle_events() */
+       while (!new_thread) {
+               handle_events(vcoreid);
+               mcs_lock_notifsafe(&queue_lock, &local_qn);
+               new_thread = TAILQ_FIRST(&ready_queue);
+               if (new_thread) {
+                       TAILQ_REMOVE(&ready_queue, new_thread, next);
+                       TAILQ_INSERT_TAIL(&active_queue, new_thread, next);
+                       threads_active++;
+                       threads_ready--;
+               }
+               mcs_unlock_notifsafe(&queue_lock, &local_qn);
        }
-       mcs_unlock_notifsafe(&queue_lock, &local_qn);
        /* Instead of yielding, you could spin, turn off the core, set an alarm,
         * whatever.  You want some logic to decide this.  Uthread code wil have
         * helpers for this (like how we provide run_uthread()) */
        if (!new_thread) {
-               /* TODO: consider doing something more intelligent here */
+               /* Note, we currently don't get here (due to the while loop) */
                printd("[P] No threads, vcore %d is yielding\n", vcore_id());
-               sys_yield(0);
+               /* Not actually yielding - just spin for now, so we can get syscall
+                * unblocking events */
+               vcore_idle();
+               //sys_yield(0);
                assert(0);
        }
+       assert(((struct uthread*)new_thread)->state != UT_RUNNING);
        run_uthread((struct uthread*)new_thread);
        assert(0);
 }
@@ -132,6 +162,7 @@ struct uthread *pth_thread_create(void (*func)(void), void *udata)
        struct pthread_tcb *pthread;
        pthread_attr_t *attr = (pthread_attr_t*)udata;
        pthread = (pthread_t)calloc(1, sizeof(struct pthread_tcb));
+       assert(pthread);
        pthread->stacksize = PTHREAD_STACK_SIZE;        /* default */
        pthread->id = get_next_pid();
        pthread->detached = FALSE;                              /* default */
@@ -163,6 +194,9 @@ void pth_thread_runnable(struct uthread *uthread)
        TAILQ_INSERT_TAIL(&ready_queue, pthread, next);
        threads_ready++;
        mcs_unlock_notifsafe(&queue_lock, &local_qn);
+       /* Smarter schedulers should look at the num_vcores() and how much work is
+        * going on to make a decision about how many vcores to request. */
+       vcore_request(threads_ready);
 }
 
 /* The calling thread is yielding.  Do what you need to do to restart (like put
@@ -199,54 +233,73 @@ void pth_thread_exit(struct uthread *uthread)
        /* TODO: race on detach state */
        if (pthread->detached)
                free(pthread);
-       /* Once we do this, our joiner can free us.  He won't free us if we're
-        * detached, but there is still a potential race there (since he's accessing
-        * someone who is freed. */
-       pthread->finished = 1;
+       else
+               pthread->finished = 1;
 }
 
-/* Returns how many *more* vcores we want.  Smarter schedulers should look at
- * the num_vcores() and how much work is going on to make this decision. */
-unsigned int pth_vcores_wanted(void)
+void pth_preempt_pending(void)
 {
-       return 1;
 }
 
-void pth_preempt_pending(void)
+void pth_spawn_thread(uintptr_t pc_start, void *data)
 {
 }
 
-void pth_spawn_thread(uintptr_t pc_start, void *data)
+/* Restarts a uthread hanging off a syscall.  For the simple pthread case, we
+ * just make it runnable and let the main scheduler code handle it. */
+static void restart_thread(struct syscall *sysc)
 {
+       struct uthread *ut_restartee = (struct uthread*)sysc->u_data;
+       /* uthread stuff here: */
+       assert(ut_restartee);
+       assert(ut_restartee->state == UT_BLOCKED);
+       assert(ut_restartee->sysc == sysc);
+       ut_restartee->sysc = 0; /* so we don't 'reblock' on this later */
+       uthread_runnable(ut_restartee);
 }
 
-/* Eventually, this will be called from vcore context, after the current thread
- * has yielded and is trying to block on sysc. */
-void pth_blockon_sysc(struct syscall *sysc)
+/* This handler is usually run in vcore context, though I can imagine it being
+ * called by a uthread in some other threading library. */
+static void pth_handle_syscall(struct event_msg *ev_msg, unsigned int ev_type)
 {
-       printf("We tried to block on a syscall!\n");    
-       /* for now, just spin.  2LS stuff in later commits */
-       __ros_syscall_blockon(sysc);
+       struct syscall *sysc;
+       assert(in_vcore_context());
+       /* It's a bug if we don't have a msg (we're handling a syscall bit-event) */
+       assert(ev_msg);
+       /* Get the sysc from the message and just restart it */
+       sysc = ev_msg->ev_arg3;
+       assert(sysc);
+       restart_thread(sysc);
+}
 
-       #if 0
-       // testing shit.  should dont_migrate for this
+/* This will be called from vcore context, after the current thread has yielded
+ * and is trying to block on sysc.  Need to put it somewhere were we can wake it
+ * up when the sysc is done.  For now, we'll have the kernel send us an event
+ * when the syscall is done. */
+void pth_blockon_sysc(struct syscall *sysc)
+{
+       int old_flags;
+       bool need_to_restart = FALSE;
        uint32_t vcoreid = vcore_id();
-       struct event_queue local_ev_q = {0}, *ev_q = &local_ev_q;
-       ev_q->ev_mbox = &__procdata.vcore_preempt_data[vcoreid].ev_mbox;
-       ev_q->ev_flags = EVENT_IPI;
-       ev_q->ev_vcore = vcoreid;
+
+       assert(current_uthread->state == UT_BLOCKED);
+       /* rip from the active queue */
+       struct mcs_lock_qnode local_qn = {0};
+       struct pthread_tcb *pthread = (struct pthread_tcb*)current_uthread;
+       mcs_lock_notifsafe(&queue_lock, &local_qn);
+       threads_active--;
+       TAILQ_REMOVE(&active_queue, pthread, next);
+       mcs_unlock_notifsafe(&queue_lock, &local_qn);
+
+       /* Set things up so we can wake this thread up later */
        sysc->u_data = current_uthread;
-       wmb();
-       sysc->ev_q = ev_q;
-       if (sysc->flags & (SC_DONE | SC_PROGRESS)) {
-               // try and atomically swap out our u_data.  if we got it, we restart/
-               // handle it.  o/w, the receiver of the message restarts.  we can't just
-               // let them do it, since the kernel might have checked the ev_q before
-               // we wrote it (but after it wrote the flags).
-       } else {
-               //we're done
+       /* Register our vcore's syscall ev_q to hear about this syscall. */
+       if (!register_evq(sysc, &sysc_mgmt[vcoreid].ev_q)) {
+               /* Lost the race with the call being done.  The kernel won't send the
+                * event.  Just restart him. */
+               restart_thread(sysc);
        }
-       #endif
+       /* GIANT WARNING: do not touch the thread after this point. */
 }
 
 /* Pthread interface stuff and helpers */
@@ -265,7 +318,7 @@ int pthread_attr_destroy(pthread_attr_t *a)
 
 static void __pthread_free_stack(struct pthread_tcb *pt)
 {
-       assert(!munmap(pt->stacktop - PTHREAD_STACK_SIZE, PTHREAD_STACK_SIZE));
+       assert(!munmap(pt->stacktop - pt->stacksize, pt->stacksize));
 }
 
 static int __pthread_allocate_stack(struct pthread_tcb *pt)
@@ -369,7 +422,7 @@ int pthread_mutexattr_settype(pthread_mutexattr_t* attr, int type)
 int pthread_mutex_init(pthread_mutex_t* m, const pthread_mutexattr_t* attr)
 {
   m->attr = attr;
-  m->lock = 0;
+  atomic_init(&m->lock, 0);
   return 0;
 }
 
@@ -396,7 +449,7 @@ int pthread_mutex_lock(pthread_mutex_t* m)
 
 int pthread_mutex_trylock(pthread_mutex_t* m)
 {
-  return atomic_swap(&m->lock,1) == 0 ? 0 : EBUSY;
+  return atomic_swap(&m->lock, 1) == 0 ? 0 : EBUSY;
 }
 
 int pthread_mutex_unlock(pthread_mutex_t* m)
@@ -404,7 +457,7 @@ int pthread_mutex_unlock(pthread_mutex_t* m)
   /* Need to prevent the compiler (and some arches) from reordering older
    * stores */
   wmb();
-  m->lock = 0;
+  atomic_set(&m->lock, 0);
   return 0;
 }
 
@@ -449,11 +502,11 @@ int pthread_cond_signal(pthread_cond_t *c)
 
 int pthread_cond_wait(pthread_cond_t *c, pthread_mutex_t *m)
 {
-  int old_waiter = c->next_waiter;
-  int my_waiter = c->next_waiter;
+  uint32_t old_waiter = c->next_waiter;
+  uint32_t my_waiter = c->next_waiter;
   
   //allocate a slot
-  while (atomic_swap (& (c->in_use[my_waiter]), SLOT_IN_USE) == SLOT_IN_USE)
+  while (atomic_swap_u32(& (c->in_use[my_waiter]), SLOT_IN_USE) == SLOT_IN_USE)
   {
     my_waiter = (my_waiter + 1) % MAX_PTHREADS;
     assert (old_waiter != my_waiter);  // do not want to wrap around
@@ -515,7 +568,7 @@ void pthread_exit(void *ret)
 
 int pthread_once(pthread_once_t* once_control, void (*init_routine)(void))
 {
-  if(atomic_swap(once_control,1) == 0)
+  if (atomic_swap_u32(once_control, 1) == 0)
     init_routine();
   return 0;
 }
@@ -563,6 +616,7 @@ int pthread_barrier_destroy(pthread_barrier_t* b)
 
 int pthread_detach(pthread_t thread)
 {
+       /* TODO: race on this state.  Someone could be trying to join now */
        thread->detached = TRUE;
        return 0;
 }