diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2015-09-09 18:38:16 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-09-10 16:29:01 -0400 |
commit | d097c0240ae8085dd39aa6ca9bd9960969b2b38e (patch) | |
tree | 5986975db044124663cadc69809529b87ec9c55e /kernel/kmod.c | |
parent | b6b50a814d0ece9c1f98f2b3b5c2a251a5c9a211 (diff) |
kmod: remove unecessary explicit wide CPU affinity setting
Khelper is affine to all CPUs. Now since it creates the
call_usermodehelper_exec_[a]sync() kernel threads, those inherit the wide
affinity.
As such explicitly forcing a wide affinity from those kernel threads
is like a no-op.
Just remove it. It's needless and it breaks CPU isolation users who
rely on workqueue affinity tuning.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Rik van Riel <riel@redhat.com>
Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Cc: Christoph Lameter <cl@linux.com>
Cc: Tejun Heo <tj@kernel.org>
Cc: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel/kmod.c')
-rw-r--r-- | kernel/kmod.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/kernel/kmod.c b/kernel/kmod.c index 2d83511e9610..d910b6378fb6 100644 --- a/kernel/kmod.c +++ b/kernel/kmod.c | |||
@@ -224,9 +224,6 @@ static int call_usermodehelper_exec_async(void *data) | |||
224 | flush_signal_handlers(current, 1); | 224 | flush_signal_handlers(current, 1); |
225 | spin_unlock_irq(¤t->sighand->siglock); | 225 | spin_unlock_irq(¤t->sighand->siglock); |
226 | 226 | ||
227 | /* We can run anywhere, unlike our parent keventd(). */ | ||
228 | set_cpus_allowed_ptr(current, cpu_all_mask); | ||
229 | |||
230 | /* | 227 | /* |
231 | * Our parent is keventd, which runs with elevated scheduling priority. | 228 | * Our parent is keventd, which runs with elevated scheduling priority. |
232 | * Avoid propagating that into the userspace child. | 229 | * Avoid propagating that into the userspace child. |