[PATCH 2/2] arm64: bpf: add BPF XADD instruction
David Miller
davem at davemloft.net
Wed Nov 11 11:01:06 PST 2015
From: Will Deacon <will.deacon at arm.com>
Date: Wed, 11 Nov 2015 17:44:01 +0000
> On Wed, Nov 11, 2015 at 12:35:48PM -0500, David Miller wrote:
>> From: Alexei Starovoitov <alexei.starovoitov at gmail.com>
>> Date: Wed, 11 Nov 2015 09:27:00 -0800
>>
>> > BPF_XADD == atomic_add() in kernel. period.
>> > we are not going to deprecate it or introduce something else.
>>
>> Agreed, it makes no sense to try and tie C99 or whatever atomic
>> semantics to something that is already clearly defined to have
>> exactly kernel atomic_add() semantics.
>
> ... and which is emitted by LLVM when asked to compile __sync_fetch_and_add,
> which has clearly defined (yet conflicting) semantics.
Alexei clearly stated that he knows about this issue and will fully
fix this up in LLVM.
What more do you need to hear from him once he's stated that he is
aware and is working on it? Meanwhile you should make your JIT emit
what is expected, rather than arguing to change the semantics.
Thanks.
More information about the linux-arm-kernel
mailing list