diff options
author | Eric Sandeen <sandeen@sandeen.net> | 2007-02-10 02:37:16 -0500 |
---|---|---|
committer | Tim Shimmin <tes@sgi.com> | 2007-02-10 02:37:16 -0500 |
commit | f7c99b6fc7b3791cd24e0763cd4967d744c164a3 (patch) | |
tree | 543581c6ebe8987ff83b6d8b3425e5c2723bbf3d /fs/xfs/xfs_attr.c | |
parent | 39058a0e12a8b2dcb8f9345ecad52dbcfc120ef7 (diff) |
[XFS] Remove unused argument to xfs_bmap_finish
The firstblock argument to xfs_bmap_finish is not used by that function.
Remove it and cleanup the code a bit.
Patch provided by Eric Sandeen.
SGI-PV: 960196
SGI-Modid: xfs-linux-melb:xfs-kern:28034a
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
Signed-off-by: David Chinner <dgc@sgi.com>
Signed-off-by: Tim Shimmin <tes@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_attr.c')
-rw-r--r-- | fs/xfs/xfs_attr.c | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/fs/xfs/xfs_attr.c b/fs/xfs/xfs_attr.c index a5cb0972bb8a..9d358ffce4e5 100644 --- a/fs/xfs/xfs_attr.c +++ b/fs/xfs/xfs_attr.c | |||
@@ -349,7 +349,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen, | |||
349 | error = xfs_attr_shortform_to_leaf(&args); | 349 | error = xfs_attr_shortform_to_leaf(&args); |
350 | if (!error) { | 350 | if (!error) { |
351 | error = xfs_bmap_finish(&args.trans, args.flist, | 351 | error = xfs_bmap_finish(&args.trans, args.flist, |
352 | *args.firstblock, &committed); | 352 | &committed); |
353 | } | 353 | } |
354 | if (error) { | 354 | if (error) { |
355 | ASSERT(committed); | 355 | ASSERT(committed); |
@@ -976,7 +976,7 @@ xfs_attr_leaf_addname(xfs_da_args_t *args) | |||
976 | error = xfs_attr_leaf_to_node(args); | 976 | error = xfs_attr_leaf_to_node(args); |
977 | if (!error) { | 977 | if (!error) { |
978 | error = xfs_bmap_finish(&args->trans, args->flist, | 978 | error = xfs_bmap_finish(&args->trans, args->flist, |
979 | *args->firstblock, &committed); | 979 | &committed); |
980 | } | 980 | } |
981 | if (error) { | 981 | if (error) { |
982 | ASSERT(committed); | 982 | ASSERT(committed); |
@@ -1077,7 +1077,6 @@ xfs_attr_leaf_addname(xfs_da_args_t *args) | |||
1077 | if (!error) { | 1077 | if (!error) { |
1078 | error = xfs_bmap_finish(&args->trans, | 1078 | error = xfs_bmap_finish(&args->trans, |
1079 | args->flist, | 1079 | args->flist, |
1080 | *args->firstblock, | ||
1081 | &committed); | 1080 | &committed); |
1082 | } | 1081 | } |
1083 | if (error) { | 1082 | if (error) { |
@@ -1155,7 +1154,7 @@ xfs_attr_leaf_removename(xfs_da_args_t *args) | |||
1155 | /* bp is gone due to xfs_da_shrink_inode */ | 1154 | /* bp is gone due to xfs_da_shrink_inode */ |
1156 | if (!error) { | 1155 | if (!error) { |
1157 | error = xfs_bmap_finish(&args->trans, args->flist, | 1156 | error = xfs_bmap_finish(&args->trans, args->flist, |
1158 | *args->firstblock, &committed); | 1157 | &committed); |
1159 | } | 1158 | } |
1160 | if (error) { | 1159 | if (error) { |
1161 | ASSERT(committed); | 1160 | ASSERT(committed); |
@@ -1310,7 +1309,6 @@ restart: | |||
1310 | if (!error) { | 1309 | if (!error) { |
1311 | error = xfs_bmap_finish(&args->trans, | 1310 | error = xfs_bmap_finish(&args->trans, |
1312 | args->flist, | 1311 | args->flist, |
1313 | *args->firstblock, | ||
1314 | &committed); | 1312 | &committed); |
1315 | } | 1313 | } |
1316 | if (error) { | 1314 | if (error) { |
@@ -1350,7 +1348,7 @@ restart: | |||
1350 | error = xfs_da_split(state); | 1348 | error = xfs_da_split(state); |
1351 | if (!error) { | 1349 | if (!error) { |
1352 | error = xfs_bmap_finish(&args->trans, args->flist, | 1350 | error = xfs_bmap_finish(&args->trans, args->flist, |
1353 | *args->firstblock, &committed); | 1351 | &committed); |
1354 | } | 1352 | } |
1355 | if (error) { | 1353 | if (error) { |
1356 | ASSERT(committed); | 1354 | ASSERT(committed); |
@@ -1462,7 +1460,6 @@ restart: | |||
1462 | if (!error) { | 1460 | if (!error) { |
1463 | error = xfs_bmap_finish(&args->trans, | 1461 | error = xfs_bmap_finish(&args->trans, |
1464 | args->flist, | 1462 | args->flist, |
1465 | *args->firstblock, | ||
1466 | &committed); | 1463 | &committed); |
1467 | } | 1464 | } |
1468 | if (error) { | 1465 | if (error) { |
@@ -1597,7 +1594,7 @@ xfs_attr_node_removename(xfs_da_args_t *args) | |||
1597 | error = xfs_da_join(state); | 1594 | error = xfs_da_join(state); |
1598 | if (!error) { | 1595 | if (!error) { |
1599 | error = xfs_bmap_finish(&args->trans, args->flist, | 1596 | error = xfs_bmap_finish(&args->trans, args->flist, |
1600 | *args->firstblock, &committed); | 1597 | &committed); |
1601 | } | 1598 | } |
1602 | if (error) { | 1599 | if (error) { |
1603 | ASSERT(committed); | 1600 | ASSERT(committed); |
@@ -1649,7 +1646,6 @@ xfs_attr_node_removename(xfs_da_args_t *args) | |||
1649 | if (!error) { | 1646 | if (!error) { |
1650 | error = xfs_bmap_finish(&args->trans, | 1647 | error = xfs_bmap_finish(&args->trans, |
1651 | args->flist, | 1648 | args->flist, |
1652 | *args->firstblock, | ||
1653 | &committed); | 1649 | &committed); |
1654 | } | 1650 | } |
1655 | if (error) { | 1651 | if (error) { |
@@ -2093,7 +2089,7 @@ xfs_attr_rmtval_set(xfs_da_args_t *args) | |||
2093 | args->flist, NULL); | 2089 | args->flist, NULL); |
2094 | if (!error) { | 2090 | if (!error) { |
2095 | error = xfs_bmap_finish(&args->trans, args->flist, | 2091 | error = xfs_bmap_finish(&args->trans, args->flist, |
2096 | *args->firstblock, &committed); | 2092 | &committed); |
2097 | } | 2093 | } |
2098 | if (error) { | 2094 | if (error) { |
2099 | ASSERT(committed); | 2095 | ASSERT(committed); |
@@ -2249,7 +2245,7 @@ xfs_attr_rmtval_remove(xfs_da_args_t *args) | |||
2249 | NULL, &done); | 2245 | NULL, &done); |
2250 | if (!error) { | 2246 | if (!error) { |
2251 | error = xfs_bmap_finish(&args->trans, args->flist, | 2247 | error = xfs_bmap_finish(&args->trans, args->flist, |
2252 | *args->firstblock, &committed); | 2248 | &committed); |
2253 | } | 2249 | } |
2254 | if (error) { | 2250 | if (error) { |
2255 | ASSERT(committed); | 2251 | ASSERT(committed); |