diff options
| author | KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> | 2011-05-19 02:08:58 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2011-05-28 11:02:57 -0400 |
| commit | 1e1b6c511d1b23cb7c3b619d82fc7bd9f620565d (patch) | |
| tree | c50e88412c8b42264177dc125f74a30f9c7a82d9 /include | |
| parent | 1e876231785d82443a5ac8b6c660e9f51bc5dede (diff) | |
cpuset: Fix cpuset_cpus_allowed_fallback(), don't update tsk->rt.nr_cpus_allowed
The rule is, we have to update tsk->rt.nr_cpus_allowed if we change
tsk->cpus_allowed. Otherwise RT scheduler may confuse.
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/4DD4B3FA.5060901@jp.fujitsu.com
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/cpuset.h | 2 | ||||
| -rw-r--r-- | include/linux/sched.h | 7 |
2 files changed, 8 insertions, 1 deletions
diff --git a/include/linux/cpuset.h b/include/linux/cpuset.h index f20eb8f16025..e9eaec522655 100644 --- a/include/linux/cpuset.h +++ b/include/linux/cpuset.h | |||
| @@ -146,7 +146,7 @@ static inline void cpuset_cpus_allowed(struct task_struct *p, | |||
| 146 | 146 | ||
| 147 | static inline int cpuset_cpus_allowed_fallback(struct task_struct *p) | 147 | static inline int cpuset_cpus_allowed_fallback(struct task_struct *p) |
| 148 | { | 148 | { |
| 149 | cpumask_copy(&p->cpus_allowed, cpu_possible_mask); | 149 | do_set_cpus_allowed(p, cpu_possible_mask); |
| 150 | return cpumask_any(cpu_active_mask); | 150 | return cpumask_any(cpu_active_mask); |
| 151 | } | 151 | } |
| 152 | 152 | ||
diff --git a/include/linux/sched.h b/include/linux/sched.h index dc8871295a5a..8da84b7bc1b8 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h | |||
| @@ -1841,9 +1841,16 @@ static inline void rcu_copy_process(struct task_struct *p) | |||
| 1841 | #endif | 1841 | #endif |
| 1842 | 1842 | ||
| 1843 | #ifdef CONFIG_SMP | 1843 | #ifdef CONFIG_SMP |
| 1844 | extern void do_set_cpus_allowed(struct task_struct *p, | ||
| 1845 | const struct cpumask *new_mask); | ||
| 1846 | |||
| 1844 | extern int set_cpus_allowed_ptr(struct task_struct *p, | 1847 | extern int set_cpus_allowed_ptr(struct task_struct *p, |
| 1845 | const struct cpumask *new_mask); | 1848 | const struct cpumask *new_mask); |
| 1846 | #else | 1849 | #else |
| 1850 | static inline void do_set_cpus_allowed(struct task_struct *p, | ||
| 1851 | const struct cpumask *new_mask) | ||
| 1852 | { | ||
| 1853 | } | ||
| 1847 | static inline int set_cpus_allowed_ptr(struct task_struct *p, | 1854 | static inline int set_cpus_allowed_ptr(struct task_struct *p, |
| 1848 | const struct cpumask *new_mask) | 1855 | const struct cpumask *new_mask) |
| 1849 | { | 1856 | { |
