summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Braun <rbraun@sceen.net>2018-08-12 15:25:16 +0200
committerRichard Braun <rbraun@sceen.net>2018-08-12 15:25:16 +0200
commit56c5702a8c6b192a21f031747357eaa6170cec72 (patch)
tree360383b6a2e0e656226b933ee0d6b8b4e0740cac
parentd50bbf0c7a73ee6f817535e9d0e7792ea7894a65 (diff)
kern/mutex/mutex_plain: fix macro names
-rw-r--r--kern/mutex/mutex_plain.c11
-rw-r--r--kern/mutex/mutex_plain_i.h21
2 files changed, 16 insertions, 16 deletions
diff --git a/kern/mutex/mutex_plain.c b/kern/mutex/mutex_plain.c
index bc49ca0b..7e021a94 100644
--- a/kern/mutex/mutex_plain.c
+++ b/kern/mutex/mutex_plain.c
@@ -83,9 +83,10 @@ mutex_plain_lock_slow_common(struct mutex *mutex, bool timed, uint64_t ticks)
sleepq = sleepq_lend(mutex, false);
for (;;) {
- state = atomic_swap(&mutex->state, MUTEX_CONTENDED, ATOMIC_RELEASE);
+ state = atomic_swap(&mutex->state, MUTEX_PLAIN_CONTENDED,
+ ATOMIC_RELEASE);
- if (state == MUTEX_UNLOCKED) {
+ if (state == MUTEX_PLAIN_UNLOCKED) {
break;
}
@@ -105,8 +106,8 @@ mutex_plain_lock_slow_common(struct mutex *mutex, bool timed, uint64_t ticks)
if (sleepq_empty(sleepq)) {
mutex_plain_inc_sc(MUTEX_PLAIN_SC_ERROR_DOWNGRADES);
- atomic_cas(&mutex->state, MUTEX_CONTENDED,
- MUTEX_LOCKED, ATOMIC_RELAXED);
+ atomic_cas(&mutex->state, MUTEX_PLAIN_CONTENDED,
+ MUTEX_PLAIN_LOCKED, ATOMIC_RELAXED);
}
goto out;
@@ -116,7 +117,7 @@ mutex_plain_lock_slow_common(struct mutex *mutex, bool timed, uint64_t ticks)
if (sleepq_empty(sleepq)) {
mutex_plain_inc_sc(MUTEX_PLAIN_SC_DOWNGRADES);
- atomic_store(&mutex->state, MUTEX_LOCKED, ATOMIC_RELAXED);
+ atomic_store(&mutex->state, MUTEX_PLAIN_LOCKED, ATOMIC_RELAXED);
}
out:
diff --git a/kern/mutex/mutex_plain_i.h b/kern/mutex/mutex_plain_i.h
index d3ce2d30..37ede489 100644
--- a/kern/mutex/mutex_plain_i.h
+++ b/kern/mutex/mutex_plain_i.h
@@ -32,15 +32,14 @@
#include <kern/init.h>
#include <kern/mutex_types.h>
-/* TODO Fix namespace */
-#define MUTEX_UNLOCKED 0
-#define MUTEX_LOCKED 1
-#define MUTEX_CONTENDED 2
+#define MUTEX_PLAIN_UNLOCKED 0
+#define MUTEX_PLAIN_LOCKED 1
+#define MUTEX_PLAIN_CONTENDED 2
static inline void
mutex_plain_init(struct mutex *mutex)
{
- mutex->state = MUTEX_UNLOCKED;
+ mutex->state = MUTEX_PLAIN_UNLOCKED;
}
static inline bool
@@ -49,7 +48,7 @@ mutex_plain_locked(const struct mutex *mutex)
unsigned int state;
state = atomic_load(&mutex->state, ATOMIC_RELAXED);
- return (state != MUTEX_UNLOCKED);
+ return (state != MUTEX_PLAIN_UNLOCKED);
}
static inline int
@@ -57,10 +56,10 @@ mutex_plain_lock_fast(struct mutex *mutex)
{
unsigned int state;
- state = atomic_cas(&mutex->state, MUTEX_UNLOCKED,
- MUTEX_LOCKED, ATOMIC_ACQUIRE);
+ state = atomic_cas(&mutex->state, MUTEX_PLAIN_UNLOCKED,
+ MUTEX_PLAIN_LOCKED, ATOMIC_ACQUIRE);
- if (unlikely(state != MUTEX_UNLOCKED)) {
+ if (unlikely(state != MUTEX_PLAIN_UNLOCKED)) {
return EBUSY;
}
@@ -72,9 +71,9 @@ mutex_plain_unlock_fast(struct mutex *mutex)
{
unsigned int state;
- state = atomic_swap(&mutex->state, MUTEX_UNLOCKED, ATOMIC_RELEASE);
+ state = atomic_swap(&mutex->state, MUTEX_PLAIN_UNLOCKED, ATOMIC_RELEASE);
- if (unlikely(state == MUTEX_CONTENDED)) {
+ if (unlikely(state == MUTEX_PLAIN_CONTENDED)) {
return EBUSY;
}