diff options
author | Christoph Hellwig <hch@infradead.org> | 2012-07-04 10:54:47 -0400 |
---|---|---|
committer | Ben Myers <bpm@sgi.com> | 2012-07-29 17:00:31 -0400 |
commit | 08358906ed78f6ab4d3ff8e4fd1b87b9a4aea645 (patch) | |
tree | 47884ad2cba2ede418c6f4716c778c9d6ec88e81 /fs/xfs/xfs_ialloc.c | |
parent | f2ecc5e453134a13c3b2b0f2cac52ab2d5c540d7 (diff) |
xfs: remove the alloc_done argument to xfs_dialloc
We can simplify check the IO_agbp pointer for being non-NULL instead of
passing another argument through two layers of function calls.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Mark Tinguely <tinguely@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_ialloc.c')
-rw-r--r-- | fs/xfs/xfs_ialloc.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/xfs/xfs_ialloc.c b/fs/xfs/xfs_ialloc.c index a124b9f88aae..2b70952c9d8c 100644 --- a/fs/xfs/xfs_ialloc.c +++ b/fs/xfs/xfs_ialloc.c | |||
@@ -895,7 +895,6 @@ xfs_dialloc( | |||
895 | umode_t mode, | 895 | umode_t mode, |
896 | int okalloc, | 896 | int okalloc, |
897 | struct xfs_buf **IO_agbp, | 897 | struct xfs_buf **IO_agbp, |
898 | boolean_t *alloc_done, | ||
899 | xfs_ino_t *inop) | 898 | xfs_ino_t *inop) |
900 | { | 899 | { |
901 | struct xfs_buf *agbp; | 900 | struct xfs_buf *agbp; |
@@ -955,7 +954,6 @@ xfs_dialloc( | |||
955 | * or in which we can allocate some inodes. Iterate through the | 954 | * or in which we can allocate some inodes. Iterate through the |
956 | * allocation groups upward, wrapping at the end. | 955 | * allocation groups upward, wrapping at the end. |
957 | */ | 956 | */ |
958 | *alloc_done = B_FALSE; | ||
959 | while (!agi->agi_freecount) { | 957 | while (!agi->agi_freecount) { |
960 | /* | 958 | /* |
961 | * Don't do anything if we're not supposed to allocate | 959 | * Don't do anything if we're not supposed to allocate |
@@ -982,7 +980,6 @@ xfs_dialloc( | |||
982 | * us again where we left off. | 980 | * us again where we left off. |
983 | */ | 981 | */ |
984 | ASSERT(be32_to_cpu(agi->agi_freecount) > 0); | 982 | ASSERT(be32_to_cpu(agi->agi_freecount) > 0); |
985 | *alloc_done = B_TRUE; | ||
986 | *IO_agbp = agbp; | 983 | *IO_agbp = agbp; |
987 | *inop = NULLFSINO; | 984 | *inop = NULLFSINO; |
988 | return 0; | 985 | return 0; |