diff options
| author | Jan Kara <jack@suse.cz> | 2010-10-20 22:32:02 +0200 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2011-01-06 17:03:56 +0100 |
| commit | 7abc2e45e48ca04206949682402d5d55bc64a16b (patch) | |
| tree | 9ef57aea07cb8c14b21f3849c8335b116967a42e /fs | |
| parent | 0484b1cedc053cf88a046da5f08bc00747e533cb (diff) | |
| download | kernel_samsung_crespo-7abc2e45e48ca04206949682402d5d55bc64a16b.zip kernel_samsung_crespo-7abc2e45e48ca04206949682402d5d55bc64a16b.tar.gz kernel_samsung_crespo-7abc2e45e48ca04206949682402d5d55bc64a16b.tar.bz2 | |
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.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/udf/balloc.c b/fs/udf/balloc.c index b608efa..306ee39 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; |