summaryrefslogtreecommitdiffstats
path: root/block
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@lst.de>2021-12-09 07:31:28 +0100
committerJens Axboe <axboe@kernel.dk>2021-12-16 10:59:02 -0700
commit8472161b77c41d260c5ba0af6bf940269b297bb6 (patch)
tree1e4aadcc25be7f8501fcdc2269b4b6c9e3cf7d08 /block
parenta411cd3cfdc5bbd1329d5b33dbf39e2b5213969d (diff)
downloadlinux-stable-8472161b77c41d260c5ba0af6bf940269b297bb6.tar.gz
linux-stable-8472161b77c41d260c5ba0af6bf940269b297bb6.tar.bz2
linux-stable-8472161b77c41d260c5ba0af6bf940269b297bb6.zip
block: fold get_task_io_context into set_task_ioprio
Fold get_task_io_context into its only caller, and simplify the code as no reference to the I/O context is required to just set the ioprio field. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Jan Kara <jack@suse.cz> Link: https://lore.kernel.org/r/20211209063131.18537-9-hch@lst.de Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block')
-rw-r--r--block/blk-ioc.c52
1 files changed, 14 insertions, 38 deletions
diff --git a/block/blk-ioc.c b/block/blk-ioc.c
index c25ce2f3eb19..1ba7cfedca2d 100644
--- a/block/blk-ioc.c
+++ b/block/blk-ioc.c
@@ -268,41 +268,9 @@ static struct io_context *create_task_io_context(struct task_struct *task,
return ioc;
}
-/**
- * get_task_io_context - get io_context of a task
- * @task: task of interest
- * @gfp_flags: allocation flags, used if allocation is necessary
- * @node: allocation node, used if allocation is necessary
- *
- * Return io_context of @task. If it doesn't exist, it is created with
- * @gfp_flags and @node. The returned io_context has its reference count
- * incremented.
- *
- * This function always goes through task_lock() and it's better to use
- * %current->io_context + get_io_context() for %current.
- */
-static struct io_context *get_task_io_context(struct task_struct *task,
- gfp_t gfp_flags, int node)
-{
- struct io_context *ioc;
-
- might_sleep_if(gfpflags_allow_blocking(gfp_flags));
-
- task_lock(task);
- ioc = task->io_context;
- if (unlikely(!ioc)) {
- task_unlock(task);
- return create_task_io_context(task, gfp_flags, node);
- }
- get_io_context(ioc);
- task_unlock(task);
- return ioc;
-}
-
int set_task_ioprio(struct task_struct *task, int ioprio)
{
int err;
- struct io_context *ioc;
const struct cred *cred = current_cred(), *tcred;
rcu_read_lock();
@@ -318,13 +286,21 @@ int set_task_ioprio(struct task_struct *task, int ioprio)
if (err)
return err;
- ioc = get_task_io_context(task, GFP_ATOMIC, NUMA_NO_NODE);
- if (ioc) {
- ioc->ioprio = ioprio;
- put_io_context(ioc);
- }
+ task_lock(task);
+ if (unlikely(!task->io_context)) {
+ struct io_context *ioc;
- return err;
+ task_unlock(task);
+ ioc = create_task_io_context(task, GFP_ATOMIC, NUMA_NO_NODE);
+ if (ioc) {
+ ioc->ioprio = ioprio;
+ put_io_context(ioc);
+ }
+ return 0;
+ }
+ task->io_context->ioprio = ioprio;
+ task_unlock(task);
+ return 0;
}
EXPORT_SYMBOL_GPL(set_task_ioprio);