diff options
| author |   Jens Axboe <axboe@kernel.dk> | 2021-04-01 08:38:34 -0600 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2021-04-01 08:56:28 -0600 |
| commit | 4b982bd0f383db9132e892c0c5144117359a6289 (patch) | |
| tree | 9b069f16d20a21e408f6d3821372f0c7acdf9d3d /fs/io_uring.c | |
| parent | io_uring: drop sqd lock before handling signals for SQPOLL (diff) | |
| download | linux-dev-4b982bd0f383db9132e892c0c5144117359a6289.tar.xz linux-dev-4b982bd0f383db9132e892c0c5144117359a6289.zip | |
io_uring: don't mark S_ISBLK async work as unbounded
S_ISBLK is marked as unbounded work for async preparation, because it
doesn't match S_ISREG. That is incorrect, as any read/write to a block
device is also a bounded operation. Fix it up and ensure that S_ISBLK
isn't marked unbounded.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to '')
| -rw-r--r-- | fs/io_uring.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index 6d7a1b69712b..a16b7df934d1 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -1213,7 +1213,7 @@ static void io_prep_async_work(struct io_kiocb *req) if (req->flags & REQ_F_ISREG) { if (def->hash_reg_file || (ctx->flags & IORING_SETUP_IOPOLL)) io_wq_hash_work(&req->work, file_inode(req->file)); - } else { + } else if (!req->file || !S_ISBLK(file_inode(req->file)->i_mode)) { if (def->unbound_nonreg_file) req->work.flags |= IO_WQ_WORK_UNBOUND; } |