diff options
author | Mingming Cao <cmm@us.ibm.com> | 2008-10-09 12:50:59 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2008-10-09 12:50:59 -0400 |
commit | 1f7c14c62ce63805f9574664a6c6de3633d4a354 (patch) | |
tree | 68f9f5399bc64bf7fe87dae16d8cc287717208bf /fs/ext4 | |
parent | 3fa8749e584b55f1180411ab1b51117190bac1e5 (diff) |
percpu counter: clean up percpu_counter_sum_and_set()
percpu_counter_sum_and_set() and percpu_counter_sum() is the same except
the former updates the global counter after accounting. Since we are
taking the fbc->lock to calculate the precise value of the counter in
percpu_counter_sum() anyway, it should simply set fbc->count too, as the
percpu_counter_sum_and_set() does.
This patch merges these two interfaces into one.
Signed-off-by: Mingming Cao <cmm@us.ibm.com>
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: <linux-ext4@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
-rw-r--r-- | fs/ext4/balloc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext4/balloc.c b/fs/ext4/balloc.c index e9fa960ba6da..00a94d5866c2 100644 --- a/fs/ext4/balloc.c +++ b/fs/ext4/balloc.c | |||
@@ -1624,7 +1624,7 @@ ext4_fsblk_t ext4_has_free_blocks(struct ext4_sb_info *sbi, | |||
1624 | #ifdef CONFIG_SMP | 1624 | #ifdef CONFIG_SMP |
1625 | if (free_blocks - root_blocks < FBC_BATCH) | 1625 | if (free_blocks - root_blocks < FBC_BATCH) |
1626 | free_blocks = | 1626 | free_blocks = |
1627 | percpu_counter_sum_and_set(&sbi->s_freeblocks_counter); | 1627 | percpu_counter_sum(&sbi->s_freeblocks_counter); |
1628 | #endif | 1628 | #endif |
1629 | if (free_blocks <= root_blocks) | 1629 | if (free_blocks <= root_blocks) |
1630 | /* we don't have free space */ | 1630 | /* we don't have free space */ |