diff options
| author | Pavel Begunkov <asml.silence@gmail.com> | 2020-11-07 13:20:39 +0000 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2020-12-09 12:04:01 -0700 |
| commit | 10cad2c40dcb04bb46b2bf399e00ca5ea93d36b0 (patch) | |
| tree | 91a596bc9b846b409410afafac0a3cbfbc949538 /fs/io_uring.c | |
| parent | 2e9dbe902d1020ef70f968e8675c8d2457c4ffaa (diff) | |
| download | lwn-10cad2c40dcb04bb46b2bf399e00ca5ea93d36b0.tar.gz lwn-10cad2c40dcb04bb46b2bf399e00ca5ea93d36b0.zip | |
io_uring: don't take fs for recvmsg/sendmsg
We don't even allow not plain data msg_control, which is disallowed in
__sys_{send,revb}msg_sock(). So no need in fs for IORING_OP_SENDMSG and
IORING_OP_RECVMSG. fs->lock is less contanged not as much as before, but
there are cases that can be, e.g. IOSQE_ASYNC.
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/io_uring.c')
| -rw-r--r-- | fs/io_uring.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index 3617bde95de2..33c448776731 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -852,8 +852,7 @@ static const struct io_op_def io_op_defs[] = { .pollout = 1, .needs_async_data = 1, .async_size = sizeof(struct io_async_msghdr), - .work_flags = IO_WQ_WORK_MM | IO_WQ_WORK_BLKCG | - IO_WQ_WORK_FS, + .work_flags = IO_WQ_WORK_MM | IO_WQ_WORK_BLKCG, }, [IORING_OP_RECVMSG] = { .needs_file = 1, @@ -862,8 +861,7 @@ static const struct io_op_def io_op_defs[] = { .buffer_select = 1, .needs_async_data = 1, .async_size = sizeof(struct io_async_msghdr), - .work_flags = IO_WQ_WORK_MM | IO_WQ_WORK_BLKCG | - IO_WQ_WORK_FS, + .work_flags = IO_WQ_WORK_MM | IO_WQ_WORK_BLKCG, }, [IORING_OP_TIMEOUT] = { .needs_async_data = 1, |