diff options
author | mtm <mtm@FreeBSD.org> | 2003-07-18 21:26:45 +0000 |
---|---|---|
committer | mtm <mtm@FreeBSD.org> | 2003-07-18 21:26:45 +0000 |
commit | 390cdc2c2730f34e651f9e2ea581e8cb40f937c9 (patch) | |
tree | b3c089f22b8579d601f4cb6cdb7b0676bed1959b /sys | |
parent | 70174962dcdad77ba9abb6e25c27742ba8e013d3 (diff) | |
download | FreeBSD-src-390cdc2c2730f34e651f9e2ea581e8cb40f937c9.zip FreeBSD-src-390cdc2c2730f34e651f9e2ea581e8cb40f937c9.tar.gz |
Remove a lock held across casuptr() that snuck in last commit.
Diffstat (limited to 'sys')
-rw-r--r-- | sys/kern/kern_umtx.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/sys/kern/kern_umtx.c b/sys/kern/kern_umtx.c index 787155d..f0a99e3 100644 --- a/sys/kern/kern_umtx.c +++ b/sys/kern/kern_umtx.c @@ -292,10 +292,13 @@ _umtx_unlock(struct thread *td, struct _umtx_unlock_args *uap) uq = umtx_lookup(td, umtx); if (uq != NULL && ((blocked = TAILQ_FIRST(&uq->uq_tdq)) != NULL && - TAILQ_NEXT(blocked, td_umtx) != NULL)) + TAILQ_NEXT(blocked, td_umtx) != NULL)) { + UMTX_UNLOCK(); old = casuptr((intptr_t *)&umtx->u_owner, UMTX_UNOWNED, UMTX_CONTESTED); - UMTX_UNLOCK(); + } else { + UMTX_UNLOCK(); + } } else { UMTX_UNLOCK(); old = casuptr((intptr_t *)&umtx->u_owner, |