summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2010-10-20 22:32:02 +0200
committerJan Kara <jack@suse.cz>2011-01-06 17:03:56 +0100
commit7abc2e45e48ca04206949682402d5d55bc64a16b (patch)
tree9ef57aea07cb8c14b21f3849c8335b116967a42e /fs
parentudf: Remove BKL from udf_put_super() and udf_remount_fs() (diff)
downloadlinux-7abc2e45e48ca04206949682402d5d55bc64a16b.tar.xz
linux-7abc2e45e48ca04206949682402d5d55bc64a16b.zip
udf: Call udf_add_free_space() for more blocks at once in udf_free_blocks()
There's no need to call udf_add_free_space() for one block at a time. It saves us noticeable amount of work and yields different result from the original code only if the filesystem is corrupted and bitmap bit is already cleared. In such case counter of free blocks is probably wrong anyways so the change does not matter. Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs')
-rw-r--r--fs/udf/balloc.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/udf/balloc.c b/fs/udf/balloc.c
index b608efaa4cee..306ee39ef2c3 100644
--- a/fs/udf/balloc.c
+++ b/fs/udf/balloc.c
@@ -157,10 +157,9 @@ static void udf_bitmap_free_blocks(struct super_block *sb,
udf_debug("bit %ld already set\n", bit + i);
udf_debug("byte=%2x\n",
((char *)bh->b_data)[(bit + i) >> 3]);
- } else {
- udf_add_free_space(sb, sbi->s_partition, 1);
}
}
+ udf_add_free_space(sb, sbi->s_partition, count);
mark_buffer_dirty(bh);
if (overflow) {
block += count;