diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2006-10-13 17:25:45 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2006-10-20 09:13:40 -0400 |
commit | 23591256d61354e20f12e98d7a496ad5c23de74c (patch) | |
tree | 3013a6e873a6e0a1d55fbace2fba638a978108c0 /fs/gfs2/log.c | |
parent | 42fb00838a644d03f9a2a5fbbe0b668a5ff5df4d (diff) |
[GFS2] Fix bmap to map extents properly
This fix means that bmap will map extents of the length requested
by the VFS rather than guessing at it, or just mapping one block
at a time. The other callers of gfs2_block_map are audited to ensure
they send the correct max extent lengths (i.e. set bh->b_size correctly).
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/log.c')
-rw-r--r-- | fs/gfs2/log.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/gfs2/log.c b/fs/gfs2/log.c index 72eec6542d6a..0cace3da9dbb 100644 --- a/fs/gfs2/log.c +++ b/fs/gfs2/log.c | |||
@@ -312,10 +312,12 @@ void gfs2_log_release(struct gfs2_sbd *sdp, unsigned int blks) | |||
312 | 312 | ||
313 | static u64 log_bmap(struct gfs2_sbd *sdp, unsigned int lbn) | 313 | static u64 log_bmap(struct gfs2_sbd *sdp, unsigned int lbn) |
314 | { | 314 | { |
315 | struct inode *inode = sdp->sd_jdesc->jd_inode; | ||
315 | int error; | 316 | int error; |
316 | struct buffer_head bh_map; | 317 | struct buffer_head bh_map = { .b_state = 0, .b_blocknr = 0 }; |
317 | 318 | ||
318 | error = gfs2_block_map(sdp->sd_jdesc->jd_inode, lbn, 0, &bh_map, 1); | 319 | bh_map.b_size = 1 << inode->i_blkbits; |
320 | error = gfs2_block_map(inode, lbn, 0, &bh_map); | ||
319 | if (error || !bh_map.b_blocknr) | 321 | if (error || !bh_map.b_blocknr) |
320 | printk(KERN_INFO "error=%d, dbn=%llu lbn=%u", error, bh_map.b_blocknr, lbn); | 322 | printk(KERN_INFO "error=%d, dbn=%llu lbn=%u", error, bh_map.b_blocknr, lbn); |
321 | gfs2_assert_withdraw(sdp, !error && bh_map.b_blocknr); | 323 | gfs2_assert_withdraw(sdp, !error && bh_map.b_blocknr); |