Skip to contentSkip to conversation

arch-arm: Consider armv7ve arch as well

PATCH In review Khem Raj 3 participants 3 messages Oct 5, 2019 – Nov 5, 2019

lore
Share & tools
lore

Khem Raj

author
#0 lore raw

PATCH 1/1 arch-arm: Consider armv7ve arch as well

---
 arch/arch-arm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arch-arm.h +1 -1
index 78cb2ebe..4f722967 100644
@@ -11,7 +11,7 @@
 #define nop             __asm__ __volatile__("mov\tr0,r0\t@ nop\n\t")
 #define read_barrier()	__asm__ __volatile__ ("" : : : "memory")
 #define write_barrier()	__asm__ __volatile__ ("" : : : "memory")
-#elif defined(__ARM_ARCH_7A__) || defined(__ARM_ARCH_8A__)
+#elif defined(__ARM_ARCH_7A__) || defined(__ARM_ARCH_8A__) || defined(__ARM_ARCH_7VE__)
 #define	nop		__asm__ __volatile__ ("nop")
 #define read_barrier()	__sync_synchronize()
 #define write_barrier()	__sync_synchronize()
Signature · 1 line
2.23.0
#1 lore raw

Hi Khem,

It might have been nicer for 7VE to come before 8A but it's easy to get the gist...

Jens - I think this patch may have been overlooked...

Quoted 22 lines · Khem Raj
---
 arch/arch-arm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arch-arm.h +1 -2
index 78cb2ebe..4f722967 100644
@@ -11,7 +11,7 @@
 #define nop             __asm__ __volatile__("mov\tr0,r0\t@ nop\n\t")
 #define read_barrier() __asm__ __volatile__ ("" : : : "memory")
 #define write_barrier()        __asm__ __volatile__ ("" : : : "memory")
-#elif defined(__ARM_ARCH_7A__) || defined(__ARM_ARCH_8A__)
+#elif defined(__ARM_ARCH_7A__) || defined(__ARM_ARCH_8A__) || defined(__ARM_ARCH_7VE__)
 #define        nop             __asm__ __volatile__ ("nop")
 #define read_barrier() __sync_synchronize()
 #define write_barrier()        __sync_synchronize()
--

2.23.0

Signature · 1 line
Sitsofe | http://sucs.org/~sits/
#2 lore raw
Quoted 6 lines · Sitsofe Wheeler

Hi Khem,

It might have been nicer for 7VE to come before 8A but it's easy to get the gist...

Jens - I think this patch may have been overlooked...

It did, thanks for bringing it to my attention. Now applied, thanks.

Signature · 1 line
Jens Axboe