Re: [External Mail][f2fs-dev] [PATCH 05/13] f2fs: set inline tail flag when create inode

[Huang Jianan <huangjianan@xiaomi.com>]

On 2024/9/3 16:54, Wu Bo via Linux-f2fs-devel wrote:
Hi Bo,
> 
> Set inline tail flag to enable this feature when new inode is created.
> Inherit the conditions from inline data.
> 
> Signed-off-by: Wu Bo <bo.wu@vivo.com>
> ---
>   fs/f2fs/namei.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c
> index 38b4750475db..13c295ea19de 100644
> --- a/fs/f2fs/namei.c
> +++ b/fs/f2fs/namei.c
> @@ -315,6 +315,9 @@ static struct inode *f2fs_new_inode(struct mnt_idmap *idmap,
>          /* Should enable inline_data after compression set */
>          if (test_opt(sbi, INLINE_DATA) && f2fs_may_inline_data(inode))
>                  set_inode_flag(inode, FI_INLINE_DATA);
> +       /* Inherit the conditions from inline data */
> +       if (test_opt(sbi, INLINE_TAIL) && f2fs_has_inline_data(inode))
> +               set_inode_flag(inode, FI_INLINE_TAIL);

Should f2fs_post_read_required() be checked here, like inline data?

Thanks,
Jianan

> 
>          if (name && !test_opt(sbi, DISABLE_EXT_IDENTIFY))
>                  set_file_temperature(sbi, inode, name);
> --
> 2.35.3
> 
> 
> 
> _______________________________________________
> Linux-f2fs-devel mailing list
> Linux-f2fs-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel