[PATCH] f2fs: Don't try to split extents shorter than F2FS_MIN_EXTENT_LEN

[PATCH] f2fs: Don't try to split extents shorter than F2FS_MIN_EXTENT_LEN

[Fan Li]

Since only parts of extents longer than F2FS_MIN_EXTENT_LEN will 
be kept in extent cache after split, extents already shorter than 
F2FS_MIN_EXTENT_LEN don't need to try split at all.


Signed-off-by: Fan Li <fanofcode.li@samsung.com>
---
 fs/f2fs/data.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 37a2632..e2996cb 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -585,7 +585,7 @@ static bool f2fs_update_extent_tree(struct inode *inode,
pgoff_t fofs,
 	__detach_extent_node(sbi, et, en);
 
 	/* 2. if extent can be split more, split and insert the left part */
-	if (dei.len > 1) {
+	if (dei.len > F2FS_MIN_EXTENT_LEN) {
 		/*  insert left part of split extent into cache */
 		if (fofs - dei.fofs >= F2FS_MIN_EXTENT_LEN) {
 			set_extent_info(&ei, dei.fofs, dei.blk,
-- 
1.7.9.5


------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/

Re: [PATCH] f2fs: Don't try to split extents shorter than F2FS_MIN_EXTENT_LEN

[Chao Yu]

> -----Original Message-----
> From: Fan Li [mailto:fanofcode.li@samsung.com]
> Sent: Wednesday, July 08, 2015 4:03 PM
> To: jaegeuk@kernel.org
> Cc: linux-f2fs-devel@lists.sourceforge.net
> Subject: [f2fs-dev] [PATCH] f2fs: Don't try to split extents shorter than F2FS_MIN_EXTENT_LEN
> 
> Since only parts of extents longer than F2FS_MIN_EXTENT_LEN will
> be kept in extent cache after split, extents already shorter than
> F2FS_MIN_EXTENT_LEN don't need to try split at all.
> 
> 
> Signed-off-by: Fan Li <fanofcode.li@samsung.com>

Looks good to me.

Reviewed-by: Chao Yu <chao2.yu@samsung.com>


------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/