From: Atish Patra <atish.patra@wdc.com> To: linux-riscv@lists.infradead.org Cc: "Atish Patra" <atish.patra@wdc.com>, "Alan Kao" <alankao@andestech.com>, "Albert Ou" <aou@eecs.berkeley.edu>, "Andreas Schwab" <schwab@suse.de>, "Anup Patel" <anup@brainfault.org>, "Daniel Lezcano" <daniel.lezcano@linaro.org>, "Dmitriy Cherkasov" <dmitriy@oss-tech.org>, "Jason Cooper" <jason@lakedaemon.net>, linux-kernel@vger.kernel.org, "Marc Zyngier" <marc.zyngier@arm.com>, "Michael Clark" <michaeljclark@mac.com>, "Palmer Dabbelt" <palmer@sifive.com>, "Patrick Stählin" <me@packi.ch>, "Thomas Gleixner" <tglx@linutronix.de>, "Zong Li" <zongbox@gmail.com> Subject: [PATCH v2 1/8] RISC-V: Do not wait indefinitely in __cpu_up Date: Tue, 8 Jan 2019 01:38:31 -0800 [thread overview] Message-ID: <1546940318-9752-2-git-send-email-atish.patra@wdc.com> (raw) In-Reply-To: <1546940318-9752-1-git-send-email-atish.patra@wdc.com> In SMP path, __cpu_up waits for other CPU to come online indefinitely. This is wrong as other CPU might be disabled in machine mode and possible CPU is set to the cpus present in DT. Introduce a completion variable and waits only for a second. Signed-off-by: Atish Patra <atish.patra@wdc.com> Reviewed-by: Anup Patel <anup@brainfault.org> --- arch/riscv/kernel/smpboot.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/arch/riscv/kernel/smpboot.c b/arch/riscv/kernel/smpboot.c index fc185eca..32e14572 100644 --- a/arch/riscv/kernel/smpboot.c +++ b/arch/riscv/kernel/smpboot.c @@ -39,6 +39,7 @@ void *__cpu_up_stack_pointer[NR_CPUS]; void *__cpu_up_task_pointer[NR_CPUS]; +static DECLARE_COMPLETION(cpu_running); void __init smp_prepare_boot_cpu(void) { @@ -81,6 +82,7 @@ void __init setup_smp(void) int __cpu_up(unsigned int cpu, struct task_struct *tidle) { + int ret = 0; int hartid = cpuid_to_hartid_map(cpu); tidle->thread_info.cpu = cpu; @@ -96,10 +98,15 @@ int __cpu_up(unsigned int cpu, struct task_struct *tidle) task_stack_page(tidle) + THREAD_SIZE); WRITE_ONCE(__cpu_up_task_pointer[hartid], tidle); - while (!cpu_online(cpu)) - cpu_relax(); + wait_for_completion_timeout(&cpu_running, + msecs_to_jiffies(1000)); - return 0; + if (!cpu_online(cpu)) { + pr_crit("CPU%u: failed to come online\n", cpu); + ret = -EIO; + } + + return ret; } void __init smp_cpus_done(unsigned int max_cpus) @@ -125,6 +132,7 @@ asmlinkage void __init smp_callin(void) * a local TLB flush right now just in case. */ local_flush_tlb_all(); + complete(&cpu_running); /* * Disable preemption before enabling interrupts, so we don't try to * schedule a CPU that hasn't actually started yet. -- 2.7.4
WARNING: multiple messages have this Message-ID (diff)
From: Atish Patra <atish.patra@wdc.com> To: linux-riscv@lists.infradead.org Cc: "Patrick Stählin" <me@packi.ch>, "Albert Ou" <aou@eecs.berkeley.edu>, "Jason Cooper" <jason@lakedaemon.net>, "Alan Kao" <alankao@andestech.com>, "Dmitriy Cherkasov" <dmitriy@oss-tech.org>, "Anup Patel" <anup@brainfault.org>, "Daniel Lezcano" <daniel.lezcano@linaro.org>, linux-kernel@vger.kernel.org, "Michael Clark" <michaeljclark@mac.com>, "Atish Patra" <atish.patra@wdc.com>, "Palmer Dabbelt" <palmer@sifive.com>, "Andreas Schwab" <schwab@suse.de>, "Marc Zyngier" <marc.zyngier@arm.com>, "Thomas Gleixner" <tglx@linutronix.de>, "Zong Li" <zongbox@gmail.com> Subject: [PATCH v2 1/8] RISC-V: Do not wait indefinitely in __cpu_up Date: Tue, 8 Jan 2019 01:38:31 -0800 [thread overview] Message-ID: <1546940318-9752-2-git-send-email-atish.patra@wdc.com> (raw) In-Reply-To: <1546940318-9752-1-git-send-email-atish.patra@wdc.com> In SMP path, __cpu_up waits for other CPU to come online indefinitely. This is wrong as other CPU might be disabled in machine mode and possible CPU is set to the cpus present in DT. Introduce a completion variable and waits only for a second. Signed-off-by: Atish Patra <atish.patra@wdc.com> Reviewed-by: Anup Patel <anup@brainfault.org> --- arch/riscv/kernel/smpboot.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/arch/riscv/kernel/smpboot.c b/arch/riscv/kernel/smpboot.c index fc185eca..32e14572 100644 --- a/arch/riscv/kernel/smpboot.c +++ b/arch/riscv/kernel/smpboot.c @@ -39,6 +39,7 @@ void *__cpu_up_stack_pointer[NR_CPUS]; void *__cpu_up_task_pointer[NR_CPUS]; +static DECLARE_COMPLETION(cpu_running); void __init smp_prepare_boot_cpu(void) { @@ -81,6 +82,7 @@ void __init setup_smp(void) int __cpu_up(unsigned int cpu, struct task_struct *tidle) { + int ret = 0; int hartid = cpuid_to_hartid_map(cpu); tidle->thread_info.cpu = cpu; @@ -96,10 +98,15 @@ int __cpu_up(unsigned int cpu, struct task_struct *tidle) task_stack_page(tidle) + THREAD_SIZE); WRITE_ONCE(__cpu_up_task_pointer[hartid], tidle); - while (!cpu_online(cpu)) - cpu_relax(); + wait_for_completion_timeout(&cpu_running, + msecs_to_jiffies(1000)); - return 0; + if (!cpu_online(cpu)) { + pr_crit("CPU%u: failed to come online\n", cpu); + ret = -EIO; + } + + return ret; } void __init smp_cpus_done(unsigned int max_cpus) @@ -125,6 +132,7 @@ asmlinkage void __init smp_callin(void) * a local TLB flush right now just in case. */ local_flush_tlb_all(); + complete(&cpu_running); /* * Disable preemption before enabling interrupts, so we don't try to * schedule a CPU that hasn't actually started yet. -- 2.7.4 _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2019-01-08 9:38 UTC|newest] Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-01-08 9:38 [PATCH v2 0/8] Various SMP related fixes Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-08 9:38 ` Atish Patra [this message] 2019-01-08 9:38 ` [PATCH v2 1/8] RISC-V: Do not wait indefinitely in __cpu_up Atish Patra 2019-01-15 13:51 ` Christoph Hellwig 2019-01-15 13:51 ` Christoph Hellwig 2019-01-18 2:35 ` Atish Patra 2019-01-18 2:35 ` Atish Patra 2019-01-18 7:20 ` Christoph Hellwig 2019-01-18 7:20 ` Christoph Hellwig 2019-01-08 9:38 ` [PATCH v2 2/8] RISC-V: Move cpuid to hartid mapping to SMP Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-15 13:52 ` Christoph Hellwig 2019-01-15 13:52 ` Christoph Hellwig 2019-01-18 2:08 ` Atish Patra 2019-01-18 2:08 ` Atish Patra 2019-01-08 9:38 ` [PATCH v2 3/8] RISC-V: Remove NR_CPUs check during hartid search from DT Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-08 11:48 ` Anup Patel 2019-01-08 11:48 ` Anup Patel 2019-01-15 13:52 ` Christoph Hellwig 2019-01-15 13:52 ` Christoph Hellwig 2019-01-08 9:38 ` [PATCH v2 4/8] RISC-V: Allow hartid-to-cpuid function to fail Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-08 11:49 ` Anup Patel 2019-01-08 11:49 ` Anup Patel 2019-01-15 13:53 ` Christoph Hellwig 2019-01-15 13:53 ` Christoph Hellwig 2019-01-08 9:38 ` [PATCH v2 5/8] RISC-V: Compare cpuid with NR_CPUS before mapping Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-08 11:49 ` Anup Patel 2019-01-08 11:49 ` Anup Patel 2019-01-15 13:53 ` Christoph Hellwig 2019-01-15 13:53 ` Christoph Hellwig 2019-01-08 9:38 ` [PATCH v2 6/8] RISC-V: Add required checks during clock source init Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-08 11:56 ` Anup Patel 2019-01-08 11:56 ` Anup Patel 2019-01-18 2:10 ` Atish Patra 2019-01-18 2:10 ` Atish Patra 2019-01-15 13:54 ` Christoph Hellwig 2019-01-15 13:54 ` Christoph Hellwig 2019-01-08 9:38 ` [PATCH v2 7/8] RISC-V: Check and continue in case of an invalid cpuid Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-08 11:59 ` Anup Patel 2019-01-08 11:59 ` Anup Patel 2019-01-18 2:10 ` Atish Patra 2019-01-18 2:10 ` Atish Patra 2019-01-15 13:55 ` Christoph Hellwig 2019-01-15 13:55 ` Christoph Hellwig 2019-01-08 9:38 ` [PATCH v2 8/8] RISC-V: Assign hwcap only according to current cpu Atish Patra 2019-01-08 9:38 ` Atish Patra 2019-01-08 10:33 ` Atish Patra 2019-01-08 10:33 ` Atish Patra 2019-01-08 12:01 ` Anup Patel 2019-01-08 12:01 ` Anup Patel 2019-01-15 13:56 ` Christoph Hellwig 2019-01-15 13:56 ` Christoph Hellwig 2019-01-18 2:13 ` Atish Patra 2019-01-18 2:13 ` Atish Patra
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1546940318-9752-2-git-send-email-atish.patra@wdc.com \ --to=atish.patra@wdc.com \ --cc=alankao@andestech.com \ --cc=anup@brainfault.org \ --cc=aou@eecs.berkeley.edu \ --cc=daniel.lezcano@linaro.org \ --cc=dmitriy@oss-tech.org \ --cc=jason@lakedaemon.net \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=marc.zyngier@arm.com \ --cc=me@packi.ch \ --cc=michaeljclark@mac.com \ --cc=palmer@sifive.com \ --cc=schwab@suse.de \ --cc=tglx@linutronix.de \ --cc=zongbox@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.