diff options
| author | jeff <jeff@FreeBSD.org> | 2005-03-13 11:57:39 +0000 |
|---|---|---|
| committer | jeff <jeff@FreeBSD.org> | 2005-03-13 11:57:39 +0000 |
| commit | c376ae435b71dba28c26da9e532da125946047f0 (patch) | |
| tree | 1f0a80600907ef01f5aff24656b76130e9589f60 /sys/ufs | |
| parent | a71cdca338d54f3150d43c613b14946adb1b8cf6 (diff) | |
| download | FreeBSD-src-c376ae435b71dba28c26da9e532da125946047f0.zip FreeBSD-src-c376ae435b71dba28c26da9e532da125946047f0.tar.gz | |
- Don't drop the lock in ufs_inactive().
- Also in ufs_inactive, don't acquire the vnode interlock where it isn't
strictly needed. Also owning the vnode interlock while calling vprint()
will cause locking assertions to trip.
Sponsored by: Isilon Systems, Inc.
Diffstat (limited to 'sys/ufs')
| -rw-r--r-- | sys/ufs/ufs/ufs_inode.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/sys/ufs/ufs/ufs_inode.c b/sys/ufs/ufs/ufs_inode.c index 4cebcd2..6cce782 100644 --- a/sys/ufs/ufs/ufs_inode.c +++ b/sys/ufs/ufs/ufs_inode.c @@ -74,11 +74,8 @@ ufs_inactive(ap) mode_t mode; int error = 0; - VI_LOCK(vp); if (prtactive && vp->v_usecount != 0) vprint("ufs_inactive: pushing active", vp); - VI_UNLOCK(vp); - /* * Ignore inodes related to stale file handles. */ @@ -122,7 +119,6 @@ ufs_inactive(ap) } } out: - VOP_UNLOCK(vp, 0, td); /* * If we are done with the inode, reclaim it * so that it can be reused immediately. |
