Mirrors/criu
1
0
Fork 0
mirror of https://github.com/checkpoint-restore/criu.git synced 2026-07-20 01:54:28 +00:00
Code Issues Projects Releases Wiki Activity

zdtm/atomic: atomic_inc should return the old value

Browse source 
00:24:30.269:    44: ERR: test.c:208: Test exited on unexpected signal

It's a correct name for this function. For example, we use this name
here criu/arch/ppc64/include/asm/atomic.h

And you can find the same set of functions here:
test/zdtm/lib/arch/aarch64/include/asm/atomic.h

I think this code is more readable (understandable) than just
return t - 1.

Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com>
This commit is contained in:
Andrei Vagin 2016-04-27 03:26:00 +03:00 • committed by Pavel Emelyanov
parent 9d03d77f20
commit 7fdbfba40a
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
test/zdtm/lib/arch/ppc64/include/asm/atomic.h
View file

@ -47,7 +47,7 @@ ATOMIC_OP(sub, subf)
#undef ATOMIC_OP
static __inline__ int atomic_inc(atomic_t *v)
static __inline__ int atomic_inc_return(atomic_t *v)
{
int t;
@ -65,6 +65,11 @@ static __inline__ int atomic_inc(atomic_t *v)
return t;
}
static __inline__ int atomic_inc(atomic_t *v)
{
return atomic_inc_return(v) - 1;
}
static __inline__ void atomic_dec(atomic_t *v)
{
int t;