summaryrefslogtreecommitdiffstats
path: root/sys/kern/kern_mutex.c
diff options
context:
space:
mode:
authorjhb <jhb@FreeBSD.org>2004-08-04 20:18:45 +0000
committerjhb <jhb@FreeBSD.org>2004-08-04 20:18:45 +0000
commitc75eeac1dfdfc56d2aadab24b5ff9a35f2c42d46 (patch)
tree6b816a946ea3b88a6556df9bc12b49efeb2f2c7d /sys/kern/kern_mutex.c
parent1f29360fadc7c072b586a85e336874c798f0b701 (diff)
downloadFreeBSD-src-c75eeac1dfdfc56d2aadab24b5ff9a35f2c42d46.zip
FreeBSD-src-c75eeac1dfdfc56d2aadab24b5ff9a35f2c42d46.tar.gz
Cache the value of curthread in the _get_sleep_lock() and _get_spin_lock()
macros and pass the value to the associated _mtx_*() functions to avoid more curthread dereferences in the function implementations. This provided a very modest perf improvement in some benchmarks. Suggested by: rwatson Tested by: scottl
Diffstat (limited to 'sys/kern/kern_mutex.c')
-rw-r--r--sys/kern/kern_mutex.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/sys/kern/kern_mutex.c b/sys/kern/kern_mutex.c
index 7b66186..884eeda 100644
--- a/sys/kern/kern_mutex.c
+++ b/sys/kern/kern_mutex.c
@@ -417,10 +417,10 @@ _mtx_trylock(struct mtx *m, int opts, const char *file, int line)
* sleep waiting for it), or if we need to recurse on it.
*/
void
-_mtx_lock_sleep(struct mtx *m, int opts, const char *file, int line)
+_mtx_lock_sleep(struct mtx *m, struct thread *td, int opts, const char *file,
+ int line)
{
struct turnstile *ts;
- struct thread *td = curthread;
#if defined(SMP) && !defined(NO_ADAPTIVE_MUTEXES)
struct thread *owner;
#endif
@@ -562,7 +562,8 @@ _mtx_lock_sleep(struct mtx *m, int opts, const char *file, int line)
* is handled inline.
*/
void
-_mtx_lock_spin(struct mtx *m, int opts, const char *file, int line)
+_mtx_lock_spin(struct mtx *m, struct thread *td, int opts, const char *file,
+ int line)
{
int i = 0;
@@ -570,7 +571,7 @@ _mtx_lock_spin(struct mtx *m, int opts, const char *file, int line)
CTR1(KTR_LOCK, "_mtx_lock_spin: %p spinning", m);
for (;;) {
- if (_obtain_lock(m, curthread))
+ if (_obtain_lock(m, td))
break;
/* Give interrupts a chance while we spin. */
OpenPOWER on IntegriCloud