diff options
author | Dave Chinner <dchinner@redhat.com> | 2013-08-26 23:25:43 -0400 |
---|---|---|
committer | Ben Myers <bpm@sgi.com> | 2013-08-29 11:51:35 -0400 |
commit | 0f0d334595105d982ea22ae1d5947723e462344f (patch) | |
tree | a370da8a2aa1e0ae7207b87fe5419fbc857c13e4 /fs | |
parent | 84a5b7300c724f4000f689c410aeae3242b4f034 (diff) |
xfs: fix bad dquot buffer size in log recovery readahead
xfstests xfs/087 fails 100% reliably with this assert:
XFS (vdb): Mounting Filesystem
XFS (vdb): Starting recovery (logdev: internal)
XFS: Assertion failed: bp->b_flags & XBF_STALE, file: fs/xfs/xfs_buf.c, line: 548
while trying to read a dquot buffer in xlog_recover_dquot_ra_pass2().
The issue is that the buffer length to read that is passed to
xfs_buf_readahead is in units of filesystem blocks, not disk blocks.
(i.e. FSB, not daddr). Fix it but putting the correct conversion in
place.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Ben Myers <bpm@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/xfs/xfs_log_recover.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c index 90b756f7117c..006ee288246d 100644 --- a/fs/xfs/xfs_log_recover.c +++ b/fs/xfs/xfs_log_recover.c | |||
@@ -3202,7 +3202,7 @@ xlog_recover_dquot_ra_pass2( | |||
3202 | ASSERT(dq_f->qlf_len == 1); | 3202 | ASSERT(dq_f->qlf_len == 1); |
3203 | 3203 | ||
3204 | xfs_buf_readahead(mp->m_ddev_targp, dq_f->qlf_blkno, | 3204 | xfs_buf_readahead(mp->m_ddev_targp, dq_f->qlf_blkno, |
3205 | dq_f->qlf_len, NULL); | 3205 | XFS_FSB_TO_BB(mp, dq_f->qlf_len), NULL); |
3206 | } | 3206 | } |
3207 | 3207 | ||
3208 | STATIC void | 3208 | STATIC void |