diff options
author | Ingo Molnar <mingo@kernel.org> | 2017-02-05 09:38:10 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2017-03-02 02:42:24 -0500 |
commit | 0c98d344fe5c27f6e4bce42ac503e9e9a51c7d1d (patch) | |
tree | 98a45a561f3cfe6a1be9942cfdf23af5f70ca7dd /kernel/sched/cpupri.c | |
parent | 59ddbcb2f45b958cf1f11f122b666cbcf50cd57b (diff) |
sched/core: Remove the tsk_cpus_allowed() wrapper
So the original intention of tsk_cpus_allowed() was to 'future-proof'
the field - but it's pretty ineffectual at that, because half of
the code uses ->cpus_allowed directly ...
Also, the wrapper makes the code longer than the original expression!
So just get rid of it. This also shrinks <linux/sched.h> a bit.
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel/sched/cpupri.c')
-rw-r--r-- | kernel/sched/cpupri.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/sched/cpupri.c b/kernel/sched/cpupri.c index 11e9705bf937..981fcd7dc394 100644 --- a/kernel/sched/cpupri.c +++ b/kernel/sched/cpupri.c | |||
@@ -103,11 +103,11 @@ int cpupri_find(struct cpupri *cp, struct task_struct *p, | |||
103 | if (skip) | 103 | if (skip) |
104 | continue; | 104 | continue; |
105 | 105 | ||
106 | if (cpumask_any_and(tsk_cpus_allowed(p), vec->mask) >= nr_cpu_ids) | 106 | if (cpumask_any_and(&p->cpus_allowed, vec->mask) >= nr_cpu_ids) |
107 | continue; | 107 | continue; |
108 | 108 | ||
109 | if (lowest_mask) { | 109 | if (lowest_mask) { |
110 | cpumask_and(lowest_mask, tsk_cpus_allowed(p), vec->mask); | 110 | cpumask_and(lowest_mask, &p->cpus_allowed, vec->mask); |
111 | 111 | ||
112 | /* | 112 | /* |
113 | * We have to ensure that we have at least one bit | 113 | * We have to ensure that we have at least one bit |