diff options
author | phk <phk@FreeBSD.org> | 2005-03-15 08:07:07 +0000 |
---|---|---|
committer | phk <phk@FreeBSD.org> | 2005-03-15 08:07:07 +0000 |
commit | 124bf5e823dfe2ddde94c57660a3beffc2f99b57 (patch) | |
tree | 222a537cdd030d8083d8dae9e650ae3be7de0573 /sys/fs/hpfs | |
parent | a7b3bd456408d157545ff8c8ffd7060ae2070886 (diff) | |
download | FreeBSD-src-124bf5e823dfe2ddde94c57660a3beffc2f99b57.zip FreeBSD-src-124bf5e823dfe2ddde94c57660a3beffc2f99b57.tar.gz |
Simplify the vfs_hash calling convention.
Diffstat (limited to 'sys/fs/hpfs')
-rw-r--r-- | sys/fs/hpfs/hpfs_vfsops.c | 27 |
1 files changed, 3 insertions, 24 deletions
diff --git a/sys/fs/hpfs/hpfs_vfsops.c b/sys/fs/hpfs/hpfs_vfsops.c index ae96087..e3f24e9 100644 --- a/sys/fs/hpfs/hpfs_vfsops.c +++ b/sys/fs/hpfs/hpfs_vfsops.c @@ -459,16 +459,13 @@ hpfs_vget( struct vnode *vp; struct hpfsnode *hp; struct buf *bp; - struct thread *td = curthread; /* XXX */ int error; dprintf(("hpfs_vget(0x%x): ",ino)); error = vfs_hash_get(mp, ino, flags, curthread, vpp); - if (error) + if (error || *vpp != NULL) return (error); - if (*vpp != NULL) - return (0); *vpp = NULL; hp = NULL; @@ -516,30 +513,12 @@ hpfs_vget( hp->h_mode = hpmp->hpm_mode; hp->h_devvp = hpmp->hpm_devvp; - /* - * Exclusively lock the vnode before adding to hash. Note, that we - * must not release nor downgrade the lock (despite flags argument - * says) till it is fully initialized. - */ - lockmgr(vp->v_vnlock, LK_EXCLUSIVE, (struct mtx *)0, td); - - /* - * Atomicaly (in terms of vfs_hash operations) check the hash for - * duplicate of vnode being created and add it to the hash. If a - * duplicate vnode was found, it will be vget()ed from hash for us. - */ - if ((error = vfs_hash_insert(vp, ino, flags, curthread, vpp)) != 0) { + error = vfs_hash_insert(vp, ino, flags, curthread, vpp); + if (error || *vpp) { vput(vp); - *vpp = NULL; return (error); } - /* We lost the race, then throw away our vnode and return existing */ - if (*vpp != NULL) { - vput(vp); - return (0); - } - VREF(hp->h_devvp); error = bread(hpmp->hpm_devvp, ino, FNODESIZE, NOCRED, &bp); |