Removed event overflow handling
[akaros.git] / user / pthread / pthread.c
index 463660a..224cee9 100644 (file)
@@ -23,8 +23,9 @@ pthread_once_t init_once = PTHREAD_ONCE_INIT;
 int threads_ready = 0;
 int threads_active = 0;
 
-/* Array of syscall event queues, one per vcore, alloced in pth_init() */
-struct event_queue *sysc_evq;
+/* 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);
@@ -42,8 +43,7 @@ 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,
-                               bool overflow);
+static void pth_handle_syscall(struct event_msg *ev_msg, unsigned int ev_type);
 
 struct schedule_ops pthread_sched_ops = {
        pth_init,
@@ -79,22 +79,22 @@ struct uthread *pth_init(void)
 
        /* Handle syscall events.  Using small ev_qs, with no internal ev_mbox. */
        ev_handlers[EV_SYSCALL] = pth_handle_syscall;
-       sysc_evq = malloc(sizeof(struct event_queue) * max_vcores());
-       assert(sysc_evq);
-       /* 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). */
+       /* 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++) {
-               sysc_evq[i].ev_mbox =  &__procdata.vcore_preempt_data[i].ev_mbox;
-               sysc_evq[i].ev_flags = EVENT_IPI;               /* totally up to you */
-               sysc_evq[i].ev_vcore = 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++;
@@ -246,32 +246,27 @@ 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.
- *
- * TODO: ought to have a queue of waiters on syscalls, and this should remove it
- * from the list.  The list is needed in case we miss syscall events. */
+ * just make it runnable and let the main scheduler code handle it. */
 static void restart_thread(struct syscall *sysc)
 {
-       struct uthread *restartee = (struct uthread*)sysc->u_data;
-       assert(restartee);
-       assert(restartee->state == UT_BLOCKED);
-       assert(restartee->sysc == sysc);
-       restartee->sysc = 0;    /* so we don't 'reblock' on this later */
-       uthread_runnable(restartee);
+       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);
 }
 
 /* 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,
-                               bool overflow)
+static void pth_handle_syscall(struct event_msg *ev_msg, unsigned int ev_type)
 {
        struct syscall *sysc;
        assert(in_vcore_context());
-       /* TODO: handle overflow!! */
-       if (overflow)
-               printf("FUUUUUUUUUUUUUUUUCK, OVERFLOW!!!!!!!\n");
-       if (!ev_msg) /* just as bad as overflow */
-               return;
+       /* 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);
@@ -285,6 +280,7 @@ void pth_blockon_sysc(struct syscall *sysc)
 {
        int old_flags;
        bool need_to_restart = FALSE;
+       uint32_t vcoreid = vcore_id();
 
        assert(current_uthread->state == UT_BLOCKED);
        /* rip from the active queue */
@@ -295,14 +291,10 @@ void pth_blockon_sysc(struct syscall *sysc)
        TAILQ_REMOVE(&active_queue, pthread, next);
        mcs_unlock_notifsafe(&queue_lock, &local_qn);
 
-       /* TODO: need to register the sysc or uthread in case we lose the
-        * message.  can put it on a (per-core) tailq or something and rip it
-        * out when it unblocks. */
-
        /* Set things up so we can wake this thread up later */
        sysc->u_data = current_uthread;
        /* Register our vcore's syscall ev_q to hear about this syscall. */
-       if (!register_evq(sysc, &sysc_evq[vcore_id()])) {
+       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);
@@ -430,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;
 }
 
@@ -457,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)
@@ -465,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;
 }
 
@@ -510,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
@@ -576,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;
 }