diff options
author | bde <bde@FreeBSD.org> | 2004-02-14 04:41:13 +0000 |
---|---|---|
committer | bde <bde@FreeBSD.org> | 2004-02-14 04:41:13 +0000 |
commit | 4dca0a78ca378a90328358f3bfee84726fb0ad36 (patch) | |
tree | 461dd67fde6923278570d56f8130b249cad7da27 /sys/ufs | |
parent | 67995bef83d9d8c84403dab3348106cf5b861d10 (diff) | |
download | FreeBSD-src-4dca0a78ca378a90328358f3bfee84726fb0ad36.zip FreeBSD-src-4dca0a78ca378a90328358f3bfee84726fb0ad36.tar.gz |
Fixed some style bugs:
- don't unlock the vnode after vinvalbuf() only to have to relock it
almost immediately.
- don't refer to devices classified by vn_isdisk() as block devices.
Diffstat (limited to 'sys/ufs')
-rw-r--r-- | sys/ufs/ffs/ffs_vfsops.c | 28 |
1 files changed, 10 insertions, 18 deletions
diff --git a/sys/ufs/ffs/ffs_vfsops.c b/sys/ufs/ffs/ffs_vfsops.c index 6486c31..298f462 100644 --- a/sys/ufs/ffs/ffs_vfsops.c +++ b/sys/ufs/ffs/ffs_vfsops.c @@ -313,7 +313,7 @@ ffs_mount(mp, path, data, ndp, td) /* * Not an update, or updating the name: look up the name - * and verify that it refers to a sensible block device. + * and verify that it refers to a sensible disk device. */ NDINIT(ndp, LOOKUP, FOLLOW, UIO_USERSPACE, args.fspec, td); if ((error = namei(ndp)) != 0) @@ -416,20 +416,15 @@ ffs_reload(mp, cred, td) */ devvp = VFSTOUFS(mp)->um_devvp; vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, td); - error = vinvalbuf(devvp, 0, cred, td, 0, 0); - VOP_UNLOCK(devvp, 0, td); - if (error) + if (vinvalbuf(devvp, 0, cred, td, 0, 0) != 0) panic("ffs_reload: dirty1"); - /* * Only VMIO the backing device if the backing device is a real - * block device. + * disk device. See ffs_mountfs() for more details. */ - if (vn_isdisk(devvp, NULL)) { - vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, td); + if (vn_isdisk(devvp, NULL)) vfs_object_create(devvp, td, td->td_ucred); - VOP_UNLOCK(devvp, 0, td); - } + VOP_UNLOCK(devvp, 0, td); /* * Step 2: re-read superblock from disk. @@ -588,24 +583,21 @@ ffs_mountfs(devvp, mp, td) return (EBUSY); vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, td); error = vinvalbuf(devvp, V_SAVE, cred, td, 0, 0); - VOP_UNLOCK(devvp, 0, td); - if (error) + if (error) { + VOP_UNLOCK(devvp, 0, td); return (error); + } /* * Only VMIO the backing device if the backing device is a real - * block device. + * disk device. * Note that it is optional that the backing device be VMIOed. This * increases the opportunity for metadata caching. */ - if (vn_isdisk(devvp, NULL)) { - vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, td); + if (vn_isdisk(devvp, NULL)) vfs_object_create(devvp, td, cred); - VOP_UNLOCK(devvp, 0, td); - } ronly = (mp->mnt_flag & MNT_RDONLY) != 0; - vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, td); /* * XXX: open the device with read and write access even if only * read access is needed now. Write access is needed if the |