On 07/21, Yufen Yu wrote:
> If f2fs_iget cannot search inode from inode cache, it will read
> the inode from disk. The processing of read from disk and init inode
> should not dirty the inode, as the commit 530e07042002 ("f2fs:
> don't mark compressed inode dirty during f2fs_iget()").

f2fs_truncate can make this?

552         if (file_should_truncate(inode) &&
553                         !is_sbi_flag_set(sbi, SBI_POR_DOING)) {
554                 ret = f2fs_truncate(inode);
555                 if (ret)
556                         goto bad_inode;


> 
> Thus, we can remove the unnecessary f2fs_inode_synced().
> 
> Signed-off-by: Yufen Yu <[email protected]>
> ---
>  fs/f2fs/inode.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c
> index fc55f5bd1fcc..ce87b66c6dfe 100644
> --- a/fs/f2fs/inode.c
> +++ b/fs/f2fs/inode.c
> @@ -562,7 +562,6 @@ struct inode *f2fs_iget(struct super_block *sb, unsigned 
> long ino)
>       return inode;
>  
>  bad_inode:
> -     f2fs_inode_synced(inode);
>       iget_failed(inode);
>       trace_f2fs_iget_exit(inode, ret);
>       return ERR_PTR(ret);
> -- 
> 2.31.1


_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to