diff options
author | Eric Sandeen <sandeen@sandeen.net> | 2012-03-28 13:21:11 -0400 |
---|---|---|
committer | Ben Myers <bpm@sgi.com> | 2013-07-09 14:21:15 -0400 |
commit | a69c7c077228b0bef38ccb1d385c099e132fe54b (patch) | |
tree | a94f8ab8798dd70b0b272ece0f95c1f0e55d7691 /fs/xfs/xfs_inode.c | |
parent | 83e782e1a1cc0159888e58e14dfc8f3289663338 (diff) |
xfs: use XFS_BMAP_BMDR_SPACE vs. XFS_BROOT_SIZE_ADJ
XFS_BROOT_SIZE_ADJ is an undocumented macro which accounts for
the difference in size between the on-disk and in-core btree
root. It's much clearer to just use the newly-added
XFS_BMAP_BMDR_SPACE macro which gives us the on-disk size
directly.
In one case, we must test that the if_broot exists before
applying the macro, however.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-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_inode.c')
-rw-r--r-- | fs/xfs/xfs_inode.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c index 9ecfe1e559fc..b78481f99d9d 100644 --- a/fs/xfs/xfs_inode.c +++ b/fs/xfs/xfs_inode.c | |||
@@ -2156,8 +2156,8 @@ xfs_iroot_realloc( | |||
2156 | np = (char *)XFS_BMAP_BROOT_PTR_ADDR(mp, ifp->if_broot, 1, | 2156 | np = (char *)XFS_BMAP_BROOT_PTR_ADDR(mp, ifp->if_broot, 1, |
2157 | (int)new_size); | 2157 | (int)new_size); |
2158 | ifp->if_broot_bytes = (int)new_size; | 2158 | ifp->if_broot_bytes = (int)new_size; |
2159 | ASSERT(ifp->if_broot_bytes <= | 2159 | ASSERT(XFS_BMAP_BMDR_SPACE(ifp->if_broot) <= |
2160 | XFS_IFORK_SIZE(ip, whichfork) + XFS_BROOT_SIZE_ADJ(ip)); | 2160 | XFS_IFORK_SIZE(ip, whichfork)); |
2161 | memmove(np, op, cur_max * (uint)sizeof(xfs_dfsbno_t)); | 2161 | memmove(np, op, cur_max * (uint)sizeof(xfs_dfsbno_t)); |
2162 | return; | 2162 | return; |
2163 | } | 2163 | } |
@@ -2210,8 +2210,9 @@ xfs_iroot_realloc( | |||
2210 | kmem_free(ifp->if_broot); | 2210 | kmem_free(ifp->if_broot); |
2211 | ifp->if_broot = new_broot; | 2211 | ifp->if_broot = new_broot; |
2212 | ifp->if_broot_bytes = (int)new_size; | 2212 | ifp->if_broot_bytes = (int)new_size; |
2213 | ASSERT(ifp->if_broot_bytes <= | 2213 | if (ifp->if_broot) |
2214 | XFS_IFORK_SIZE(ip, whichfork) + XFS_BROOT_SIZE_ADJ(ip)); | 2214 | ASSERT(XFS_BMAP_BMDR_SPACE(ifp->if_broot) <= |
2215 | XFS_IFORK_SIZE(ip, whichfork)); | ||
2215 | return; | 2216 | return; |
2216 | } | 2217 | } |
2217 | 2218 | ||
@@ -2522,9 +2523,8 @@ xfs_iflush_fork( | |||
2522 | if ((iip->ili_fields & brootflag[whichfork]) && | 2523 | if ((iip->ili_fields & brootflag[whichfork]) && |
2523 | (ifp->if_broot_bytes > 0)) { | 2524 | (ifp->if_broot_bytes > 0)) { |
2524 | ASSERT(ifp->if_broot != NULL); | 2525 | ASSERT(ifp->if_broot != NULL); |
2525 | ASSERT(ifp->if_broot_bytes <= | 2526 | ASSERT(XFS_BMAP_BMDR_SPACE(ifp->if_broot) <= |
2526 | (XFS_IFORK_SIZE(ip, whichfork) + | 2527 | XFS_IFORK_SIZE(ip, whichfork)); |
2527 | XFS_BROOT_SIZE_ADJ(ip))); | ||
2528 | xfs_bmbt_to_bmdr(mp, ifp->if_broot, ifp->if_broot_bytes, | 2528 | xfs_bmbt_to_bmdr(mp, ifp->if_broot, ifp->if_broot_bytes, |
2529 | (xfs_bmdr_block_t *)cp, | 2529 | (xfs_bmdr_block_t *)cp, |
2530 | XFS_DFORK_SIZE(dip, mp, whichfork)); | 2530 | XFS_DFORK_SIZE(dip, mp, whichfork)); |