summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNamjae Jeon <namjae.jeon@samsung.com>2013-06-08 21:25:40 +0900
committerChanho Park <chanho61.park@samsung.com>2014-03-20 17:34:39 +0900
commit0b8880809b85c183811f2936fddc09c299d9067d (patch)
tree6e3c7c684c6d7d40118489b784ef6d59524d636d
parentfb252a3936509c5a7f9042088d92445eb7998527 (diff)
downloadlinux-3.10-0b8880809b85c183811f2936fddc09c299d9067d.tar.gz
linux-3.10-0b8880809b85c183811f2936fddc09c299d9067d.tar.bz2
linux-3.10-0b8880809b85c183811f2936fddc09c299d9067d.zip
f2fs: optimise the truncate_data_blocks_range() range
The function truncate_data_blocks_range() decrements the valid block count of inode via dec_valid_block_count(). Since this function updates the i_blocks field of inode, we can update this field once we have calculated total the number of blocks to be freed. Therefore we can decrement valid blocks outside of the for loop. if (nr_free) { + dec_valid_block_count(sbi, dn->inode, nr_free); set_page_dirty(dn->node_page); sync_inode_page(dn); } 'nr_free' tells the total number of blocks freed. So, we can just directly pass this value to dec_valid_block_count() and update the i_blocks. Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com> Signed-off-by: Pankaj Kumar <pankaj.km@samsung.com> Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
-rw-r--r--fs/f2fs/file.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
index 85b665d176a..2f649b84852 100644
--- a/fs/f2fs/file.c
+++ b/fs/f2fs/file.c
@@ -185,10 +185,10 @@ int truncate_data_blocks_range(struct dnode_of_data *dn, int count)
update_extent_cache(NULL_ADDR, dn);
invalidate_blocks(sbi, blkaddr);
- dec_valid_block_count(sbi, dn->inode, 1);
nr_free++;
}
if (nr_free) {
+ dec_valid_block_count(sbi, dn->inode, nr_free);
set_page_dirty(dn->node_page);
sync_inode_page(dn);
}