lib: Fix atomic64_add_unless return value convention
authorLuca Barbieri <luca@luca-barbieri.com>
Mon, 1 Mar 2010 18:55:47 +0000 (19:55 +0100)
committerH. Peter Anvin <hpa@zytor.com>
Mon, 1 Mar 2010 19:38:46 +0000 (11:38 -0800)
atomic64_add_unless must return 1 if it perfomed the add and 0 otherwise.
The generic implementation did the opposite thing.

Reported-by: H. Peter Anvin <hpa@zytor.com>
Confirmed-by: Paul Mackerras <paulus@samba.org>
Signed-off-by: Luca Barbieri <luca@luca-barbieri.com>
LKML-Reference: <1267469749-11878-4-git-send-email-luca@luca-barbieri.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
lib/atomic64.c

index 8bee16ec75242d756e19dd67a2173520468474db..a21c12bc727ca973908e34e7ddbbdfab62eeb5ed 100644 (file)
@@ -162,12 +162,12 @@ int atomic64_add_unless(atomic64_t *v, long long a, long long u)
 {
        unsigned long flags;
        spinlock_t *lock = lock_addr(v);
-       int ret = 1;
+       int ret = 0;
 
        spin_lock_irqsave(lock, flags);
        if (v->counter != u) {
                v->counter += a;
-               ret = 0;
+               ret = 1;
        }
        spin_unlock_irqrestore(lock, flags);
        return ret;