From: Matt Benjamin <matt@linuxbox.com>
Date: Sun, 13 Jun 2010 22:24:48 +0000 (-0400)
Subject: mcas cleanup inc/dec macros
X-Git-Tag: openafs-devel-1_5_75~140
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=77684b64dbf721a9a5d41aa1c70fd175fd27b329;p=packages%2Fo%2Fopenafs.git

mcas cleanup inc/dec macros

fix type of intermediate value used in CAS operations, and reformat slightly.

LICENSE BSD

Change-Id: If49be8e020e0f735d67290440b1a74caccf90bf0
Reviewed-on: http://gerrit.openafs.org/2152
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/mcas/portable_defns.h b/src/mcas/portable_defns.h
index 19a4b6653..91f79f353 100644
--- a/src/mcas/portable_defns.h
+++ b/src/mcas/portable_defns.h
@@ -70,7 +70,7 @@ typedef unsigned long int_addr_t;
 
 #define ADD_TO(_v,_x)                                                   \
 do {                                                                    \
-    int __val = (_v), __newval;                                         \
+    unsigned long __val = (_v), __newval;                               \
     while ( (__newval = CASIO(&(_v),__val,__val+(_x))) != __val )       \
         __val = __newval;                                               \
 } while ( 0 )
@@ -80,24 +80,24 @@ do {                                                                    \
  * example, the operation which uniquely incremented _v from 0 to 1, and
  * all equivalent threshold assertions */
 
-#define ADD_TO_RETURNING_OLD(_v,_x,_o)									\
+#define ADD_TO_RETURNING_OLD(_v,_x,_o)                                  \
 do {                                                                    \
-    int __val = (_v), __newval;                                         \
+    unsigned long __val = (_v), __newval;                               \
     while ( (__newval = CASIO(&(_v),__val,__val+(_x))) != __val )       \
         __val = __newval;                                               \
 	_o = __val;															\
 } while ( 0 )
 
-#define SUB_FROM(_v,_x)													\
+#define SUB_FROM(_v,_x)                                                 \
 do {                                                                    \
-    int __val = (_v), __newval;                                         \
+    unsigned long __val = (_v), __newval;                               \
     while ( (__newval = CASIO(&(_v),__val,__val-(_x))) != __val )       \
-        __val = __newval;												\
+        __val = __newval;                                               \
 } while ( 0 )
 
-#define SUB_FROM_RETURNING_OLD(_v,_x,_o)								\
+#define SUB_FROM_RETURNING_OLD(_v,_x,_o)                                \
 do {                                                                    \
-    int __val = (_v), __newval;                                         \
+    unsigned long __val = (_v), __newval;                               \
     while ( (__newval = CASIO(&(_v),__val,__val-(_x))) != __val )       \
         __val = __newval;                                               \
 	_o = __val;															\