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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <9f381694-2573-4f72-9177-54e2b5c07652@xiaomi.com>
Date: Tue, 3 Sep 2024 09:50:41 +0000
From: Huang Jianan <huangjianan@...omi.com>
To: Wu Bo <wubo.oduw@...il.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
CC: Jaegeuk Kim <jaegeuk@...nel.org>, "linux-f2fs-devel@...ts.sourceforge.net"
	<linux-f2fs-devel@...ts.sourceforge.net>
Subject: Re: [External Mail][f2fs-dev] [PATCH 05/13] f2fs: set inline tail
 flag when create inode

On 2024/9/3 16:54, Wu Bo via Linux-f2fs-devel wrote:
> [外部邮件] 此邮件来源于小米公司外部，请谨慎处理。若对邮件安全性存疑，请将邮件转发给misec@...omi.com进行反馈
> 
> 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@...o.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@...ts.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel