sched: Always clear user_cpus_ptr in do_set_cpus_allowed()
The do_set_cpus_allowed() function is used by either kthread_bind() or select_fallback_rq(). In both cases the user affinity (if any) should be destroyed too. Suggested-by: Peter Zijlstra <peterz@infradead.org> Signed-off-by: Waiman Long <longman@redhat.com> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org> Link: https://lkml.kernel.org/r/20220922180041.1768141-6-longman@redhat.com
This commit is contained in:
parent
da01903281
commit
851a723e45
1 changed files with 7 additions and 1 deletions
|
@ -2593,14 +2593,20 @@ __do_set_cpus_allowed(struct task_struct *p, struct affinity_context *ctx)
|
||||||
set_next_task(rq, p);
|
set_next_task(rq, p);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Used for kthread_bind() and select_fallback_rq(), in both cases the user
|
||||||
|
* affinity (if any) should be destroyed too.
|
||||||
|
*/
|
||||||
void do_set_cpus_allowed(struct task_struct *p, const struct cpumask *new_mask)
|
void do_set_cpus_allowed(struct task_struct *p, const struct cpumask *new_mask)
|
||||||
{
|
{
|
||||||
struct affinity_context ac = {
|
struct affinity_context ac = {
|
||||||
.new_mask = new_mask,
|
.new_mask = new_mask,
|
||||||
.flags = 0,
|
.user_mask = NULL,
|
||||||
|
.flags = SCA_USER, /* clear the user requested mask */
|
||||||
};
|
};
|
||||||
|
|
||||||
__do_set_cpus_allowed(p, &ac);
|
__do_set_cpus_allowed(p, &ac);
|
||||||
|
kfree(ac.user_mask);
|
||||||
}
|
}
|
||||||
|
|
||||||
int dup_user_cpus_ptr(struct task_struct *dst, struct task_struct *src,
|
int dup_user_cpus_ptr(struct task_struct *dst, struct task_struct *src,
|
||||||
|
|
Loading…
Add table
Reference in a new issue