diff options
author | bde <bde@FreeBSD.org> | 2004-02-12 16:52:24 +0000 |
---|---|---|
committer | bde <bde@FreeBSD.org> | 2004-02-12 16:52:24 +0000 |
commit | a2bb8cef87810c49e816ce1f3557b5e44468a29a (patch) | |
tree | d177a6394d8ddd2665382d2314d4f9e577a30868 /sys/ufs | |
parent | 9ced58b737baf0c407799f685ecbe50c13004ef5 (diff) | |
download | FreeBSD-src-a2bb8cef87810c49e816ce1f3557b5e44468a29a.zip FreeBSD-src-a2bb8cef87810c49e816ce1f3557b5e44468a29a.tar.gz |
Fixed some minor style bugs (English usage and formatting of binary
operators) in and near revs.1.169-1.170 (open mode bandaid). This
(or better a proper fix) should have been done before cloning the
bandaid to many other file systems.
Diffstat (limited to 'sys/ufs')
-rw-r--r-- | sys/ufs/ffs/ffs_vfsops.c | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/sys/ufs/ffs/ffs_vfsops.c b/sys/ufs/ffs/ffs_vfsops.c index 9e27c2c..6996a93 100644 --- a/sys/ufs/ffs/ffs_vfsops.c +++ b/sys/ufs/ffs/ffs_vfsops.c @@ -610,14 +610,15 @@ ffs_mountfs(devvp, mp, td) ronly = (mp->mnt_flag & MNT_RDONLY) != 0; vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, td); /* - * XXX: We don't re-VOP_OPEN in FREAD|FWRITE mode if the filesystem - * XXX: is subsequently remounted, so open it FREAD|FWRITE from the - * XXX: start to avoid getting trashed later on. + * XXX: open the device with read and write access even if only + * read access is needed now. Write access is needed if the + * filesystem is ever mounted read/write, and we don't change the + * access mode for remounts. */ #ifdef notyet - error = VOP_OPEN(devvp, ronly ? FREAD : FREAD|FWRITE, FSCRED, td, -1); + error = VOP_OPEN(devvp, ronly ? FREAD : FREAD | FWRITE, FSCRED, td, -1); #else - error = VOP_OPEN(devvp, FREAD|FWRITE, FSCRED, td, -1); + error = VOP_OPEN(devvp, FREAD | FWRITE, FSCRED, td, -1); #endif VOP_UNLOCK(devvp, 0, td); if (error) @@ -830,11 +831,11 @@ out: devvp->v_rdev->si_mountpoint = NULL; if (bp) brelse(bp); - /* XXX: see comment above VOP_OPEN */ + /* XXX: see comment above VOP_OPEN. */ #ifdef notyet - (void)VOP_CLOSE(devvp, ronly ? FREAD : FREAD|FWRITE, cred, td); + (void)VOP_CLOSE(devvp, ronly ? FREAD : FREAD | FWRITE, cred, td); #else - (void)VOP_CLOSE(devvp, FREAD|FWRITE, cred, td); + (void)VOP_CLOSE(devvp, FREAD | FWRITE, cred, td); #endif if (ump) { free(ump->um_fs, M_UFSMNT); @@ -990,12 +991,12 @@ ffs_unmount(mp, mntflags, td) ump->um_devvp->v_rdev->si_mountpoint = NULL; vinvalbuf(ump->um_devvp, V_SAVE, NOCRED, td, 0, 0); - /* XXX: see comment above VOP_OPEN */ + /* XXX: see comment above VOP_OPEN. */ #ifdef notyet - error = VOP_CLOSE(ump->um_devvp, fs->fs_ronly ? FREAD : FREAD|FWRITE, - NOCRED, td); + error = VOP_CLOSE(ump->um_devvp, fs->fs_ronly ? FREAD : FREAD | FWRITE, + NOCRED, td); #else - error = VOP_CLOSE(ump->um_devvp, FREAD|FWRITE, NOCRED, td); + error = VOP_CLOSE(ump->um_devvp, FREAD | FWRITE, NOCRED, td); #endif vrele(ump->um_devvp); |