aboutsummaryrefslogtreecommitdiffstats
path: root/fs/xfs/xfs_mount.c
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@infradead.org>2008-08-13 02:49:04 -0400
committerLachlan McIlroy <lachlan@redback.melbourne.sgi.com>2008-08-13 02:49:04 -0400
commit77508ec8e6ed72c9ba8ca74248a7aabd664e3f57 (patch)
treed43fbde9c21fa21b8d3b791a18ee0d98af2f6102 /fs/xfs/xfs_mount.c
parent3a76c1ea07ee4e9da7c8e476e43be7e1b2bf24fb (diff)
[XFS] move root inode IRELE into xfs_unmountfs
The root inode is allocated in xfs_mountfs so it should be release in xfs_unmountfs. For the unmount case that means we do it after the the xfs_sync(mp, SYNC_WAIT | SYNC_CLOSE) in the forced shutdown case and the dmapi unmount event. Note that both reference the rip variable which might be freed by that time in case inode flushing has kicked in, so strictly speaking this might count as a bug fix SGI-PV: 981498 SGI-Modid: xfs-linux-melb:xfs-kern:31830a Signed-off-by: Christoph Hellwig <hch@infradead.org> Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_mount.c')
-rw-r--r--fs/xfs/xfs_mount.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c
index 6b4350700c0d..73f0ba1ec197 100644
--- a/fs/xfs/xfs_mount.c
+++ b/fs/xfs/xfs_mount.c
@@ -1250,6 +1250,8 @@ xfs_unmountfs(xfs_mount_t *mp)
1250 __uint64_t resblks; 1250 __uint64_t resblks;
1251 int error = 0; 1251 int error = 0;
1252 1252
1253 IRELE(mp->m_rootip);
1254
1253 /* 1255 /*
1254 * We can potentially deadlock here if we have an inode cluster 1256 * We can potentially deadlock here if we have an inode cluster
1255 * that has been freed has it's buffer still pinned in memory because 1257 * that has been freed has it's buffer still pinned in memory because