eal/arm64: remove the braces in memory barrier macros
authorJia He <jia.he@hxt-semitech.com>
Mon, 22 Jan 2018 04:41:26 +0000 (20:41 -0800)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 29 Jan 2018 14:14:53 +0000 (15:14 +0100)
for the code as follows:
if (condition)
rte_smp_rmb();
else
rte_smp_wmb();
Without this patch, compiler will report this error:
error: 'else' without a previous 'if'

Fixes: 84733fd0d75e ("eal/arm64: fix memory barrier definition")
Cc: stable@dpdk.org
Signed-off-by: Jia He <jia.he@hxt-semitech.com>
Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
lib/librte_eal/common/include/arch/arm/rte_atomic_64.h

index ee0d0d1..97060e4 100644 (file)
@@ -15,8 +15,8 @@ extern "C" {
 
 #include "generic/rte_atomic.h"
 
-#define dsb(opt)  { asm volatile("dsb " #opt : : : "memory"); }
-#define dmb(opt)  { asm volatile("dmb " #opt : : : "memory"); }
+#define dsb(opt) asm volatile("dsb " #opt : : : "memory")
+#define dmb(opt) asm volatile("dmb " #opt : : : "memory")
 
 #define rte_mb() dsb(sy)