diff mbox series

[1/1] io_uring: reuse io_should_terminate_tw() for cmds

Message ID 8a88dd6e4ed8e6c00c6552af0c20c9de02e458de.1736955455.git.asml.silence@gmail.com (mailing list archive)
State New
Headers show
Series [1/1] io_uring: reuse io_should_terminate_tw() for cmds | expand

Commit Message

Pavel Begunkov Jan. 15, 2025, 3:40 p.m. UTC
io_uring_cmd_work() rolled a hard coded version of
io_should_terminate_tw() to avoid conflicts, but now it's time to
converge them.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
---
 io_uring/uring_cmd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jens Axboe Jan. 15, 2025, 3:54 p.m. UTC | #1
On Wed, 15 Jan 2025 15:40:48 +0000, Pavel Begunkov wrote:
> io_uring_cmd_work() rolled a hard coded version of
> io_should_terminate_tw() to avoid conflicts, but now it's time to
> converge them.
> 
> 

Applied, thanks!

[1/1] io_uring: reuse io_should_terminate_tw() for cmds
      (no commit info)

Best regards,
diff mbox series

Patch

diff --git a/io_uring/uring_cmd.c b/io_uring/uring_cmd.c
index d6ff803dbbe1..d235043db21e 100644
--- a/io_uring/uring_cmd.c
+++ b/io_uring/uring_cmd.c
@@ -105,7 +105,7 @@  static void io_uring_cmd_work(struct io_kiocb *req, struct io_tw_state *ts)
 	struct io_uring_cmd *ioucmd = io_kiocb_to_cmd(req, struct io_uring_cmd);
 	unsigned int flags = IO_URING_F_COMPLETE_DEFER;
 
-	if (current->flags & (PF_EXITING | PF_KTHREAD))
+	if (io_should_terminate_tw())
 		flags |= IO_URING_F_TASK_DEAD;
 
 	/* task_work executor checks the deffered list completion */