diff options
| author | Jens Axboe <axboe@kernel.dk> | 2020-01-26 19:53:12 +0300 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2020-01-26 19:56:05 +0300 |
| commit | 73e08e711d9c1d79fae01daed4b0e1fee5f8a275 (patch) | |
| tree | 2275ecb4663dd1fa186fe09e4ba69a8b9424a2d4 | |
| parent | 1292e972fff2b2d81e139e0c2fe5f50249e78c58 (diff) | |
| download | linux-73e08e711d9c1d79fae01daed4b0e1fee5f8a275.tar.xz | |
Revert "io_uring: only allow submit from owning task"
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>
| -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 5953d7f13690..e5b502091804 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -5161,12 +5161,6 @@ SYSCALL_DEFINE6(io_uring_enter, unsigned int, fd, u32, to_submit, } else if (to_submit) { struct mm_struct *cur_mm; - 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); /* already have mm, so io_submit_sqes() won't try to grab it */ |