summaryrefslogtreecommitdiff
path: root/fs/xfs
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@infradead.org>2010-04-28 12:28:54 +0000
committerAlex Elder <aelder@sgi.com>2010-05-19 09:58:17 -0500
commit8699bb0a480193e62d5ccb9c86e2c26b407090a8 (patch)
treeaabcc9cd0800cf5c0d0605d46a104ed825f90db6 /fs/xfs
parent9563b3d8998c78d5b7e718b546d5f68037c494fe (diff)
xfs: report iomap_offset and iomap_bsize in block base
Report the iomap_offset and iomap_bsize fields of struct xfs_iomap in terms of fsblocks instead of in terms of disk blocks. Shift the byte conversions into the callers temporarily, but they will disappear or get cleaned up later. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Alex Elder <aelder@sgi.com>
Diffstat (limited to 'fs/xfs')
-rw-r--r--fs/xfs/linux-2.6/xfs_aops.c36
-rw-r--r--fs/xfs/xfs_iomap.c5
2 files changed, 27 insertions, 14 deletions
diff --git a/fs/xfs/linux-2.6/xfs_aops.c b/fs/xfs/linux-2.6/xfs_aops.c
index 577d0b448732..303a779406c0 100644
--- a/fs/xfs/linux-2.6/xfs_aops.c
+++ b/fs/xfs/linux-2.6/xfs_aops.c
@@ -320,11 +320,16 @@ xfs_map_blocks(
STATIC int
xfs_iomap_valid(
+ struct inode *inode,
xfs_iomap_t *iomapp,
loff_t offset)
{
- return offset >= iomapp->iomap_offset &&
- offset < iomapp->iomap_offset + iomapp->iomap_bsize;
+ struct xfs_mount *mp = XFS_I(inode)->i_mount;
+ xfs_off_t iomap_offset = XFS_FSB_TO_B(mp, iomapp->iomap_offset);
+ xfs_off_t iomap_bsize = XFS_FSB_TO_B(mp, iomapp->iomap_bsize);
+
+ return offset >= iomap_offset &&
+ offset < iomap_offset + iomap_bsize;
}
/*
@@ -561,11 +566,13 @@ xfs_map_buffer(
xfs_off_t offset)
{
sector_t bn;
+ struct xfs_mount *m = XFS_I(inode)->i_mount;
+ xfs_off_t iomap_offset = XFS_FSB_TO_B(m, mp->iomap_offset);
ASSERT(mp->iomap_bn != IOMAP_DADDR_NULL);
bn = (mp->iomap_bn >> (inode->i_blkbits - BBSHIFT)) +
- ((offset - mp->iomap_offset) >> inode->i_blkbits);
+ ((offset - iomap_offset) >> inode->i_blkbits);
ASSERT(bn || XFS_IS_REALTIME_INODE(XFS_I(inode)));
@@ -806,7 +813,7 @@ xfs_convert_page(
else
type = IOMAP_DELAY;
- if (!xfs_iomap_valid(mp, offset)) {
+ if (!xfs_iomap_valid(inode, mp, offset)) {
done = 1;
continue;
}
@@ -1116,7 +1123,7 @@ xfs_page_state_convert(
}
if (iomap_valid)
- iomap_valid = xfs_iomap_valid(&iomap, offset);
+ iomap_valid = xfs_iomap_valid(inode, &iomap, offset);
/*
* First case, map an unwritten extent and prepare for
@@ -1171,7 +1178,7 @@ xfs_page_state_convert(
&iomap, flags);
if (err)
goto error;
- iomap_valid = xfs_iomap_valid(&iomap, offset);
+ iomap_valid = xfs_iomap_valid(inode, &iomap, offset);
}
if (iomap_valid) {
xfs_map_at_offset(inode, bh, &iomap, offset);
@@ -1201,7 +1208,7 @@ xfs_page_state_convert(
&iomap, flags);
if (err)
goto error;
- iomap_valid = xfs_iomap_valid(&iomap, offset);
+ iomap_valid = xfs_iomap_valid(inode, &iomap, offset);
}
/*
@@ -1241,7 +1248,11 @@ xfs_page_state_convert(
xfs_start_page_writeback(page, 1, count);
if (ioend && iomap_valid) {
- offset = (iomap.iomap_offset + iomap.iomap_bsize - 1) >>
+ struct xfs_mount *m = XFS_I(inode)->i_mount;
+ xfs_off_t iomap_offset = XFS_FSB_TO_B(m, iomap.iomap_offset);
+ xfs_off_t iomap_bsize = XFS_FSB_TO_B(m, iomap.iomap_bsize);
+
+ offset = (iomap_offset + iomap_bsize - 1) >>
PAGE_CACHE_SHIFT;
tlast = min_t(pgoff_t, offset, last_index);
xfs_cluster_write(inode, page->index + 1, &iomap, &ioend,
@@ -1512,11 +1523,14 @@ __xfs_get_blocks(
}
if (direct || size > (1 << inode->i_blkbits)) {
- xfs_off_t iomap_delta = offset - iomap.iomap_offset;
+ struct xfs_mount *mp = XFS_I(inode)->i_mount;
+ xfs_off_t iomap_offset = XFS_FSB_TO_B(mp, iomap.iomap_offset);
+ xfs_off_t iomap_delta = offset - iomap_offset;
+ xfs_off_t iomap_bsize = XFS_FSB_TO_B(mp, iomap.iomap_bsize);
- ASSERT(iomap.iomap_bsize - iomap_delta > 0);
+ ASSERT(iomap_bsize - iomap_delta > 0);
offset = min_t(xfs_off_t,
- iomap.iomap_bsize - iomap_delta, size);
+ iomap_bsize - iomap_delta, size);
bh_result->b_size = (ssize_t)min_t(xfs_off_t, LONG_MAX, offset);
}
diff --git a/fs/xfs/xfs_iomap.c b/fs/xfs/xfs_iomap.c
index c6b409e0f013..49b5ad22a9d8 100644
--- a/fs/xfs/xfs_iomap.c
+++ b/fs/xfs/xfs_iomap.c
@@ -64,11 +64,10 @@ xfs_imap_to_bmap(
int imaps, /* Number of imap entries */
int flags)
{
- xfs_mount_t *mp = ip->i_mount;
xfs_fsblock_t start_block;
- iomapp->iomap_offset = XFS_FSB_TO_B(mp, imap->br_startoff);
- iomapp->iomap_bsize = XFS_FSB_TO_B(mp, imap->br_blockcount);
+ iomapp->iomap_offset = imap->br_startoff;
+ iomapp->iomap_bsize = imap->br_blockcount;
iomapp->iomap_flags = flags;
start_block = imap->br_startblock;