diff options
| author | Jackie Liu <liuyun01@kylinos.cn> | 2019-08-09 10:40:39 +0800 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2019-08-08 21:37:04 -0600 |
| commit | 0520db454c29f1d96cda6cf6cedeb93df65301e8 (patch) | |
| tree | 5a5ec8b32b56d7e3edd82f3d54731ecec90ad17b | |
| parent | 72d7f5a5fa82686fd0b37b1c3ac293c8b3e5f6db (diff) | |
Add arm64 memory barriers support
making liburing on arm64 platform failed. let's support it.
root@Kylin:/# cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux buster/sid"
root@Kylin:/# gcc -v
gcc version 8.2.0 (Debian 8.2.0-20)
root@Kylin:~/liburing# make
make[1]: Entering directory '/root/liburing/src'
cc -g -fomit-frame-pointer -O2 -Wall -Iinclude/ -c -o setup.ol setup.c
In file included from include/liburing.h:14,
from setup.c:10:
include/liburing.h: In function 'io_uring_cq_advance':
include/liburing/barrier.h:73:2: warning: implicit declaration of function 'smp_mb'; did you mean 'smp_wmb'? [-Wimplicit-function-declaration]
smp_mb(); \
^~~~~~
include/liburing.h:111:3: note: in expansion of macro 'smp_store_release'
smp_store_release(cq->khead, *cq->khead + nr);
^~~~~~~~~~~~~~~~~
Signed-off-by: Jackie Liu <liuyun01@kylinos.cn>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
| -rw-r--r-- | src/include/liburing/barrier.h | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/src/include/liburing/barrier.h b/src/include/liburing/barrier.h index 98be9e5..b98193b 100644 --- a/src/include/liburing/barrier.h +++ b/src/include/liburing/barrier.h @@ -56,7 +56,20 @@ do { \ barrier(); \ ___p1; \ }) -#else /* defined(__x86_64__) || defined(__i386__) */ + +#elif defined(__aarch64__) +/* Adapted from arch/arm64/include/asm/barrier.h */ +#define dmb(opt) asm volatile("dmb " #opt : : : "memory") +#define dsb(opt) asm volatile("dsb " #opt : : : "memory") + +#define mb() dsb(sy) +#define rmb() dsb(ld) +#define wmb() dsb(st) +#define smp_mb() dmb(ish) +#define smp_rmb() dmb(ishld) +#define smp_wmb() dmb(ishst) + +#else /* defined(__x86_64__) || defined(__i386__) || defined(__aarch64__) */ /* * Add arch appropriate definitions. Be safe and use full barriers for * archs we don't have support for. |