diff options
author | Christoph Hellwig <hch@infradead.org> | 2008-08-13 02:49:04 -0400 |
---|---|---|
committer | Lachlan McIlroy <lachlan@redback.melbourne.sgi.com> | 2008-08-13 02:49:04 -0400 |
commit | 77508ec8e6ed72c9ba8ca74248a7aabd664e3f57 (patch) | |
tree | d43fbde9c21fa21b8d3b791a18ee0d98af2f6102 /fs/xfs/xfs_mount.c | |
parent | 3a76c1ea07ee4e9da7c8e476e43be7e1b2bf24fb (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.c | 2 |
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 |