Changeset 228666c in mainline for kernel/arch/ia32/include/atomic.h

Timestamp:

2010-02-20T18:41:53Z (15 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

b03a666

Parents:

bc9da2a

Message:

introduce atomic_count_t as the explicit type of the internal value in atomic_t (this is probably better than the chaotic mix of int/long)
atomic_count_t is defined as unsigned, for signed semantics you can cast it to atomic_signed_t

File:

• kernel/arch/ia32/include/atomic.h (modified) (7 diffs)

kernel/arch/ia32/include/atomic.h

@@ -40 +40 @@
 #include <preemption.h>
 
-static inline void atomic_inc(atomic_t *val) {
+static inline void atomic_inc(atomic_t *val)
+{
 #ifdef CONFIG_SMP
         asm volatile (
@@ -54 +55 @@
 }
 
-static inline void atomic_dec(atomic_t *val) {
+static inline void atomic_dec(atomic_t *val)
+{
 #ifdef CONFIG_SMP
         asm volatile (
@@ -68 +70 @@
 }
 
-static inline long atomic_postinc(atomic_t *val) 
+static inline atomic_count_t atomic_postinc(atomic_t *val)
 {
-        long r = 1;
+        atomic_count_t r = 1;
         
         asm volatile (
                 "lock xaddl %[r], %[count]\n"
-                : [count] "+m" (val->count), [r] "+r" (r)
+                : [count] "+m" (val->count),
+                  [r] "+r" (r)
         );
         
@@ -80 +83 @@
 }
 
-static inline long atomic_postdec(atomic_t *val) 
+static inline atomic_count_t atomic_postdec(atomic_t *val)
 {
-        long r = -1;
+        atomic_count_t r = -1;
         
         asm volatile (
                 "lock xaddl %[r], %[count]\n"
-                : [count] "+m" (val->count), [r] "+r"(r)
+                : [count] "+m" (val->count),
+                  [r] "+r" (r)
         );
         
@@ -95 +99 @@
 #define atomic_predec(val)  (atomic_postdec(val) - 1)
 
-static inline uint32_t test_and_set(atomic_t *val) {
-        uint32_t v;
+static inline atomic_count_t test_and_set(atomic_t *val)
+{
+        atomic_count_t v;
         
         asm volatile (
                 "movl $1, %[v]\n"
                 "xchgl %[v], %[count]\n"
-                : [v] "=r" (v), [count] "+m" (val->count)
+                : [v] "=r" (v),
+                  [count] "+m" (val->count)
         );
         
@@ -110 +116 @@
 static inline void atomic_lock_arch(atomic_t *val)
 {
-        uint32_t tmp;
+        atomic_count_t tmp;
         
         preemption_disable();
@@ -124 +130 @@
                 "testl %[tmp], %[tmp]\n"
                 "jnz 0b\n"
-                : [count] "+m" (val->count), [tmp] "=&r" (tmp)
+                : [count] "+m" (val->count),
+                  [tmp] "=&r" (tmp)
         );
+        
         /*
          * Prevent critical section code from bleeding out this way up.