diff options
| author | jeff <jeff@FreeBSD.org> | 2005-03-13 12:11:23 +0000 |
|---|---|---|
| committer | jeff <jeff@FreeBSD.org> | 2005-03-13 12:11:23 +0000 |
| commit | 5f59e0cd197952e4dfacaded55511c1117abbecd (patch) | |
| tree | 3f5a1d6f83038a5fcaa4151f79b10ea4886756a3 /sys/nfsclient | |
| parent | 1c1edae55c3cc94bc3225e577f7d477761028b16 (diff) | |
| download | FreeBSD-src-5f59e0cd197952e4dfacaded55511c1117abbecd.zip FreeBSD-src-5f59e0cd197952e4dfacaded55511c1117abbecd.tar.gz | |
- It is no longer necessary to lock and unlock the vnode in nfs_close() as
the top level does this for us now.
Sponsored by: Isilon Systems, Inc.
Diffstat (limited to 'sys/nfsclient')
| -rw-r--r-- | sys/nfsclient/nfs_vnops.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/sys/nfsclient/nfs_vnops.c b/sys/nfsclient/nfs_vnops.c index 5451fff..d3059a26 100644 --- a/sys/nfsclient/nfs_vnops.c +++ b/sys/nfsclient/nfs_vnops.c @@ -537,11 +537,8 @@ nfs_close(struct vop_close_args *ap) int cm = nfsv3_commit_on_close ? 1 : 0; error = nfs_flush(vp, MNT_WAIT, ap->a_td, cm); /* np->n_flag &= ~NMODIFIED; */ - } else { - VOP_LOCK(vp, LK_EXCLUSIVE, curthread); + } else error = nfs_vinvalbuf(vp, V_SAVE, ap->a_td, 1); - VOP_UNLOCK(vp, 0, curthread); - } } /* * Invalidate the attribute cache in all cases. @@ -3144,9 +3141,7 @@ nfsfifo_close(struct vop_close_args *ap) vattr.va_atime = np->n_atim; if (np->n_flag & NUPD) vattr.va_mtime = np->n_mtim; - vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, ap->a_td); (void)VOP_SETATTR(vp, &vattr, ap->a_cred, ap->a_td); - VOP_UNLOCK(vp, 0, ap->a_td); } } return (fifo_specops.vop_close(ap)); |
