Fixes bug with run_current_uthread()
[akaros.git] / user / parlib / ucq.c
index 1e58500..dc2abfe 100644 (file)
@@ -14,6 +14,7 @@
 #include <assert.h>
 #include <stdio.h>
 #include <rassert.h> /* for the static_assert() */
+#include <vcore.h>
 
 /* Initializes a ucq.  You pass in addresses of mmaped pages for the main page
  * (prod_idx) and the spare page.  I recommend mmaping a big chunk and breaking
@@ -29,8 +30,8 @@ void ucq_init_raw(struct ucq *ucq, uintptr_t pg1, uintptr_t pg2)
        ucq->prod_overflow = FALSE;
        atomic_set(&ucq->nr_extra_pgs, 0);
        atomic_set(&ucq->spare_pg, pg2);
-       static_assert(sizeof(struct mcs_lock) <= sizeof(ucq->u_lock));
-       mcs_lock_init((struct mcs_lock*)(&ucq->u_lock));
+       static_assert(sizeof(struct mcs_pdr_lock) <= sizeof(ucq->u_lock));
+       mcs_pdr_init((struct mcs_pdr_lock*)(&ucq->u_lock));
        ucq->ucq_ready = TRUE;
 }
 
@@ -55,6 +56,7 @@ void ucq_free_pgs(struct ucq *ucq)
        assert(pg1 && pg2);
        munmap((void*)pg1, PGSIZE);
        munmap((void*)pg2, PGSIZE);
+       mcs_pdr_fini((struct mcs_pdr_lock*)&ucq->u_lock);
 }
 
 /* Consumer side, returns 0 on success and fills *msg with the ev_msg.  If the
@@ -66,8 +68,7 @@ int get_ucq_msg(struct ucq *ucq, struct event_msg *msg)
        struct msg_container *my_msg;
        /* Locking stuff.  Would be better with a spinlock, if we had them, since
         * this should be lightly contested.  */
-       struct mcs_lock_qnode local_qn = {0};
-       struct mcs_lock *ucq_lock = (struct mcs_lock*)(&ucq->u_lock);
+       struct mcs_pdr_lock *ucq_lock = (struct mcs_pdr_lock*)(&ucq->u_lock);
 
        do {
 loop_top:
@@ -83,13 +84,13 @@ loop_top:
                if (slot_is_good(my_idx))
                        goto claim_slot;
                /* Slot is bad, let's try and fix it */
-               mcs_lock_notifsafe(ucq_lock, &local_qn);
+               mcs_pdr_lock(ucq_lock);
                /* Reread the idx, in case someone else fixed things up while we
                 * were waiting/fighting for the lock */
                my_idx = atomic_read(&ucq->cons_idx);
                if (slot_is_good(my_idx)) {
                        /* Someone else fixed it already, let's just try to get out */
-                       mcs_unlock_notifsafe(ucq_lock, &local_qn);
+                       mcs_pdr_unlock(ucq_lock);
                        /* Make sure this new slot has a producer (ucq isn't empty) */
                        if (my_idx == atomic_read(&ucq->prod_idx))
                                return -1;
@@ -111,8 +112,12 @@ loop_top:
                 * slots based off the new counter index (cons_idx) */
                /* Now free up the old page.  Need to make sure all other consumers are
                 * done.  We spin til enough are done, like an inverted refcnt. */
-               while (atomic_read(&old_page->header.nr_cons) < NR_MSG_PER_PAGE)
-                       cpu_relax();
+               while (atomic_read(&old_page->header.nr_cons) < NR_MSG_PER_PAGE) {
+                       /* spinning on userspace here, specifically, another vcore and we
+                        * don't know who it is.  This will spin a bit, then make sure they
+                        * aren't preeempted */
+                       cpu_relax_vc(vcore_id());       /* pass in self to check everyone else*/
+               }
                /* Now the page is done.  0 its metadata and give it up. */
                old_page->header.cons_next_pg = 0;
                atomic_set(&old_page->header.nr_cons, 0);
@@ -127,7 +132,7 @@ loop_top:
                }
                /* All fixed up, unlock.  Other consumers may lock and check to make
                 * sure things are done. */
-               mcs_unlock_notifsafe(ucq_lock, &local_qn);
+               mcs_pdr_unlock(ucq_lock);
                /* Now that everything is fixed, try again from the top */
                goto loop_top;
 claim_slot:
@@ -152,3 +157,10 @@ claim_slot:
        atomic_inc(&((struct ucq_page*)PTE_ADDR(my_idx))->header.nr_cons);
        return 0;
 }
+
+bool ucq_is_empty(struct ucq *ucq)
+{
+       /* The ucq is empty if the consumer and producer are on the same 'next'
+        * slot. */
+       return (atomic_read(&ucq->cons_idx) == atomic_read(&ucq->prod_idx));
+}