Changes in kernel/generic/src/synch/spinlock.c [78de83de:05882233] in mainline

File:

• kernel/generic/src/synch/spinlock.c (modified) (6 diffs)

kernel/generic/src/synch/spinlock.c

@@ -56 +56 @@
 void spinlock_initialize(spinlock_t *lock, const char *name)
 {
-        atomic_flag_clear_explicit(&lock->flag, memory_order_relaxed);
+        atomic_set(&lock->val, 0);
 #ifdef CONFIG_DEBUG_SPINLOCK
         lock->name = name;
@@ -79 +79 @@
 
         preemption_disable();
-        while (atomic_flag_test_and_set_explicit(&lock->flag, memory_order_acquire)) {
+        while (test_and_set(&lock->val)) {
                 /*
                  * We need to be careful about particular locks
@@ -115 +115 @@
         if (deadlock_reported)
                 printf("cpu%u: not deadlocked\n", CPU->id);
+
+        /*
+         * Prevent critical section code from bleeding out this way up.
+         */
+        CS_ENTER_BARRIER();
 }
 
@@ -127 +132 @@
         ASSERT_SPINLOCK(spinlock_locked(lock), lock);
 
-        atomic_flag_clear_explicit(&lock->flag, memory_order_release);
+        /*
+         * Prevent critical section code from bleeding out this way down.
+         */
+        CS_LEAVE_BARRIER();
+
+        atomic_set(&lock->val, 0);
         preemption_enable();
 }
@@ -146 +156 @@
 {
         preemption_disable();
-        bool ret = !atomic_flag_test_and_set_explicit(&lock->flag, memory_order_acquire);
+        bool ret = !test_and_set(&lock->val);
+
+        /*
+         * Prevent critical section code from bleeding out this way up.
+         */
+        CS_ENTER_BARRIER();
 
         if (!ret)
@@ -161 +176 @@
 bool spinlock_locked(spinlock_t *lock)
 {
-        // XXX: Atomic flag doesn't support simple atomic read (by design),
-        //      so instead we test_and_set and then clear if necessary.
-        //      This function is only used inside assert, so we don't need
-        //      any preemption_disable/enable here.
-
-        bool ret = atomic_flag_test_and_set_explicit(&lock->flag, memory_order_relaxed);
-        if (!ret)
-                atomic_flag_clear_explicit(&lock->flag, memory_order_relaxed);
-        return ret;
+        return atomic_get(&lock->val) != 0;
 }