diff options
author | asomers <asomers@FreeBSD.org> | 2015-12-21 22:19:22 +0000 |
---|---|---|
committer | asomers <asomers@FreeBSD.org> | 2015-12-21 22:19:22 +0000 |
commit | ec468634856757e0c5bc091c7ffb4e64d1385397 (patch) | |
tree | 542cb66a20a783d96ac83927ea17c16a44054d27 /sbin/mount | |
parent | e072c1a126983056e33db2f81c093b7b720daea8 (diff) | |
download | FreeBSD-src-ec468634856757e0c5bc091c7ffb4e64d1385397.zip FreeBSD-src-ec468634856757e0c5bc091c7ffb4e64d1385397.tar.gz |
Fix "mount -a" for NFS and ZFS filesystems with shared mountpoints
sbin/mount.c
Check whether an fstab entry has the same fstype as a mounted
filesystem before declaring it to be mounted. This will allow NFS
filesystems that share a mountpoint with a local filesystem to be
automatically mounted at boot.
This is not such an unusual situation. For example, if somebody uses
the standard installer with a ZFS root, he'll get a /usr/home
filesystem, even though he may choose to mount /usr/home over NFS.
Reviewed by: trasz
MFC after: 4 weeks
Sponsored by: Spectra Logic Corp
Differential Revision: https://reviews.freebsd.org/D4556
Diffstat (limited to 'sbin/mount')
-rw-r--r-- | sbin/mount/mount.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/sbin/mount/mount.c b/sbin/mount/mount.c index 6427fc8..8c68467 100644 --- a/sbin/mount/mount.c +++ b/sbin/mount/mount.c @@ -485,10 +485,18 @@ ismounted(struct fstab *fs, struct statfs *mntbuf, int mntsize) strlcpy(realfsfile, fs->fs_file, sizeof(realfsfile)); } + /* + * Consider the filesystem to be mounted if: + * It has the same mountpoint as a mounted filesytem, and + * It has the same type as that same mounted filesystem, and + * It has the same device name as that same mounted filesystem, OR + * It is a nonremountable filesystem + */ for (i = mntsize - 1; i >= 0; --i) if (strcmp(realfsfile, mntbuf[i].f_mntonname) == 0 && + strcmp(fs->fs_vfstype, mntbuf[i].f_fstypename) == 0 && (!isremountable(fs->fs_vfstype) || - strcmp(fs->fs_spec, mntbuf[i].f_mntfromname) == 0)) + (strcmp(fs->fs_spec, mntbuf[i].f_mntfromname) == 0))) return (1); return (0); } |