diff options
author | Dave Chinner <dchinner@redhat.com> | 2013-06-05 12:09:08 +1000 |
---|---|---|
committer | Ben Myers <bpm@sgi.com> | 2013-06-05 11:19:10 -0500 |
commit | 0a32c26e720a8b38971d0685976f4a7d63f9e2ef (patch) | |
tree | 37e87629f9a62e91b67d29d16c757d74709aaf7f /fs/xfs/xfs_log_recover.c | |
parent | a775ad778073d55744ed6709ccede36310638911 (diff) | |
download | linux-stable-0a32c26e720a8b38971d0685976f4a7d63f9e2ef.tar.gz linux-stable-0a32c26e720a8b38971d0685976f4a7d63f9e2ef.tar.bz2 linux-stable-0a32c26e720a8b38971d0685976f4a7d63f9e2ef.zip |
xfs: inode unlinked list needs to recalculate the inode CRC
The inode unlinked list manipulations operate directly on the inode
buffer, and so bypass the inode CRC calculation mechanisms. Hence an
inode on the unlinked list has an invalid CRC. Fix this by
recalculating the CRC whenever we modify an unlinked list pointer in
an inode, ncluding during log recovery. This is trivial to do and
results in unlinked list operations always leaving a consistent
inode in the buffer.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Mark Tinguely <tinguely@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_log_recover.c')
-rw-r--r-- | fs/xfs/xfs_log_recover.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c index 83088d96e6c4..45a85ff84da1 100644 --- a/fs/xfs/xfs_log_recover.c +++ b/fs/xfs/xfs_log_recover.c @@ -1912,6 +1912,15 @@ xlog_recover_do_inode_buffer( buffer_nextp = (xfs_agino_t *)xfs_buf_offset(bp, next_unlinked_offset); *buffer_nextp = *logged_nextp; + + /* + * If necessary, recalculate the CRC in the on-disk inode. We + * have to leave the inode in a consistent state for whoever + * reads it next.... + */ + xfs_dinode_calc_crc(mp, (struct xfs_dinode *) + xfs_buf_offset(bp, i * mp->m_sb.sb_inodesize)); + } return 0; |