diff options
author | rwatson <rwatson@FreeBSD.org> | 2004-01-23 21:07:52 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2004-01-23 21:07:52 +0000 |
commit | 597deaa7b82de573cdd1aee2920ecf0bbce8ab0b (patch) | |
tree | 36e3b87c6461c0d0034aec33446e4e60242bd51c /sbin | |
parent | 67d83922288af8f8de614c65bfc3661494fc8e98 (diff) | |
download | FreeBSD-src-597deaa7b82de573cdd1aee2920ecf0bbce8ab0b.zip FreeBSD-src-597deaa7b82de573cdd1aee2920ecf0bbce8ab0b.tar.gz |
Don't grab Giant in crfree(), since prison_free() no longer requires it.
The uidinfo code appears to be MPSAFE, and is referenced without Giant
elsewhere. While this grab of Giant was only made in fairly rare
circumstances (actually GC'ing on refcount==0), grabbing Giant here
potentially introduces lock order issues with any locks held by the
caller. So this probably won't help performance much unless you change
credentials a lot in an application, and leave a lot of file descriptors
and cached credentials around. However, it simplifies locking down
consumers of the credential interfaces.
Bumped into by: sam
Appeased: tjr
Diffstat (limited to 'sbin')
0 files changed, 0 insertions, 0 deletions