From 77684b64dbf721a9a5d41aa1c70fd175fd27b329 Mon Sep 17 00:00:00 2001 From: Matt Benjamin Date: Sun, 13 Jun 2010 18:24:48 -0400 Subject: [PATCH] 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 Tested-by: Derrick Brashear --- src/mcas/portable_defns.h | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) 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; \ -- 2.39.5