MCS locks have a "notification-safe" variant
[akaros.git] / user / pthread / pthread.c
index 0b782fd..1d31349 100644 (file)
@@ -76,10 +76,10 @@ struct uthread *pth_init(void)
        assert(t->id == 0);
 
        /* Put the new pthread on the active queue */
-       mcs_lock_lock(&queue_lock, &local_qn);
+       mcs_lock_notifsafe(&queue_lock, &local_qn);
        threads_active++;
        TAILQ_INSERT_TAIL(&active_queue, t, next);
-       mcs_lock_unlock(&queue_lock, &local_qn);
+       mcs_unlock_notifsafe(&queue_lock, &local_qn);
        return (struct uthread*)t;
 }
 
@@ -95,7 +95,7 @@ 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_lock(&queue_lock, &local_qn);
+       mcs_lock_notifsafe(&queue_lock, &local_qn);
        new_thread = TAILQ_FIRST(&ready_queue);
        if (new_thread) {
                TAILQ_REMOVE(&ready_queue, new_thread, next);
@@ -103,7 +103,7 @@ void __attribute__((noreturn)) pth_sched_entry(void)
                threads_active++;
                threads_ready--;
        }
-       mcs_lock_unlock(&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()) */
@@ -157,10 +157,10 @@ void pth_thread_runnable(struct uthread *uthread)
        struct mcs_lock_qnode local_qn = {0};
        /* Insert the newly created thread into the ready queue of threads.
         * It will be removed from this queue later when vcore_entry() comes up */
-       mcs_lock_lock(&queue_lock, &local_qn);
+       mcs_lock_notifsafe(&queue_lock, &local_qn);
        TAILQ_INSERT_TAIL(&ready_queue, pthread, next);
        threads_ready++;
-       mcs_lock_unlock(&queue_lock, &local_qn);
+       mcs_unlock_notifsafe(&queue_lock, &local_qn);
 }
 
 /* The calling thread is yielding.  Do what you need to do to restart (like put
@@ -173,12 +173,12 @@ void pth_thread_yield(struct uthread *uthread)
        /* Take from the active list, and put on the ready list (tail).  Don't do
         * this until we are done completely with the thread, since it can be
         * restarted somewhere else. */
-       mcs_lock_lock(&queue_lock, &local_qn);
+       mcs_lock_notifsafe(&queue_lock, &local_qn);
        threads_active--;
        TAILQ_REMOVE(&active_queue, pthread, next);
        threads_ready++;
        TAILQ_INSERT_TAIL(&ready_queue, pthread, next);
-       mcs_lock_unlock(&queue_lock, &local_qn);
+       mcs_unlock_notifsafe(&queue_lock, &local_qn);
 }
 
 /* Thread is exiting, do your 2LS specific stuff.  You're in vcore context.
@@ -188,10 +188,10 @@ void pth_thread_exit(struct uthread *uthread)
        struct pthread_tcb *pthread = (struct pthread_tcb*)uthread;
        struct mcs_lock_qnode local_qn = {0};
        /* Remove from the active runqueue */
-       mcs_lock_lock(&queue_lock, &local_qn);
+       mcs_lock_notifsafe(&queue_lock, &local_qn);
        threads_active--;
        TAILQ_REMOVE(&active_queue, pthread, next);
-       mcs_lock_unlock(&queue_lock, &local_qn);
+       mcs_unlock_notifsafe(&queue_lock, &local_qn);
        /* Cleanup, mirroring pth_thread_create() */
        __pthread_free_stack(pthread);
        /* TODO: race on detach state */