diff options
author | Jens Axboe <axboe@kernel.dk> | 2020-01-26 09:53:12 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-01-29 16:45:24 +0100 |
commit | b29d143a690a660b27e7e84c01dd541d38fe9372 (patch) | |
tree | 30779ce0498bc32f0bd5e45de0120ad88cf2ba78 | |
parent | 29ce06b6b6d45ce461ddb723502a0827b8d8c9fb (diff) | |
download | linux-stable-b29d143a690a660b27e7e84c01dd541d38fe9372.tar.gz linux-stable-b29d143a690a660b27e7e84c01dd541d38fe9372.tar.bz2 linux-stable-b29d143a690a660b27e7e84c01dd541d38fe9372.zip |
Revert "io_uring: only allow submit from owning task"
commit 73e08e711d9c1d79fae01daed4b0e1fee5f8a275 upstream.
This ends up being too restrictive for tasks that willingly fork and
share the ring between forks. Andres reports that this breaks his
postgresql work. Since we're close to 5.5 release, revert this change
for now.
Cc: stable@vger.kernel.org
Fixes: 44d282796f81 ("io_uring: only allow submit from owning task")
Reported-by: Andres Freund <andres@anarazel.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | fs/io_uring.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index b1c9ad1fb9e1..709671faaed6 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -3716,12 +3716,6 @@ SYSCALL_DEFINE6(io_uring_enter, unsigned int, fd, u32, to_submit, wake_up(&ctx->sqo_wait); submitted = to_submit; } else if (to_submit) { - if (current->mm != ctx->sqo_mm || - current_cred() != ctx->creds) { - ret = -EPERM; - goto out; - } - to_submit = min(to_submit, ctx->sq_entries); mutex_lock(&ctx->uring_lock); |