remove little-endian assumption from arm atomic.h
authorRich Felker <dalias@aerifal.cx>
Sun, 8 Jul 2012 04:05:08 +0000 (00:05 -0400)
committerRich Felker <dalias@aerifal.cx>
Sun, 8 Jul 2012 04:05:08 +0000 (00:05 -0400)
this hidden endian dependency had left big endian arm badly broken.

arch/arm/atomic.h

index 8dc31b2ef496852dcbb010ffd8957bc15e7aff2d..f434a0c652dea795ec9de531a899f16a0b76ed46 100644 (file)
@@ -99,14 +99,16 @@ static inline void a_or(volatile int *p, int v)
 
 static inline void a_and_64(volatile uint64_t *p, uint64_t v)
 {
-       a_and((int *)p, v);
-       a_and((int *)p+1, v>>32);
+       union { uint64_t v; uint32_t r[2]; } u = { v };
+       a_and((int *)p, u.r[0]);
+       a_and((int *)p+1, u.r[1]);
 }
 
 static inline void a_or_64(volatile uint64_t *p, uint64_t v)
 {
-       a_or((int *)p, v);
-       a_or((int *)p+1, v>>32);
+       union { uint64_t v; uint32_t r[2]; } u = { v };
+       a_or((int *)p, u.r[0]);
+       a_or((int *)p+1, u.r[1]);
 }
 
 #endif