diff options
author | Richard Braun <rbraun@sceen.net> | 2017-07-25 21:26:57 +0200 |
---|---|---|
committer | Richard Braun <rbraun@sceen.net> | 2017-07-25 21:26:57 +0200 |
commit | cfcd4d2d503c0f34cb975ca5c8d130af4039121d (patch) | |
tree | aed402c1970b4a79d26edc418b2e6b6119e4403e /kern/rtmutex.c | |
parent | 35a39b5b7e89870daea062d3fe433b15c7697827 (diff) |
kern/rtmutex: improve variable naming
Use the same names as in the mutex_adaptive module.
Diffstat (limited to 'kern/rtmutex.c')
-rw-r--r-- | kern/rtmutex.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/kern/rtmutex.c b/kern/rtmutex.c index fcf5f358..db239206 100644 --- a/kern/rtmutex.c +++ b/kern/rtmutex.c @@ -36,11 +36,11 @@ void rtmutex_lock_slow(struct rtmutex *rtmutex) { struct turnstile *turnstile; - uintptr_t owner, prev_owner; /* TODO Review names */ + uintptr_t self, owner; struct thread *thread; uintptr_t bits; - owner = (uintptr_t)thread_self(); + self = (uintptr_t)thread_self(); turnstile = turnstile_lend(rtmutex); @@ -49,14 +49,14 @@ rtmutex_lock_slow(struct rtmutex *rtmutex) bits = RTMUTEX_CONTENDED; for (;;) { - prev_owner = atomic_cas_acquire(&rtmutex->owner, bits, owner | bits); - assert((prev_owner & bits) == bits); + owner = atomic_cas_acquire(&rtmutex->owner, bits, self | bits); + assert((owner & bits) == bits); - if (prev_owner == bits) { + if (owner == bits) { break; } - thread = (struct thread *)(prev_owner & RTMUTEX_OWNER_MASK); + thread = (struct thread *)(owner & RTMUTEX_OWNER_MASK); turnstile_wait(turnstile, "rtmutex", thread); bits |= RTMUTEX_FORCE_WAIT; } @@ -64,8 +64,8 @@ rtmutex_lock_slow(struct rtmutex *rtmutex) turnstile_own(turnstile); if (turnstile_empty(turnstile)) { - prev_owner = atomic_swap(&rtmutex->owner, owner, ATOMIC_RELAXED); - assert(prev_owner == (owner | bits)); + owner = atomic_swap(&rtmutex->owner, self, ATOMIC_RELAXED); + assert(owner == (self | bits)); } turnstile_return(turnstile); @@ -82,16 +82,16 @@ void rtmutex_unlock_slow(struct rtmutex *rtmutex) { struct turnstile *turnstile; - uintptr_t owner, prev_owner; + uintptr_t self, owner; - owner = (uintptr_t)thread_self(); + self = (uintptr_t)thread_self(); turnstile = turnstile_acquire(rtmutex); assert(turnstile != NULL); - prev_owner = atomic_swap_release(&rtmutex->owner, - RTMUTEX_FORCE_WAIT | RTMUTEX_CONTENDED); - assert((prev_owner & RTMUTEX_OWNER_MASK) == owner); + owner = atomic_swap_release(&rtmutex->owner, + RTMUTEX_FORCE_WAIT | RTMUTEX_CONTENDED); + assert((owner & RTMUTEX_OWNER_MASK) == self); turnstile_disown(turnstile); turnstile_signal(turnstile); |