f2fs: allow F2FS_IPU_NOCACHE for pinned file

[PATCH] f2fs: allow F2FS_IPU_NOCACHE for pinned file

Posted by Jaegeuk Kim 2 months, 2 weeks ago

This patch allows f2fs to submit bios of in-place writes on pinned file.

Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
---
 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

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

Posted by Chao Yu 2 months, 1 week ago

On 2024/9/15 11:22, Jaegeuk Kim via Linux-f2fs-devel wrote:
> This patch allows f2fs to submit bios of in-place writes on pinned file.
> 
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

Reviewed-by: Chao Yu <chao@kernel.org>

Thanks,

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

Posted by Daeho Jeong 2 months, 1 week ago

On Sat, Sep 14, 2024 at 8:24 PM Jaegeuk Kim via Linux-f2fs-devel
<linux-f2fs-devel@lists.sourceforge.net> wrote:
>
> This patch allows f2fs to submit bios of in-place writes on pinned file.
>
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
> ---
>  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@google.com>

Thanks,


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