public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] f2fs: set highest IO priority for checkpoint thread
@ 2025-03-03 22:17 Jaegeuk Kim
  2025-03-04 18:23 ` [f2fs-dev] " Daeho Jeong
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Jaegeuk Kim @ 2025-03-03 22:17 UTC (permalink / raw)
  To: linux-kernel, linux-f2fs-devel; +Cc: Jaegeuk Kim

The checkpoint is the top priority thread which can stop all the filesystem
operations. Let's make it RT priority.

Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
---
 fs/f2fs/checkpoint.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index bd890738b94d..8dbb815a35c0 100644
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
@@ -21,7 +21,7 @@
 #include "iostat.h"
 #include <trace/events/f2fs.h>
 
-#define DEFAULT_CHECKPOINT_IOPRIO (IOPRIO_PRIO_VALUE(IOPRIO_CLASS_BE, 3))
+#define DEFAULT_CHECKPOINT_IOPRIO (IOPRIO_PRIO_VALUE(IOPRIO_CLASS_RT, 3))
 
 static struct kmem_cache *ino_entry_slab;
 struct kmem_cache *f2fs_inode_entry_slab;
-- 
2.48.1.711.g2feabab25a-goog


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [f2fs-dev] [PATCH] f2fs: set highest IO priority for checkpoint thread
  2025-03-03 22:17 [PATCH] f2fs: set highest IO priority for checkpoint thread Jaegeuk Kim
@ 2025-03-04 18:23 ` Daeho Jeong
  2025-03-05  1:28 ` Chao Yu
  2025-03-05 14:50 ` patchwork-bot+f2fs
  2 siblings, 0 replies; 4+ messages in thread
From: Daeho Jeong @ 2025-03-04 18:23 UTC (permalink / raw)
  To: Jaegeuk Kim; +Cc: linux-kernel, linux-f2fs-devel

On Mon, Mar 3, 2025 at 2:19 PM Jaegeuk Kim via Linux-f2fs-devel
<linux-f2fs-devel@lists.sourceforge.net> wrote:
>
> The checkpoint is the top priority thread which can stop all the filesystem
> operations. Let's make it RT priority.
>
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
> ---
>  fs/f2fs/checkpoint.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
> index bd890738b94d..8dbb815a35c0 100644
> --- a/fs/f2fs/checkpoint.c
> +++ b/fs/f2fs/checkpoint.c
> @@ -21,7 +21,7 @@
>  #include "iostat.h"
>  #include <trace/events/f2fs.h>
>
> -#define DEFAULT_CHECKPOINT_IOPRIO (IOPRIO_PRIO_VALUE(IOPRIO_CLASS_BE, 3))
> +#define DEFAULT_CHECKPOINT_IOPRIO (IOPRIO_PRIO_VALUE(IOPRIO_CLASS_RT, 3))
>
>  static struct kmem_cache *ino_entry_slab;
>  struct kmem_cache *f2fs_inode_entry_slab;
> --
> 2.48.1.711.g2feabab25a-goog
>

Reviewed-by: Daeho Jeong <daehojeong@google.com>

>
>
> _______________________________________________
> 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] 4+ messages in thread

* Re: [f2fs-dev] [PATCH] f2fs: set highest IO priority for checkpoint thread
  2025-03-03 22:17 [PATCH] f2fs: set highest IO priority for checkpoint thread Jaegeuk Kim
  2025-03-04 18:23 ` [f2fs-dev] " Daeho Jeong
@ 2025-03-05  1:28 ` Chao Yu
  2025-03-05 14:50 ` patchwork-bot+f2fs
  2 siblings, 0 replies; 4+ messages in thread
From: Chao Yu @ 2025-03-05  1:28 UTC (permalink / raw)
  To: Jaegeuk Kim, linux-kernel, linux-f2fs-devel; +Cc: chao

On 3/4/25 06:17, Jaegeuk Kim via Linux-f2fs-devel wrote:
> The checkpoint is the top priority thread which can stop all the filesystem
> operations. Let's make it RT priority.
> 
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

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

Thanks,

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [f2fs-dev] [PATCH] f2fs: set highest IO priority for checkpoint thread
  2025-03-03 22:17 [PATCH] f2fs: set highest IO priority for checkpoint thread Jaegeuk Kim
  2025-03-04 18:23 ` [f2fs-dev] " Daeho Jeong
  2025-03-05  1:28 ` Chao Yu
@ 2025-03-05 14:50 ` patchwork-bot+f2fs
  2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+f2fs @ 2025-03-05 14:50 UTC (permalink / raw)
  To: Jaegeuk Kim; +Cc: linux-kernel, linux-f2fs-devel

Hello:

This patch was applied to jaegeuk/f2fs.git (dev)
by Jaegeuk Kim <jaegeuk@kernel.org>:

On Mon,  3 Mar 2025 22:17:30 +0000 you wrote:
> The checkpoint is the top priority thread which can stop all the filesystem
> operations. Let's make it RT priority.
> 
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
> ---
>  fs/f2fs/checkpoint.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Here is the summary with links:
  - [f2fs-dev] f2fs: set highest IO priority for checkpoint thread
    https://git.kernel.org/jaegeuk/f2fs/c/8a2d9f00d502

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-03-05 14:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-03 22:17 [PATCH] f2fs: set highest IO priority for checkpoint thread Jaegeuk Kim
2025-03-04 18:23 ` [f2fs-dev] " Daeho Jeong
2025-03-05  1:28 ` Chao Yu
2025-03-05 14:50 ` patchwork-bot+f2fs

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox