Re: [f2fs-dev] [PATCH] f2fs: allow F2FS_IPU_NOCACHE for pinned file

[Daeho Jeong]

On Sat, Sep 14, 2024 at 8:24 PM Jaegeuk Kim via Linux-f2fs-devel
<linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
> This patch allows f2fs to submit bios of in-place writes on pinned file.
>
> Signed-off-by: Jaegeuk Kim <jaegeuk@xxxxxxxxxx>
> ---
>  fs/f2fs/sysfs.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
> index d18931b7b094..c56e8c873935 100644
> --- a/fs/f2fs/sysfs.c
> +++ b/fs/f2fs/sysfs.c
> @@ -792,7 +792,8 @@ static ssize_t __sbi_store(struct f2fs_attr *a,
>         if (!strcmp(a->attr.name, "ipu_policy")) {
>                 if (t >= BIT(F2FS_IPU_MAX))
>                         return -EINVAL;
> -               if (t && f2fs_lfs_mode(sbi))
> +               /* allow F2FS_IPU_NOCACHE only for IPU in the pinned file */
> +               if (f2fs_lfs_mode(sbi) && (t & ~BIT(F2FS_IPU_NOCACHE)))
>                         return -EINVAL;
>                 SM_I(sbi)->ipu_policy = (unsigned int)t;
>                 return count;
> --
> 2.46.0.662.g92d0881bb0-goog
>

Reviewed-by: Daeho Jeong <daehojeong@xxxxxxxxxx>

Thanks,


>
>
> _______________________________________________
> Linux-f2fs-devel mailing list
> Linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel