Commit 8a2ba178 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Jens Axboe
Browse files

block: remove the nr_task field from struct io_context 



Nothing ever looks at ->nr_tasks, so remove it.

Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarJan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20211209063131.18537-2-hch@lst.de


Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent d62cbcf6

block/blk-ioc.c

+0 −3
Original line number Diff line number Diff line
@@ -207,7 +207,6 @@ void exit_io_context(struct task_struct *task) 
	task->io_context = NULL;

	task_unlock(task);



	atomic_dec(&ioc->nr_tasks);

	put_io_context_active(ioc);

}
@@ -259,7 +258,6 @@ static struct io_context *alloc_io_context(gfp_t gfp_flags, int node) 
		return NULL;



	atomic_long_set(&ioc->refcount, 1);

	atomic_set(&ioc->nr_tasks, 1);

	atomic_set(&ioc->active_ref, 1);

	spin_lock_init(&ioc->lock);

	INIT_RADIX_TREE(&ioc->icq_tree, GFP_ATOMIC);
@@ -339,7 +337,6 @@ int __copy_io(unsigned long clone_flags, struct task_struct *tsk) 
	if (clone_flags & CLONE_IO) {

		atomic_long_inc(&ioc->refcount);

		atomic_inc(&ioc->active_ref);

		atomic_inc(&ioc->nr_tasks);

		tsk->io_context = ioc;

	} else if (ioprio_valid(ioc->ioprio)) {

		tsk->io_context = alloc_io_context(GFP_KERNEL, NUMA_NO_NODE);

include/linux/iocontext.h

+0 −1
Original line number Diff line number Diff line
@@ -99,7 +99,6 @@ struct io_cq { 
struct io_context {

	atomic_long_t refcount;

	atomic_t active_ref;

	atomic_t nr_tasks;



	/* all the fields below are protected by this lock */

	spinlock_t lock;