From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752729AbcLHNqy (ORCPT ); Thu, 8 Dec 2016 08:46:54 -0500 Received: from terminus.zytor.com ([198.137.202.10]:59800 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751570AbcLHNqw (ORCPT ); Thu, 8 Dec 2016 08:46:52 -0500 Date: Thu, 8 Dec 2016 05:45:12 -0800 From: tip-bot for Oleg Nesterov Message-ID: Cc: tglx@linutronix.de, luto@amacapital.net, David1.Zhou@amd.com, roman.penyaev@profitbricks.com, linux-kernel@vger.kernel.org, oleg@redhat.com, tj@kernel.org, peterz@infradead.org, luto@kernel.org, mingo@kernel.org, akpm@linux-foundation.org, alexander.deucher@amd.com, hpa@zytor.com, pmladek@suse.com Reply-To: peterz@infradead.org, luto@kernel.org, akpm@linux-foundation.org, mingo@kernel.org, pmladek@suse.com, hpa@zytor.com, alexander.deucher@amd.com, tglx@linutronix.de, luto@amacapital.net, David1.Zhou@amd.com, oleg@redhat.com, roman.penyaev@profitbricks.com, linux-kernel@vger.kernel.org, tj@kernel.org In-Reply-To: <20161129175110.GA5342@redhat.com> References: <20161129175110.GA5342@redhat.com> To: linux-tip-commits@vger.kernel.org Subject: [tip:sched/core] kthread: Don't abuse kthread_create_on_cpu() in __kthread_create_worker() Git-Commit-ID: 8fb9dcbdc3619741c10c573199d804161c34c89a X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: 8fb9dcbdc3619741c10c573199d804161c34c89a Gitweb: http://git.kernel.org/tip/8fb9dcbdc3619741c10c573199d804161c34c89a Author: Oleg Nesterov AuthorDate: Tue, 29 Nov 2016 18:51:10 +0100 Committer: Thomas Gleixner CommitDate: Thu, 8 Dec 2016 14:36:20 +0100 kthread: Don't abuse kthread_create_on_cpu() in __kthread_create_worker() kthread_create_on_cpu() sets KTHREAD_IS_PER_CPU and kthread->cpu, this only makes sense if this kthread can be parked/unparked by cpuhp code. kthread workers never call kthread_parkme() so this has no effect. Change __kthread_create_worker() to simply call kthread_bind(task, cpu). The very fact that kthread_create_on_cpu() doesn't accept a generic fmt shows that it should not be used outside of smpboot.c. Now, the only reason we can not unexport this helper and move it into smpboot.c is that it sets kthread->cpu and struct kthread is not exported. And the only reason we can not kill kthread->cpu is that kthread_unpark() is used by drivers/gpu/drm/amd/scheduler/gpu_scheduler.c and thus we can not turn _unpark into kthread_unpark(struct smp_hotplug_thread *, cpu). Signed-off-by: Oleg Nesterov Tested-by: Petr Mladek Acked-by: Peter Zijlstra (Intel) Reviewed-by: Petr Mladek Cc: Chunming Zhou Cc: Roman Pen Cc: Andy Lutomirski Cc: Tejun Heo Cc: Andy Lutomirski Cc: Alex Deucher Cc: Andrew Morton Link: http://lkml.kernel.org/r/20161129175110.GA5342@redhat.com Signed-off-by: Thomas Gleixner --- kernel/kthread.c | 23 ++++++++--------------- 1 file changed, 8 insertions(+), 15 deletions(-) diff --git a/kernel/kthread.c b/kernel/kthread.c index 01d2716..956495f 100644 --- a/kernel/kthread.c +++ b/kernel/kthread.c @@ -641,6 +641,7 @@ __kthread_create_worker(int cpu, unsigned int flags, { struct kthread_worker *worker; struct task_struct *task; + int node = -1; worker = kzalloc(sizeof(*worker), GFP_KERNEL); if (!worker) @@ -648,25 +649,17 @@ __kthread_create_worker(int cpu, unsigned int flags, kthread_init_worker(worker); - if (cpu >= 0) { - char name[TASK_COMM_LEN]; - - /* - * kthread_create_worker_on_cpu() allows to pass a generic - * namefmt in compare with kthread_create_on_cpu. We need - * to format it here. - */ - vsnprintf(name, sizeof(name), namefmt, args); - task = kthread_create_on_cpu(kthread_worker_fn, worker, - cpu, name); - } else { - task = __kthread_create_on_node(kthread_worker_fn, worker, - -1, namefmt, args); - } + if (cpu >= 0) + node = cpu_to_node(cpu); + task = __kthread_create_on_node(kthread_worker_fn, worker, + node, namefmt, args); if (IS_ERR(task)) goto fail_task; + if (cpu >= 0) + kthread_bind(task, cpu); + worker->flags = flags; worker->task = task; wake_up_process(task);