diff options
| author | Dave Chinner <dchinner@redhat.com> | 2013-08-27 13:25:43 +1000 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2013-08-29 10:51:35 -0500 |
| commit | 0f0d334595105d982ea22ae1d5947723e462344f (patch) | |
| tree | a370da8a2aa1e0ae7207b87fe5419fbc857c13e4 /fs/xfs/xfs_log_recover.c | |
| parent | 84a5b7300c724f4000f689c410aeae3242b4f034 (diff) | |
| download | linux-0f0d334595105d982ea22ae1d5947723e462344f.tar.gz | |
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/xfs/xfs_log_recover.c')
| -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( ASSERT(dq_f->qlf_len == 1); xfs_buf_readahead(mp->m_ddev_targp, dq_f->qlf_blkno, - dq_f->qlf_len, NULL); + XFS_FSB_TO_BB(mp, dq_f->qlf_len), NULL); } STATIC void |