diff options
| author | Lukas Bulwahn <lukas.bulwahn@gmail.com> | 2021-12-23 13:53:00 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2021-12-23 07:10:07 -0700 |
| commit | 669a064625fa3a06ddf8a4ac1f35b7436b99f133 (patch) | |
| tree | eaa0dae6d7942bf3efe8d7cde95b20ac2e52867b | |
| parent | a16c7246368db8935652c805bc446928d0e1c0aa (diff) | |
| download | linux-starfive-669a064625fa3a06ddf8a4ac1f35b7436b99f133.tar.gz linux-starfive-669a064625fa3a06ddf8a4ac1f35b7436b99f133.tar.bz2 linux-starfive-669a064625fa3a06ddf8a4ac1f35b7436b99f133.zip | |
block: drop needless assignment in set_task_ioprio()
Commit 5fc11eebb4a9 ("block: open code create_task_io_context in
set_task_ioprio") introduces a needless assignment
'ioc = task->io_context', as the local variable ioc is not further
used before returning.
Even after the further fix, commit a957b61254a7 ("block: fix error in
handling dead task for ioprio setting"), the assignment still remains
needless.
Drop this needless assignment in set_task_ioprio().
This code smell was identified with 'make clang-analyzer'.
Fixes: 5fc11eebb4a9 ("block: open code create_task_io_context in set_task_ioprio")
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20211223125300.20691-1-lukas.bulwahn@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
| -rw-r--r-- | block/blk-ioc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/block/blk-ioc.c b/block/blk-ioc.c index 71c3a933cf16..11f49f78db32 100644 --- a/block/blk-ioc.c +++ b/block/blk-ioc.c @@ -284,12 +284,10 @@ int set_task_ioprio(struct task_struct *task, int ioprio) kmem_cache_free(iocontext_cachep, ioc); goto out; } - if (task->io_context) { + if (task->io_context) kmem_cache_free(iocontext_cachep, ioc); - ioc = task->io_context; - } else { + else task->io_context = ioc; - } } task->io_context->ioprio = ioprio; out: |