Re: [PATCH v2] atomics,cmpxchg: Privatize the inclusion of asm/cmpxchg.h

From: Davidlohr Bueso
Date: Sun Sep 13 2015 - 18:48:04 EST


On Wed, 09 Sep 2015, Boqun Feng wrote:

After commit:

atomics: add acquire/release/relaxed variants of some atomic operations

Architectures may only provide {cmp,}xchg_relaxed definitions in
asm/cmpxchg.h. Other variants, such as {cmp,}xchg, may be built in
linux/atomic.h, which means simply including asm/cmpxchg.h may not get
the definitions of all the{cmp,}xchg variants. Therefore, we should
privatize the inclusions of asm/cmpxchg.h to keep it only included in
arch/* and replace the inclusions outside with linux/atomic.h

Acked-by: Will Deacon <will.deacon@xxxxxxx>
Signed-off-by: Boqun Feng <boqun.feng@xxxxxxxxx>

fwiw,

Acked-by: Davidlohr Bueso <dave@xxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/