From 5bd51ec6e6125eed91ed13102532414e08b88d96 Mon Sep 17 00:00:00 2001 From: jeff Date: Sun, 13 Mar 2005 12:14:56 +0000 Subject: - The VI_DOOMED flag now signals the end of a vnode's relationship with the filesystem. Check that rather than VI_XLOCK. Sponsored by: Isilon Systems, Inc. --- sys/coda/coda_subr.c | 2 +- sys/fs/coda/coda_subr.c | 2 +- sys/fs/devfs/devfs_vnops.c | 11 ++--------- sys/nfsclient/nfs_bio.c | 2 +- sys/nfsclient/nfs_subs.c | 2 +- 5 files changed, 6 insertions(+), 13 deletions(-) (limited to 'sys') diff --git a/sys/coda/coda_subr.c b/sys/coda/coda_subr.c index 99eb883..878bd68 100644 --- a/sys/coda/coda_subr.c +++ b/sys/coda/coda_subr.c @@ -315,7 +315,7 @@ coda_checkunmounting(mp) MNT_ILOCK(mp); MNT_VNODE_FOREACH(vp, mp, nvp) { VI_LOCK(vp); - if (vp->v_iflag & VI_XLOCK) { + if (vp->v_iflag & VI_DOOMED) { VI_UNLOCK(vp); continue; } diff --git a/sys/fs/coda/coda_subr.c b/sys/fs/coda/coda_subr.c index 99eb883..878bd68 100644 --- a/sys/fs/coda/coda_subr.c +++ b/sys/fs/coda/coda_subr.c @@ -315,7 +315,7 @@ coda_checkunmounting(mp) MNT_ILOCK(mp); MNT_VNODE_FOREACH(vp, mp, nvp) { VI_LOCK(vp); - if (vp->v_iflag & VI_XLOCK) { + if (vp->v_iflag & VI_DOOMED) { VI_UNLOCK(vp); continue; } diff --git a/sys/fs/devfs/devfs_vnops.c b/sys/fs/devfs/devfs_vnops.c index eef811f..0fa8e30 100644 --- a/sys/fs/devfs/devfs_vnops.c +++ b/sys/fs/devfs/devfs_vnops.c @@ -329,7 +329,7 @@ devfs_close(ap) if (td && vp == td->td_proc->p_session->s_ttyvp) { SESS_LOCK(td->td_proc->p_session); VI_LOCK(vp); - if (count_dev(dev) == 2 && (vp->v_iflag & VI_XLOCK) == 0) { + if (count_dev(dev) == 2 && (vp->v_iflag & VI_DOOMED) == 0) { td->td_proc->p_session->s_ttyvp = NULL; oldvp = vp; } @@ -352,7 +352,7 @@ devfs_close(ap) if (dsw == NULL) return (ENXIO); VI_LOCK(vp); - if (vp->v_iflag & VI_XLOCK) { + if (vp->v_iflag & VI_DOOMED) { /* Forced close. */ } else if (dsw->d_flags & D_TRACKCLOSE) { /* Keep device updated on status. */ @@ -1143,13 +1143,6 @@ devfs_revoke(ap) KASSERT((ap->a_flags & REVOKEALL) != 0, ("devfs_revoke !REVOKEALL")); - /* - * If a vgone (or vclean) is already in progress, - * wait until it is done and return. - */ - if (vx_wait(vp)) - return (0); - dev = vp->v_rdev; for (;;) { dev_lock(); diff --git a/sys/nfsclient/nfs_bio.c b/sys/nfsclient/nfs_bio.c index f0f5d53..f877850 100644 --- a/sys/nfsclient/nfs_bio.c +++ b/sys/nfsclient/nfs_bio.c @@ -1235,7 +1235,7 @@ nfs_vinvalbuf(struct vnode *vp, int flags, struct thread *td, int intrflg) * being called through vclean(). It is not clear that this is * unsafe. */ - if (vp->v_iflag & VI_XLOCK) + if (vp->v_iflag & VI_DOOMED) return (0); if ((nmp->nm_flag & NFSMNT_INT) == 0) diff --git a/sys/nfsclient/nfs_subs.c b/sys/nfsclient/nfs_subs.c index 5a23acf..4fe6ee4 100644 --- a/sys/nfsclient/nfs_subs.c +++ b/sys/nfsclient/nfs_subs.c @@ -802,7 +802,7 @@ nfs_clearcommit(struct mount *mp) MNT_ILOCK(mp); MNT_VNODE_FOREACH(vp, mp, nvp) { VI_LOCK(vp); - if (vp->v_iflag & VI_XLOCK) { + if (vp->v_iflag & VI_DOOMED) { VI_UNLOCK(vp); continue; } -- cgit v1.1