diff options
| author | Pavel Begunkov <asml.silence@gmail.com> | 2022-10-16 21:30:49 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2022-10-16 17:07:53 -0600 |
| commit | 4d5059512d283dab7372d282c2fbd43c7f5a2456 (patch) | |
| tree | 384732a62cb297aee6e4f9272350bb1ac777eee2 /io_uring/io_uring.c | |
| parent | 38eddb2c75fb99b9cd78445094ca0e1bda08d102 (diff) | |
| download | linux-4d5059512d283dab7372d282c2fbd43c7f5a2456.tar.gz linux-4d5059512d283dab7372d282c2fbd43c7f5a2456.tar.bz2 linux-4d5059512d283dab7372d282c2fbd43c7f5a2456.zip | |
io_uring: kill hot path fixed file bitmap debug checks
We test file_table.bitmap in io_file_get_fixed() to check invariants,
don't do it, it's expensive and was showing up in profiles. No reports of
this triggering has come in. Move the check to the file clear instead,
which will still catch any wrong usage.
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/cf77f2ded68d2e5b2bc7355784d969837d48e023.1665891182.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'io_uring/io_uring.c')
| -rw-r--r-- | io_uring/io_uring.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/io_uring/io_uring.c b/io_uring/io_uring.c index 18aa39709fae..6e50f548de1a 100644 --- a/io_uring/io_uring.c +++ b/io_uring/io_uring.c @@ -1858,7 +1858,6 @@ inline struct file *io_file_get_fixed(struct io_kiocb *req, int fd, /* mask in overlapping REQ_F and FFS bits */ req->flags |= (file_ptr << REQ_F_SUPPORT_NOWAIT_BIT); io_req_set_rsrc_node(req, ctx, 0); - WARN_ON_ONCE(file && !test_bit(fd, ctx->file_table.bitmap)); out: io_ring_submit_unlock(ctx, issue_flags); return file; |