> Signed-off-by: Jaegeuk Kim <jaegeuk....@samsung.com>
Function looks good to me. But some nitpicking below for code cleanup..
> @@ -71,6 +71,9 @@ static int check_extent_cache(struct inode *inode, pgoff_t
> pgofs,
return value could be boolean?
> pgoff_t start_fofs, end_fofs;
> block_t start_blkaddr;
>
> + if (is_inode_flag_set(fi, FI_NO_EXTENT))
> + return 0;
> +
> read_lock(&fi->ext.ext_lock);
> if (fi->ext.len == 0) {
> read_unlock(&fi->ext.ext_lock);
> @@ -109,6 +112,7 @@ void update_extent_cache(block_t blk_addr, struct
> dnode_of_data *dn)
> struct f2fs_inode_info *fi = F2FS_I(dn->inode);
> pgoff_t fofs, start_fofs, end_fofs;
> block_t start_blkaddr, end_blkaddr;
> + int need_update = true;
bool?
------------------------------------------------------------------------------
Shape the Mobile Experience: Free Subscription
Software experts and developers: Be at the forefront of tech innovation.
Intel(R) Software Adrenaline delivers strategic insight and game-changing
conversations that shape the rapidly evolving mobile landscape. Sign up now.
http://pubads.g.doubleclick.net/gampad/clk?id=63431311&iu=/4140/ostg.clktrk
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
