diff options
author | Lachlan McIlroy <lachlan@sgi.com> | 2007-10-11 03:34:33 -0400 |
---|---|---|
committer | Lachlan McIlroy <lachlan@redback.melbourne.sgi.com> | 2008-02-07 00:44:14 -0500 |
commit | 541d7d3c4b31e2b0ac846fe6d2eb5cdbe1353095 (patch) | |
tree | d8c9cf9cf75fd3d23ebc19e5f6b646a4d807b72c /fs/xfs/xfs_vnodeops.c | |
parent | 21a62542b6d7f726d6c1d2cfbfa084f721ba4a26 (diff) |
[XFS] kill unnessecary ioops indirection
Currently there is an indirection called ioops in the XFS data I/O path.
Various functions are called by functions pointers, but there is no
coherence in what this is for, and of course for XFS itself it's entirely
unused. This patch removes it instead and significantly reduces source and
binary size of XFS while making maintaince easier.
SGI-PV: 970841
SGI-Modid: xfs-linux-melb:xfs-kern:29737a
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Signed-off-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: Tim Shimmin <tes@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_vnodeops.c')
-rw-r--r-- | fs/xfs/xfs_vnodeops.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fs/xfs/xfs_vnodeops.c b/fs/xfs/xfs_vnodeops.c index daba528bcc33..a5cd2dc4c80a 100644 --- a/fs/xfs/xfs_vnodeops.c +++ b/fs/xfs/xfs_vnodeops.c | |||
@@ -1187,7 +1187,7 @@ xfs_free_eofblocks( | |||
1187 | 1187 | ||
1188 | nimaps = 1; | 1188 | nimaps = 1; |
1189 | xfs_ilock(ip, XFS_ILOCK_SHARED); | 1189 | xfs_ilock(ip, XFS_ILOCK_SHARED); |
1190 | error = XFS_BMAPI(mp, NULL, &ip->i_iocore, end_fsb, map_len, 0, | 1190 | error = xfs_bmapi(NULL, ip, end_fsb, map_len, 0, |
1191 | NULL, 0, &imap, &nimaps, NULL, NULL); | 1191 | NULL, 0, &imap, &nimaps, NULL, NULL); |
1192 | xfs_iunlock(ip, XFS_ILOCK_SHARED); | 1192 | xfs_iunlock(ip, XFS_ILOCK_SHARED); |
1193 | 1193 | ||
@@ -3972,7 +3972,7 @@ retry: | |||
3972 | * Issue the xfs_bmapi() call to allocate the blocks | 3972 | * Issue the xfs_bmapi() call to allocate the blocks |
3973 | */ | 3973 | */ |
3974 | XFS_BMAP_INIT(&free_list, &firstfsb); | 3974 | XFS_BMAP_INIT(&free_list, &firstfsb); |
3975 | error = XFS_BMAPI(mp, tp, &ip->i_iocore, startoffset_fsb, | 3975 | error = xfs_bmapi(tp, ip, startoffset_fsb, |
3976 | allocatesize_fsb, bmapi_flag, | 3976 | allocatesize_fsb, bmapi_flag, |
3977 | &firstfsb, 0, imapp, &nimaps, | 3977 | &firstfsb, 0, imapp, &nimaps, |
3978 | &free_list, NULL); | 3978 | &free_list, NULL); |
@@ -4054,7 +4054,7 @@ xfs_zero_remaining_bytes( | |||
4054 | for (offset = startoff; offset <= endoff; offset = lastoffset + 1) { | 4054 | for (offset = startoff; offset <= endoff; offset = lastoffset + 1) { |
4055 | offset_fsb = XFS_B_TO_FSBT(mp, offset); | 4055 | offset_fsb = XFS_B_TO_FSBT(mp, offset); |
4056 | nimap = 1; | 4056 | nimap = 1; |
4057 | error = XFS_BMAPI(mp, NULL, &ip->i_iocore, offset_fsb, 1, 0, | 4057 | error = xfs_bmapi(NULL, ip, offset_fsb, 1, 0, |
4058 | NULL, 0, &imap, &nimap, NULL, NULL); | 4058 | NULL, 0, &imap, &nimap, NULL, NULL); |
4059 | if (error || nimap < 1) | 4059 | if (error || nimap < 1) |
4060 | break; | 4060 | break; |
@@ -4189,7 +4189,7 @@ xfs_free_file_space( | |||
4189 | */ | 4189 | */ |
4190 | if (rt && !XFS_SB_VERSION_HASEXTFLGBIT(&mp->m_sb)) { | 4190 | if (rt && !XFS_SB_VERSION_HASEXTFLGBIT(&mp->m_sb)) { |
4191 | nimap = 1; | 4191 | nimap = 1; |
4192 | error = XFS_BMAPI(mp, NULL, &ip->i_iocore, startoffset_fsb, | 4192 | error = xfs_bmapi(NULL, ip, startoffset_fsb, |
4193 | 1, 0, NULL, 0, &imap, &nimap, NULL, NULL); | 4193 | 1, 0, NULL, 0, &imap, &nimap, NULL, NULL); |
4194 | if (error) | 4194 | if (error) |
4195 | goto out_unlock_iolock; | 4195 | goto out_unlock_iolock; |
@@ -4204,7 +4204,7 @@ xfs_free_file_space( | |||
4204 | startoffset_fsb += mp->m_sb.sb_rextsize - mod; | 4204 | startoffset_fsb += mp->m_sb.sb_rextsize - mod; |
4205 | } | 4205 | } |
4206 | nimap = 1; | 4206 | nimap = 1; |
4207 | error = XFS_BMAPI(mp, NULL, &ip->i_iocore, endoffset_fsb - 1, | 4207 | error = xfs_bmapi(NULL, ip, endoffset_fsb - 1, |
4208 | 1, 0, NULL, 0, &imap, &nimap, NULL, NULL); | 4208 | 1, 0, NULL, 0, &imap, &nimap, NULL, NULL); |
4209 | if (error) | 4209 | if (error) |
4210 | goto out_unlock_iolock; | 4210 | goto out_unlock_iolock; |
@@ -4280,7 +4280,7 @@ xfs_free_file_space( | |||
4280 | * issue the bunmapi() call to free the blocks | 4280 | * issue the bunmapi() call to free the blocks |
4281 | */ | 4281 | */ |
4282 | XFS_BMAP_INIT(&free_list, &firstfsb); | 4282 | XFS_BMAP_INIT(&free_list, &firstfsb); |
4283 | error = XFS_BUNMAPI(mp, tp, &ip->i_iocore, startoffset_fsb, | 4283 | error = xfs_bunmapi(tp, ip, startoffset_fsb, |
4284 | endoffset_fsb - startoffset_fsb, | 4284 | endoffset_fsb - startoffset_fsb, |
4285 | 0, 2, &firstfsb, &free_list, NULL, &done); | 4285 | 0, 2, &firstfsb, &free_list, NULL, &done); |
4286 | if (error) { | 4286 | if (error) { |