diff options
| author | Jens Axboe <axboe@kernel.dk> | 2025-08-20 20:03:30 -0600 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2025-08-24 11:41:11 -0600 |
| commit | 5e73b402cbbea51bcab90fc5ee6c6d06af76ae1b (patch) | |
| tree | 0d7009b27e61cd99fdbdf9bbfd609f142b5bd294 /io_uring/io_uring.c | |
| parent | ab3ea6eac5f45669b091309f592c4ea324003053 (diff) | |
| download | linux-5e73b402cbbea51bcab90fc5ee6c6d06af76ae1b.tar.gz linux-5e73b402cbbea51bcab90fc5ee6c6d06af76ae1b.tar.bz2 linux-5e73b402cbbea51bcab90fc5ee6c6d06af76ae1b.zip | |
io_uring/kbuf: drop 'issue_flags' from io_put_kbuf(s)() arguments
Picking multiple buffers always requires the ring lock to be held across
the operation, so there's no need to pass in the issue_flags to
io_put_kbufs(). On the single buffer side, if the initial picking of a
ring buffer was unlocked, then it will have been committed already. For
legacy buffers, no locking is required, as they will simply be freed.
Link: https://lore.kernel.org/r/20250821020750.598432-3-axboe@kernel.dk
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'io_uring/io_uring.c')
| -rw-r--r-- | io_uring/io_uring.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/io_uring/io_uring.c b/io_uring/io_uring.c index e511949086dd..ee484a0e2c4f 100644 --- a/io_uring/io_uring.c +++ b/io_uring/io_uring.c @@ -1007,7 +1007,7 @@ void io_req_defer_failed(struct io_kiocb *req, s32 res) lockdep_assert_held(&req->ctx->uring_lock); req_set_fail(req); - io_req_set_res(req, res, io_put_kbuf(req, res, IO_URING_F_UNLOCKED)); + io_req_set_res(req, res, io_put_kbuf(req, res)); if (def->fail) def->fail(req); io_req_complete_defer(req); |