diff options
| author | Nghia Le <nghialm78@gmail.com> | 2021-11-03 02:05:21 +0700 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2021-11-02 14:29:12 -0600 |
| commit | 83956c86fffe0465408c7d62e925d88748075e00 (patch) | |
| tree | f0c2a915ce40db036646649661fce2eacdeb5362 /fs/io_uring.c | |
| parent | 71c9ce27bb57c59d8d7f5298e730c8096eef3d1f (diff) | |
io_uring: remove redundant assignment to ret in io_register_iowq_max_workers()
After the assignment, only exit path with label 'err' uses ret as
return value. However,before exiting through this path with label 'err',
ret is assigned with the return value of io_wq_max_workers(). Hence, the
initial assignment is redundant and can be removed.
Signed-off-by: Nghia Le <nghialm78@gmail.com>
Link: https://lore.kernel.org/r/20211102190521.28291-1-nghialm78@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/io_uring.c')
| -rw-r--r-- | fs/io_uring.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index 61c962c29235..1c8602f12c91 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -10794,7 +10794,6 @@ static __cold int io_register_iowq_max_workers(struct io_ring_ctx *ctx, memcpy(ctx->iowq_limits, new_count, sizeof(new_count)); ctx->iowq_limits_set = true; - ret = -EINVAL; if (tctx && tctx->io_wq) { ret = io_wq_max_workers(tctx->io_wq, new_count); if (ret) |