* [f2fs-dev] [PATCH] f2fs: forcibly migrate to secure space for zoned device file pinning
@ 2024-09-11 21:30 ` Daeho Jeong
0 siblings, 0 replies; 6+ messages in thread
From: Daeho Jeong @ 2024-09-11 21:30 UTC (permalink / raw)
To: linux-kernel, linux-f2fs-devel, kernel-team; +Cc: Daeho Jeong
From: Daeho Jeong <daehojeong@google.com>
We need to migrate data blocks even though it is full to secure space
for zoned device file pinning.
Signed-off-by: Daeho Jeong <daehojeong@google.com>
Fixes: 9703d69d9d15 ("f2fs: support file pinning for zoned devices")
---
fs/f2fs/gc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c
index 724bbcb447d3..aaae13493a70 100644
--- a/fs/f2fs/gc.c
+++ b/fs/f2fs/gc.c
@@ -2010,8 +2010,7 @@ int f2fs_gc_range(struct f2fs_sb_info *sbi,
.iroot = RADIX_TREE_INIT(gc_list.iroot, GFP_NOFS),
};
- do_garbage_collect(sbi, segno, &gc_list, FG_GC,
- dry_run_sections == 0);
+ do_garbage_collect(sbi, segno, &gc_list, FG_GC, true);
put_gc_inode(&gc_list);
if (!dry_run && get_valid_blocks(sbi, segno, true))
--
2.46.0.598.g6f2099f65c-goog
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] f2fs: forcibly migrate to secure space for zoned device file pinning @ 2024-09-11 21:30 ` Daeho Jeong 0 siblings, 0 replies; 6+ messages in thread From: Daeho Jeong @ 2024-09-11 21:30 UTC (permalink / raw) To: linux-kernel, linux-f2fs-devel, kernel-team; +Cc: Daeho Jeong From: Daeho Jeong <daehojeong@google.com> We need to migrate data blocks even though it is full to secure space for zoned device file pinning. Signed-off-by: Daeho Jeong <daehojeong@google.com> Fixes: 9703d69d9d15 ("f2fs: support file pinning for zoned devices") --- fs/f2fs/gc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index 724bbcb447d3..aaae13493a70 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -2010,8 +2010,7 @@ int f2fs_gc_range(struct f2fs_sb_info *sbi, .iroot = RADIX_TREE_INIT(gc_list.iroot, GFP_NOFS), }; - do_garbage_collect(sbi, segno, &gc_list, FG_GC, - dry_run_sections == 0); + do_garbage_collect(sbi, segno, &gc_list, FG_GC, true); put_gc_inode(&gc_list); if (!dry_run && get_valid_blocks(sbi, segno, true)) -- 2.46.0.598.g6f2099f65c-goog ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [f2fs-dev] [PATCH] f2fs: forcibly migrate to secure space for zoned device file pinning 2024-09-11 21:30 ` Daeho Jeong @ 2024-09-12 14:45 ` Jaegeuk Kim -1 siblings, 0 replies; 6+ messages in thread From: Jaegeuk Kim via Linux-f2fs-devel @ 2024-09-12 14:45 UTC (permalink / raw) To: Daeho Jeong; +Cc: Daeho Jeong, kernel-team, linux-kernel, linux-f2fs-devel On 09/11, Daeho Jeong wrote: > From: Daeho Jeong <daehojeong@google.com> > > We need to migrate data blocks even though it is full to secure space > for zoned device file pinning. > > Signed-off-by: Daeho Jeong <daehojeong@google.com> > Fixes: 9703d69d9d15 ("f2fs: support file pinning for zoned devices") > --- > fs/f2fs/gc.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > index 724bbcb447d3..aaae13493a70 100644 > --- a/fs/f2fs/gc.c > +++ b/fs/f2fs/gc.c > @@ -2010,8 +2010,7 @@ int f2fs_gc_range(struct f2fs_sb_info *sbi, > .iroot = RADIX_TREE_INIT(gc_list.iroot, GFP_NOFS), > }; > > - do_garbage_collect(sbi, segno, &gc_list, FG_GC, > - dry_run_sections == 0); > + do_garbage_collect(sbi, segno, &gc_list, FG_GC, true); Do you mean this, right? do_garbage_collect(sbi, segno, &gc_list, FG_GC, true, false); > put_gc_inode(&gc_list); > > if (!dry_run && get_valid_blocks(sbi, segno, true)) > -- > 2.46.0.598.g6f2099f65c-goog > _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] f2fs: forcibly migrate to secure space for zoned device file pinning @ 2024-09-12 14:45 ` Jaegeuk Kim 0 siblings, 0 replies; 6+ messages in thread From: Jaegeuk Kim @ 2024-09-12 14:45 UTC (permalink / raw) To: Daeho Jeong; +Cc: linux-kernel, linux-f2fs-devel, kernel-team, Daeho Jeong On 09/11, Daeho Jeong wrote: > From: Daeho Jeong <daehojeong@google.com> > > We need to migrate data blocks even though it is full to secure space > for zoned device file pinning. > > Signed-off-by: Daeho Jeong <daehojeong@google.com> > Fixes: 9703d69d9d15 ("f2fs: support file pinning for zoned devices") > --- > fs/f2fs/gc.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > index 724bbcb447d3..aaae13493a70 100644 > --- a/fs/f2fs/gc.c > +++ b/fs/f2fs/gc.c > @@ -2010,8 +2010,7 @@ int f2fs_gc_range(struct f2fs_sb_info *sbi, > .iroot = RADIX_TREE_INIT(gc_list.iroot, GFP_NOFS), > }; > > - do_garbage_collect(sbi, segno, &gc_list, FG_GC, > - dry_run_sections == 0); > + do_garbage_collect(sbi, segno, &gc_list, FG_GC, true); Do you mean this, right? do_garbage_collect(sbi, segno, &gc_list, FG_GC, true, false); > put_gc_inode(&gc_list); > > if (!dry_run && get_valid_blocks(sbi, segno, true)) > -- > 2.46.0.598.g6f2099f65c-goog > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [f2fs-dev] [PATCH] f2fs: forcibly migrate to secure space for zoned device file pinning 2024-09-12 14:45 ` Jaegeuk Kim @ 2024-09-12 15:53 ` Daeho Jeong -1 siblings, 0 replies; 6+ messages in thread From: Daeho Jeong @ 2024-09-12 15:53 UTC (permalink / raw) To: Jaegeuk Kim; +Cc: Daeho Jeong, kernel-team, linux-kernel, linux-f2fs-devel On Thu, Sep 12, 2024 at 7:45 AM Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > On 09/11, Daeho Jeong wrote: > > From: Daeho Jeong <daehojeong@google.com> > > > > We need to migrate data blocks even though it is full to secure space > > for zoned device file pinning. > > > > Signed-off-by: Daeho Jeong <daehojeong@google.com> > > Fixes: 9703d69d9d15 ("f2fs: support file pinning for zoned devices") > > --- > > fs/f2fs/gc.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > > index 724bbcb447d3..aaae13493a70 100644 > > --- a/fs/f2fs/gc.c > > +++ b/fs/f2fs/gc.c > > @@ -2010,8 +2010,7 @@ int f2fs_gc_range(struct f2fs_sb_info *sbi, > > .iroot = RADIX_TREE_INIT(gc_list.iroot, GFP_NOFS), > > }; > > > > - do_garbage_collect(sbi, segno, &gc_list, FG_GC, > > - dry_run_sections == 0); > > + do_garbage_collect(sbi, segno, &gc_list, FG_GC, true); > > Do you mean this, right? > > do_garbage_collect(sbi, segno, &gc_list, FG_GC, true, false); Oh, after ZUFS GC patches, we have six parameters on that. You're right. static int do_garbage_collect(struct f2fs_sb_info *sbi, unsigned int start_segno, struct gc_inode_list *gc_list, int gc_type, bool force_migrate) > > > put_gc_inode(&gc_list); > > > > if (!dry_run && get_valid_blocks(sbi, segno, true)) > > -- > > 2.46.0.598.g6f2099f65c-goog > > _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] f2fs: forcibly migrate to secure space for zoned device file pinning @ 2024-09-12 15:53 ` Daeho Jeong 0 siblings, 0 replies; 6+ messages in thread From: Daeho Jeong @ 2024-09-12 15:53 UTC (permalink / raw) To: Jaegeuk Kim; +Cc: linux-kernel, linux-f2fs-devel, kernel-team, Daeho Jeong On Thu, Sep 12, 2024 at 7:45 AM Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > On 09/11, Daeho Jeong wrote: > > From: Daeho Jeong <daehojeong@google.com> > > > > We need to migrate data blocks even though it is full to secure space > > for zoned device file pinning. > > > > Signed-off-by: Daeho Jeong <daehojeong@google.com> > > Fixes: 9703d69d9d15 ("f2fs: support file pinning for zoned devices") > > --- > > fs/f2fs/gc.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > > index 724bbcb447d3..aaae13493a70 100644 > > --- a/fs/f2fs/gc.c > > +++ b/fs/f2fs/gc.c > > @@ -2010,8 +2010,7 @@ int f2fs_gc_range(struct f2fs_sb_info *sbi, > > .iroot = RADIX_TREE_INIT(gc_list.iroot, GFP_NOFS), > > }; > > > > - do_garbage_collect(sbi, segno, &gc_list, FG_GC, > > - dry_run_sections == 0); > > + do_garbage_collect(sbi, segno, &gc_list, FG_GC, true); > > Do you mean this, right? > > do_garbage_collect(sbi, segno, &gc_list, FG_GC, true, false); Oh, after ZUFS GC patches, we have six parameters on that. You're right. static int do_garbage_collect(struct f2fs_sb_info *sbi, unsigned int start_segno, struct gc_inode_list *gc_list, int gc_type, bool force_migrate) > > > put_gc_inode(&gc_list); > > > > if (!dry_run && get_valid_blocks(sbi, segno, true)) > > -- > > 2.46.0.598.g6f2099f65c-goog > > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-09-12 15:53 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-09-11 21:30 [f2fs-dev] [PATCH] f2fs: forcibly migrate to secure space for zoned device file pinning Daeho Jeong 2024-09-11 21:30 ` Daeho Jeong 2024-09-12 14:45 ` [f2fs-dev] " Jaegeuk Kim via Linux-f2fs-devel 2024-09-12 14:45 ` Jaegeuk Kim 2024-09-12 15:53 ` [f2fs-dev] " Daeho Jeong 2024-09-12 15:53 ` Daeho Jeong
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.