diff options
author | Richard Braun <rbraun@sceen.net> | 2018-08-12 15:25:16 +0200 |
---|---|---|
committer | Richard Braun <rbraun@sceen.net> | 2018-08-12 15:25:16 +0200 |
commit | 56c5702a8c6b192a21f031747357eaa6170cec72 (patch) | |
tree | 360383b6a2e0e656226b933ee0d6b8b4e0740cac | |
parent | d50bbf0c7a73ee6f817535e9d0e7792ea7894a65 (diff) |
kern/mutex/mutex_plain: fix macro names
-rw-r--r-- | kern/mutex/mutex_plain.c | 11 | ||||
-rw-r--r-- | kern/mutex/mutex_plain_i.h | 21 |
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; } |