Rename RCU CB context to 'cannot block' context
[akaros.git] / kern / src / atomic.c
index 10d7e25..70d2805 100644 (file)
@@ -1,7 +1,3 @@
-#ifdef __SHARC__
-#pragma nosharc
-#endif
-
 #include <arch/arch.h>
 #include <arch/kdebug.h>
 
@@ -43,7 +39,7 @@ static bool can_trace(spinlock_t *lock)
 }
 
 /* spinlock and trylock call this after locking */
-static void post_lock(spinlock_t *lock, uint32_t coreid)
+static __always_inline void post_lock(spinlock_t *lock, uint32_t coreid)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[coreid];
        if ((pcpui->__lock_checking_enabled == 1) && can_trace(lock))
@@ -106,17 +102,14 @@ void spin_unlock(spinlock_t *lock)
 void spinlock_debug(spinlock_t *lock)
 {
        uintptr_t pc = lock->call_site;
-       char *func_name;
 
        if (!pc) {
                printk("Lock %p: never locked\n", lock);
                return;
        }
-       func_name = get_fn_name(pc);
-       printk("Lock %p: currently %slocked.  Last locked at [<%p>] in %s on "
-              "core %d\n", lock, spin_locked(lock) ? "" : "un", pc, func_name,
+       printk("Lock %p: currently %slocked.  Last locked at [<%p>] in %s on core %d\n",
+              lock, spin_locked(lock) ? "" : "un", pc, get_fn_name(pc),
               lock->calling_core);
-       kfree(func_name);
 }
 
 #endif /* CONFIG_SPINLOCK_DEBUG */
@@ -233,7 +226,7 @@ int commit_checklist_wait(checklist_t* list, checklist_mask_t* mask)
                cpu_relax();
 
        // list is ours and clear, set it to the settings of our list
-       COPY_BITMASK(list->mask.bits, mask->bits, mask->size); 
+       COPY_BITMASK(list->mask.bits, mask->bits, mask->size);
        return 0;
 }
 
@@ -250,21 +243,21 @@ int commit_checklist_nowait(checklist_t* list, checklist_mask_t* mask)
 // what if two different actors are waiting on the list, but for different reasons?
 // part of the problem is we are doing both set and check via the same path
 //
-// aside: we made this a lot more difficult than the usual barriers or even 
+// aside: we made this a lot more difficult than the usual barriers or even
 // the RCU grace-period checkers, since we have to worry about this construct
 // being used by others before we are done with it.
 //
 // how about this: if we want to wait on this later, we just don't release the
 // lock.  if we release it, then we don't care who comes in and grabs and starts
-// checking the list.  
-//     - regardless, there are going to be issues with people looking for a free 
-//     item.  even if they grab the lock, they may end up waiting a while and 
-//     wantint to bail (like test for a while, give up, move on, etc).  
+// checking the list.
+//     - regardless, there are going to be issues with people looking for a free
+//     item.  even if they grab the lock, they may end up waiting a while and
+//     wantint to bail (like test for a while, give up, move on, etc).
 //     - still limited in that only the setter can check, and only one person
 //     can spinwait / check for completion.  if someone else tries to wait (wanting
 //     completion), they may miss it if someone else comes in and grabs the lock
 //     to use it for a new checklist
-//             - if we had the ability to sleep and get woken up, we could have a 
+//             - if we had the ability to sleep and get woken up, we could have a
 //             queue.  actually, we could do a queue anyway, but they all spin
 //             and it's the bosses responsibility to *wake* them
 
@@ -301,6 +294,12 @@ int checklist_is_clear(checklist_t* list)
        return BITMASK_IS_CLEAR(list->mask.bits, list->mask.size);
 }
 
+// no synch guarantees - just looks at the list
+int checklist_is_full(checklist_t* list)
+{
+       return BITMASK_IS_FULL(list->mask.bits, list->mask.size);
+}
+
 // no synch guarantees - just resets the list to empty
 void reset_checklist(checklist_t* list)
 {