summaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorCarl Shapiro <cshapiro@google.com>2010-09-28 13:47:03 -0700
committerCarl Shapiro <cshapiro@google.com>2010-09-28 14:06:00 -0700
commitd55f0adfb5ec4202ad5bd5d188e66c0f6a27b0aa (patch)
tree409d6c3ac1cb97d41a9422f8e740134d7847bdfd /include
parentf62b23f2c1540221621024c850fdf615dda0c903 (diff)
downloadsystem_core-d55f0adfb5ec4202ad5bd5d188e66c0f6a27b0aa.zip
system_core-d55f0adfb5ec4202ad5bd5d188e66c0f6a27b0aa.tar.gz
system_core-d55f0adfb5ec4202ad5bd5d188e66c0f6a27b0aa.tar.bz2
Qualify the source argument of atomic loads as a const pointer.
Also normalizes the opening brace placment in a few locations. Change-Id: I8f518e933094337d5d3371321326ffc03b3a5f5a
Diffstat (limited to 'include')
-rw-r--r--include/cutils/atomic-arm.h10
-rw-r--r--include/cutils/atomic-x86.h18
-rw-r--r--include/cutils/atomic.h4
3 files changed, 20 insertions, 12 deletions
diff --git a/include/cutils/atomic-arm.h b/include/cutils/atomic-arm.h
index 0dd629d..93633c4 100644
--- a/include/cutils/atomic-arm.h
+++ b/include/cutils/atomic-arm.h
@@ -49,14 +49,14 @@ extern inline void android_memory_barrier(void)
}
#endif
-extern inline int32_t android_atomic_acquire_load(volatile int32_t *ptr)
+extern inline int32_t android_atomic_acquire_load(volatile const int32_t *ptr)
{
int32_t value = *ptr;
android_memory_barrier();
return value;
}
-extern inline int32_t android_atomic_release_load(volatile int32_t *ptr)
+extern inline int32_t android_atomic_release_load(volatile const int32_t *ptr)
{
android_memory_barrier();
return *ptr;
@@ -196,11 +196,13 @@ extern inline int32_t android_atomic_add(int32_t increment,
}
#endif
-extern inline int32_t android_atomic_inc(volatile int32_t *addr) {
+extern inline int32_t android_atomic_inc(volatile int32_t *addr)
+{
return android_atomic_add(1, addr);
}
-extern inline int32_t android_atomic_dec(volatile int32_t *addr) {
+extern inline int32_t android_atomic_dec(volatile int32_t *addr)
+{
return android_atomic_add(-1, addr);
}
diff --git a/include/cutils/atomic-x86.h b/include/cutils/atomic-x86.h
index 06b643f..12a1985 100644
--- a/include/cutils/atomic-x86.h
+++ b/include/cutils/atomic-x86.h
@@ -36,25 +36,29 @@ extern inline void android_memory_barrier(void)
}
#endif
-extern inline int32_t android_atomic_acquire_load(volatile int32_t *ptr) {
+extern inline int32_t android_atomic_acquire_load(volatile const int32_t *ptr)
+{
int32_t value = *ptr;
android_compiler_barrier();
return value;
}
-extern inline int32_t android_atomic_release_load(volatile int32_t *ptr) {
+extern inline int32_t android_atomic_release_load(volatile const int32_t *ptr)
+{
android_memory_barrier();
return *ptr;
}
extern inline void android_atomic_acquire_store(int32_t value,
- volatile int32_t *ptr) {
+ volatile int32_t *ptr)
+{
*ptr = value;
android_memory_barrier();
}
extern inline void android_atomic_release_store(int32_t value,
- volatile int32_t *ptr) {
+ volatile int32_t *ptr)
+{
android_compiler_barrier();
*ptr = value;
}
@@ -107,11 +111,13 @@ extern inline int32_t android_atomic_add(int32_t increment,
return increment;
}
-extern inline int32_t android_atomic_inc(volatile int32_t *addr) {
+extern inline int32_t android_atomic_inc(volatile int32_t *addr)
+{
return android_atomic_add(1, addr);
}
-extern inline int32_t android_atomic_dec(volatile int32_t *addr) {
+extern inline int32_t android_atomic_dec(volatile int32_t *addr)
+{
return android_atomic_add(-1, addr);
}
diff --git a/include/cutils/atomic.h b/include/cutils/atomic.h
index 3866848..a50bf0f 100644
--- a/include/cutils/atomic.h
+++ b/include/cutils/atomic.h
@@ -77,8 +77,8 @@ int32_t android_atomic_or(int32_t value, volatile int32_t* addr);
* This is only necessary if you need the memory barrier. A 32-bit read
* from a 32-bit aligned address is atomic on all supported platforms.
*/
-int32_t android_atomic_acquire_load(volatile int32_t* addr);
-int32_t android_atomic_release_load(volatile int32_t* addr);
+int32_t android_atomic_acquire_load(volatile const int32_t* addr);
+int32_t android_atomic_release_load(volatile const int32_t* addr);
/*
* Perform an atomic store with "acquire" or "release" ordering.