diff options
author | Christoph Hellwig <hch@infradead.org> | 2007-08-30 03:20:53 -0400 |
---|---|---|
committer | Tim Shimmin <tes@chook.melbourne.sgi.com> | 2007-10-15 21:45:02 -0400 |
commit | 0ce4cfd4f7dde5891d5b3e3c1a28ff7a7b4d36b3 (patch) | |
tree | 7ccb7197135e6705adcf51a40b1dea0a102ef213 /fs/xfs/xfs_mount.c | |
parent | 745f691912b700ac98607b525f3c892204c7f12f (diff) |
[XFS] kill the vfs_fsid and vfs_altfsid members in struct bhv_vfs
vfs_altfsid was just a pointer to mp->m_fixedfsid so we can trivially
replace it with the latter. vfs_fsid also was identical to m_fixedfsid
through rather obfuscated ways so we can kill it as well and simply its
only user.
SGI-PV: 969608
SGI-Modid: xfs-linux-melb:xfs-kern:29506a
Signed-off-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: David Chinner <dgc@sgi.com>
Signed-off-by: Tim Shimmin <tes@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_mount.c')
-rw-r--r-- | fs/xfs/xfs_mount.c | 22 |
1 files changed, 1 insertions, 21 deletions
diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c index 4f3488a7dc06..87bc9280b786 100644 --- a/fs/xfs/xfs_mount.c +++ b/fs/xfs/xfs_mount.c | |||
@@ -875,14 +875,11 @@ xfs_mountfs( | |||
875 | */ | 875 | */ |
876 | if ((mfsi_flags & XFS_MFSI_SECOND) == 0 && | 876 | if ((mfsi_flags & XFS_MFSI_SECOND) == 0 && |
877 | (mp->m_flags & XFS_MOUNT_NOUUID) == 0) { | 877 | (mp->m_flags & XFS_MOUNT_NOUUID) == 0) { |
878 | __uint64_t ret64; | ||
879 | if (xfs_uuid_mount(mp)) { | 878 | if (xfs_uuid_mount(mp)) { |
880 | error = XFS_ERROR(EINVAL); | 879 | error = XFS_ERROR(EINVAL); |
881 | goto error1; | 880 | goto error1; |
882 | } | 881 | } |
883 | uuid_mounted=1; | 882 | uuid_mounted=1; |
884 | ret64 = uuid_hash64(&sbp->sb_uuid); | ||
885 | memcpy(&vfsp->vfs_fsid, &ret64, sizeof(ret64)); | ||
886 | } | 883 | } |
887 | 884 | ||
888 | /* | 885 | /* |
@@ -1007,16 +1004,6 @@ xfs_mountfs( | |||
1007 | */ | 1004 | */ |
1008 | uuid_getnodeuniq(&sbp->sb_uuid, mp->m_fixedfsid); | 1005 | uuid_getnodeuniq(&sbp->sb_uuid, mp->m_fixedfsid); |
1009 | 1006 | ||
1010 | /* | ||
1011 | * The vfs structure needs to have a file system independent | ||
1012 | * way of checking for the invariant file system ID. Since it | ||
1013 | * can't look at mount structures it has a pointer to the data | ||
1014 | * in the mount structure. | ||
1015 | * | ||
1016 | * File systems that don't support user level file handles (i.e. | ||
1017 | * all of them except for XFS) will leave vfs_altfsid as NULL. | ||
1018 | */ | ||
1019 | vfsp->vfs_altfsid = (xfs_fsid_t *)mp->m_fixedfsid; | ||
1020 | mp->m_dmevmask = 0; /* not persistent; set after each mount */ | 1007 | mp->m_dmevmask = 0; /* not persistent; set after each mount */ |
1021 | 1008 | ||
1022 | xfs_dir_mount(mp); | 1009 | xfs_dir_mount(mp); |
@@ -1206,9 +1193,6 @@ int | |||
1206 | xfs_unmountfs(xfs_mount_t *mp, struct cred *cr) | 1193 | xfs_unmountfs(xfs_mount_t *mp, struct cred *cr) |
1207 | { | 1194 | { |
1208 | struct bhv_vfs *vfsp = XFS_MTOVFS(mp); | 1195 | struct bhv_vfs *vfsp = XFS_MTOVFS(mp); |
1209 | #if defined(DEBUG) || defined(INDUCE_IO_ERROR) | ||
1210 | int64_t fsid; | ||
1211 | #endif | ||
1212 | __uint64_t resblks; | 1196 | __uint64_t resblks; |
1213 | 1197 | ||
1214 | /* | 1198 | /* |
@@ -1272,11 +1256,7 @@ xfs_unmountfs(xfs_mount_t *mp, struct cred *cr) | |||
1272 | xfs_uuid_unmount(mp); | 1256 | xfs_uuid_unmount(mp); |
1273 | 1257 | ||
1274 | #if defined(DEBUG) || defined(INDUCE_IO_ERROR) | 1258 | #if defined(DEBUG) || defined(INDUCE_IO_ERROR) |
1275 | /* | 1259 | xfs_errortag_clearall(mp, 0); |
1276 | * clear all error tags on this filesystem | ||
1277 | */ | ||
1278 | memcpy(&fsid, &vfsp->vfs_fsid, sizeof(int64_t)); | ||
1279 | xfs_errortag_clearall_umount(fsid, mp->m_fsname, 0); | ||
1280 | #endif | 1260 | #endif |
1281 | XFS_IODONE(vfsp); | 1261 | XFS_IODONE(vfsp); |
1282 | xfs_mount_free(mp); | 1262 | xfs_mount_free(mp); |