Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4939602a authored by Peter Zijlstra's avatar Peter Zijlstra Committed by Ingo Molnar
Browse files

sched: Unify the ->cpus_allowed mask copy 



Currently every sched_class::set_cpus_allowed() implementation has to
copy the cpumask into task_struct::cpus_allowed, this is pointless,
put this copy in the generic code.

Signed-off-by: default avatarPeter Zijlstra <a.p.zijlstra@chello.nl>
Acked-by: default avatarThomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/n/tip-jhl5s9fckd9ptw1fzbqqlrd3@git.kernel.org


Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent fa17b507

kernel/sched.c

+3 −4
Original line number Diff line number Diff line
@@ -6161,11 +6161,10 @@ void do_set_cpus_allowed(struct task_struct *p, const struct cpumask *new_mask) 
{

	if (p->sched_class && p->sched_class->set_cpus_allowed)

		p->sched_class->set_cpus_allowed(p, new_mask);

	else {



	cpumask_copy(&p->cpus_allowed, new_mask);

	p->rt.nr_cpus_allowed = cpumask_weight(new_mask);

}

}



/*

 * This is how migration works:

kernel/sched_rt.c

+0 −3
Original line number Diff line number Diff line
@@ -1608,9 +1608,6 @@ static void set_cpus_allowed_rt(struct task_struct *p, 


		update_rt_migration(&rq->rt);

	}



	cpumask_copy(&p->cpus_allowed, new_mask);

	p->rt.nr_cpus_allowed = weight;

}



/* Assumes rq->lock is held */