diff options
author | jhb <jhb@FreeBSD.org> | 2002-03-20 21:09:09 +0000 |
---|---|---|
committer | jhb <jhb@FreeBSD.org> | 2002-03-20 21:09:09 +0000 |
commit | 715dfdbcbe8ff5e275313a83ef7be76229d87ebf (patch) | |
tree | c0aa5a835f69a0ccf83503fd8b1f6284161207b7 /sys/alpha | |
parent | 64bf9fe9fa4ffdff0c7bb06726e70267970c7f02 (diff) | |
download | FreeBSD-src-715dfdbcbe8ff5e275313a83ef7be76229d87ebf.zip FreeBSD-src-715dfdbcbe8ff5e275313a83ef7be76229d87ebf.tar.gz |
Change the way we ensure td_ucred is NULL if DIAGNOSTIC is defined.
Instead of caching the ucred reference, just go ahead and eat the
decerement and increment of the refcount. Now that Giant is pushed down
into crfree(), we no longer have to get Giant in the common case. In the
case when we are actually free'ing the ucred, we would normally free it on
the next kernel entry, so the cost there is not new, just in a different
place. This also removse td_cache_ucred from struct thread. This is
still only done #ifdef DIAGNOSTIC.
Tested on: i386, alpha
Diffstat (limited to 'sys/alpha')
-rw-r--r-- | sys/alpha/alpha/trap.c | 31 |
1 files changed, 6 insertions, 25 deletions
diff --git a/sys/alpha/alpha/trap.c b/sys/alpha/alpha/trap.c index bbd1d5a..9c2ea34 100644 --- a/sys/alpha/alpha/trap.c +++ b/sys/alpha/alpha/trap.c @@ -297,13 +297,6 @@ trap(a0, a1, a2, entry, framep) if (user) { sticks = td->td_kse->ke_sticks; td->td_frame = framep; -#ifdef DIAGNOSTIC - /* see the comment in ast() */ - if (td->td_ucred != NULL) - panic("trap(): thread got a ucred while in userspace"); - td->td_ucred = td->td_ucred_cache; - td->td_ucred_cache = NULL; -#endif if (td->td_ucred != p->p_ucred) cred_update_thread(td); } else { @@ -631,12 +624,9 @@ out: framep->tf_regs[FRAME_SP] = alpha_pal_rdusp(); userret(td, framep, sticks); mtx_assert(&Giant, MA_NOTOWNED); -#ifdef DIAGNOSTIC /* see the comment in ast() */ - if (td->td_ucred_cache) - panic("trap: thread already has cached ucred"); - td->td_ucred_cache = td->td_ucred; - td->td_ucred = NULL; -#endif /* DIAGNOSTIC */ +#ifdef DIAGNOSTIC + cred_free_thread(td); +#endif } return; @@ -705,12 +695,6 @@ syscall(code, framep) td->td_frame = framep; opc = framep->tf_regs[FRAME_PC] - 4; sticks = td->td_kse->ke_sticks; -#ifdef DIAGNOSTIC /* see the comment in ast() */ - if (td->td_ucred) - panic("syscall:thread got a cred while in userspace"); - td->td_ucred = td->td_ucred_cache; - td->td_ucred_cache = NULL; -#endif /* DIAGNOSTIC */ if (td->td_ucred != p->p_ucred) cred_update_thread(td); @@ -833,12 +817,9 @@ syscall(code, framep) */ STOPEVENT(p, S_SCX, code); -#ifdef DIAGNOSTIC /* see the comment in ast() */ - if (td->td_ucred_cache) - panic("syscall:thread already has cached ucred"); - td->td_ucred_cache = td->td_ucred; - td->td_ucred = NULL; -#endif /* DIAGNOSTIC */ +#ifdef DIAGNOSTIC + cred_free_thread(td); +#endif #ifdef WITNESS if (witness_list(td)) { panic("system call %s returning with mutex(s) held\n", |