diff options
| author | Luca Barbieri <luca@luca-barbieri.com> | 2010-02-26 06:22:41 -0500 |
|---|---|---|
| committer | H. Peter Anvin <hpa@zytor.com> | 2010-03-01 14:38:42 -0500 |
| commit | d7f6de1e9c4a12e11ba7186c70f0f40caa76f590 (patch) | |
| tree | 0f9de2c0666d298a054f2a256da3029f97369d16 | |
| parent | 8f4f202b335144bf5be5c9e5b1bc9477ecdae958 (diff) | |
x86: Implement atomic[64]_dec_if_positive()
Add support for atomic_dec_if_positive(), and
atomic64_dec_if_positive() for x86-64.
atomic64_dec_if_positive() for x86-32 was already implemented in a previous patch.
Signed-off-by: Luca Barbieri <luca@luca-barbieri.com>
LKML-Reference: <1267183361-20775-2-git-send-email-luca@luca-barbieri.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
| -rw-r--r-- | arch/x86/include/asm/atomic.h | 23 | ||||
| -rw-r--r-- | arch/x86/include/asm/atomic64_64.h | 23 | ||||
| -rw-r--r-- | lib/atomic64_test.c | 2 |
3 files changed, 47 insertions, 1 deletions
diff --git a/arch/x86/include/asm/atomic.h b/arch/x86/include/asm/atomic.h index 8f8217b9bdac..706c69492c14 100644 --- a/arch/x86/include/asm/atomic.h +++ b/arch/x86/include/asm/atomic.h | |||
| @@ -246,6 +246,29 @@ static inline int atomic_add_unless(atomic_t *v, int a, int u) | |||
| 246 | 246 | ||
| 247 | #define atomic_inc_not_zero(v) atomic_add_unless((v), 1, 0) | 247 | #define atomic_inc_not_zero(v) atomic_add_unless((v), 1, 0) |
| 248 | 248 | ||
| 249 | /* | ||
| 250 | * atomic_dec_if_positive - decrement by 1 if old value positive | ||
| 251 | * @v: pointer of type atomic_t | ||
| 252 | * | ||
| 253 | * The function returns the old value of *v minus 1, even if | ||
| 254 | * the atomic variable, v, was not decremented. | ||
| 255 | */ | ||
| 256 | static inline int atomic_dec_if_positive(atomic_t *v) | ||
| 257 | { | ||
| 258 | int c, old, dec; | ||
| 259 | c = atomic_read(v); | ||
| 260 | for (;;) { | ||
| 261 | dec = c - 1; | ||
| 262 | if (unlikely(dec < 0)) | ||
| 263 | break; | ||
| 264 | old = atomic_cmpxchg((v), c, dec); | ||
| 265 | if (likely(old == c)) | ||
| 266 | break; | ||
| 267 | c = old; | ||
| 268 | } | ||
| 269 | return dec; | ||
| 270 | } | ||
| 271 | |||
| 249 | /** | 272 | /** |
| 250 | * atomic_inc_short - increment of a short integer | 273 | * atomic_inc_short - increment of a short integer |
| 251 | * @v: pointer to type int | 274 | * @v: pointer to type int |
diff --git a/arch/x86/include/asm/atomic64_64.h b/arch/x86/include/asm/atomic64_64.h index 51c5b4056929..4d6e2cd6c88c 100644 --- a/arch/x86/include/asm/atomic64_64.h +++ b/arch/x86/include/asm/atomic64_64.h | |||
| @@ -221,4 +221,27 @@ static inline int atomic64_add_unless(atomic64_t *v, long a, long u) | |||
| 221 | 221 | ||
| 222 | #define atomic64_inc_not_zero(v) atomic64_add_unless((v), 1, 0) | 222 | #define atomic64_inc_not_zero(v) atomic64_add_unless((v), 1, 0) |
| 223 | 223 | ||
| 224 | /* | ||
| 225 | * atomic64_dec_if_positive - decrement by 1 if old value positive | ||
| 226 | * @v: pointer of type atomic_t | ||
| 227 | * | ||
| 228 | * The function returns the old value of *v minus 1, even if | ||
| 229 | * the atomic variable, v, was not decremented. | ||
| 230 | */ | ||
| 231 | static inline long atomic64_dec_if_positive(atomic64_t *v) | ||
| 232 | { | ||
| 233 | long c, old, dec; | ||
| 234 | c = atomic64_read(v); | ||
| 235 | for (;;) { | ||
| 236 | dec = c - 1; | ||
| 237 | if (unlikely(dec < 0)) | ||
| 238 | break; | ||
| 239 | old = atomic64_cmpxchg((v), c, dec); | ||
| 240 | if (likely(old == c)) | ||
| 241 | break; | ||
| 242 | c = old; | ||
| 243 | } | ||
| 244 | return dec; | ||
| 245 | } | ||
| 246 | |||
| 224 | #endif /* _ASM_X86_ATOMIC64_64_H */ | 247 | #endif /* _ASM_X86_ATOMIC64_64_H */ |
diff --git a/lib/atomic64_test.c b/lib/atomic64_test.c index 0effcacbebda..58efdabb3845 100644 --- a/lib/atomic64_test.c +++ b/lib/atomic64_test.c | |||
| @@ -112,7 +112,7 @@ static __init int test_atomic64(void) | |||
| 112 | r += one; | 112 | r += one; |
| 113 | BUG_ON(v.counter != r); | 113 | BUG_ON(v.counter != r); |
| 114 | 114 | ||
| 115 | #if defined(CONFIG_X86_32) || defined(CONFIG_MIPS) || defined(CONFIG_PPC) || defined(_ASM_GENERIC_ATOMIC64_H) | 115 | #if defined(CONFIG_X86) || defined(CONFIG_MIPS) || defined(CONFIG_PPC) || defined(_ASM_GENERIC_ATOMIC64_H) |
| 116 | INIT(onestwos); | 116 | INIT(onestwos); |
| 117 | BUG_ON(atomic64_dec_if_positive(&v) != (onestwos - 1)); | 117 | BUG_ON(atomic64_dec_if_positive(&v) != (onestwos - 1)); |
| 118 | r -= one; | 118 | r -= one; |
