From: Jan Henrik Weinstock <jan.weinstock@rwth-aachen.de> To: Stafford Horne <shorne@gmail.com>, <geert@linux-m68k.org> Cc: <jonas@southpole.se>, <stefan.kristiansson@saunalahti.fi>, <openrisc@lists.librecores.org>, <linux-kernel@vger.kernel.org> Subject: Re: [PATCH] openrisc: use device tree to determine present cpus Date: Sat, 30 Jan 2021 12:00:10 +0100 [thread overview] Message-ID: <2a018afc-d797-3a91-ffab-e55ae3b0a795@rwth-aachen.de> (raw) In-Reply-To: <20210129221643.GZ2002709@lianli.shorne-pla.net> [-- Attachment #1: Type: text/plain, Size: 1748 bytes --] Hi Stafford, Geert, thanks for your feedback. I have made the following changes to the patch: 1. use for_each_of_cpu_node 2. possible_cpus is now what is in the devicetree, up to NR_CPUS 3. present_cpus is now all possible cpus, up to max_cpus Signed-off-by: Jan Henrik Weinstock <jan.weinstock@rwth-aachen.de> --- arch/openrisc/kernel/smp.c | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/arch/openrisc/kernel/smp.c b/arch/openrisc/kernel/smp.c index 29c82ef2e..83cbf43d4 100644 --- a/arch/openrisc/kernel/smp.c +++ b/arch/openrisc/kernel/smp.c @@ -16,6 +16,7 @@ #include <linux/sched.h> #include <linux/sched/mm.h> #include <linux/irq.h> +#include <linux/of.h> #include <asm/cpuinfo.h> #include <asm/mmu_context.h> #include <asm/tlbflush.h> @@ -60,22 +61,32 @@ void __init smp_prepare_boot_cpu(void) void __init smp_init_cpus(void) { - int i; + struct device_node* cpu; + u32 cpu_id; - for (i = 0; i < NR_CPUS; i++) - set_cpu_possible(i, true); + for_each_of_cpu_node(cpu) { + if (of_property_read_u32(cpu, "reg", &cpu_id)) { + pr_warn("%s missing reg property", cpu->full_name); + continue; + } + + if (cpu_id < NR_CPUS) + set_cpu_possible(cpu_id, true); + } } void __init smp_prepare_cpus(unsigned int max_cpus) { - int i; + unsigned int cpu; /* * Initialise the present map, which describes the set of CPUs * actually populated at the present time. */ - for (i = 0; i < max_cpus; i++) - set_cpu_present(i, true); + for_each_possible_cpu(cpu) { + if (cpu < max_cpus) + set_cpu_present(cpu, true); + } } void __init smp_cpus_done(unsigned int max_cpus) -- 2.17.1 [-- Attachment #2: S/MIME Cryptographic Signature --] [-- Type: application/pkcs7-signature, Size: 5316 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Jan Henrik Weinstock <jan.weinstock@rwth-aachen.de> To: openrisc@lists.librecores.org Subject: [OpenRISC] [PATCH] openrisc: use device tree to determine present cpus Date: Sat, 30 Jan 2021 12:00:10 +0100 [thread overview] Message-ID: <2a018afc-d797-3a91-ffab-e55ae3b0a795@rwth-aachen.de> (raw) In-Reply-To: <20210129221643.GZ2002709@lianli.shorne-pla.net> Hi Stafford, Geert, thanks for your feedback. I have made the following changes to the patch: 1. use for_each_of_cpu_node 2. possible_cpus is now what is in the devicetree, up to NR_CPUS 3. present_cpus is now all possible cpus, up to max_cpus Signed-off-by: Jan Henrik Weinstock <jan.weinstock@rwth-aachen.de> --- arch/openrisc/kernel/smp.c | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/arch/openrisc/kernel/smp.c b/arch/openrisc/kernel/smp.c index 29c82ef2e..83cbf43d4 100644 --- a/arch/openrisc/kernel/smp.c +++ b/arch/openrisc/kernel/smp.c @@ -16,6 +16,7 @@ #include <linux/sched.h> #include <linux/sched/mm.h> #include <linux/irq.h> +#include <linux/of.h> #include <asm/cpuinfo.h> #include <asm/mmu_context.h> #include <asm/tlbflush.h> @@ -60,22 +61,32 @@ void __init smp_prepare_boot_cpu(void) void __init smp_init_cpus(void) { - int i; + struct device_node* cpu; + u32 cpu_id; - for (i = 0; i < NR_CPUS; i++) - set_cpu_possible(i, true); + for_each_of_cpu_node(cpu) { + if (of_property_read_u32(cpu, "reg", &cpu_id)) { + pr_warn("%s missing reg property", cpu->full_name); + continue; + } + + if (cpu_id < NR_CPUS) + set_cpu_possible(cpu_id, true); + } } void __init smp_prepare_cpus(unsigned int max_cpus) { - int i; + unsigned int cpu; /* * Initialise the present map, which describes the set of CPUs * actually populated@the present time. */ - for (i = 0; i < max_cpus; i++) - set_cpu_present(i, true); + for_each_possible_cpu(cpu) { + if (cpu < max_cpus) + set_cpu_present(cpu, true); + } } void __init smp_cpus_done(unsigned int max_cpus) -- 2.17.1 -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/pkcs7-signature Size: 5316 bytes Desc: S/MIME Cryptographic Signature URL: <http://lists.librecores.org/pipermail/openrisc/attachments/20210130/0e05546d/attachment.bin>
next prev parent reply other threads:[~2021-01-30 11:01 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-01-29 18:29 [PATCH] openrisc: use device tree to determine present cpus Jan Henrik Weinstock 2021-01-29 18:29 ` [OpenRISC] " Jan Henrik Weinstock 2021-01-29 22:16 ` Stafford Horne 2021-01-29 22:16 ` [OpenRISC] " Stafford Horne 2021-01-30 11:00 ` Jan Henrik Weinstock [this message] 2021-01-30 11:00 ` Jan Henrik Weinstock 2021-01-30 23:03 ` Stafford Horne 2021-01-30 23:03 ` [OpenRISC] " Stafford Horne 2021-01-30 23:11 ` Stafford Horne 2021-01-30 23:11 ` [OpenRISC] " Stafford Horne 2021-01-31 8:22 ` Jan Henrik Weinstock 2021-01-31 8:22 ` [OpenRISC] " Jan Henrik Weinstock 2021-01-31 21:27 ` Stafford Horne 2021-01-31 21:27 ` [OpenRISC] " Stafford Horne 2021-02-01 11:49 ` [PATCH v2] " Jan Henrik Weinstock 2021-02-01 11:49 ` [OpenRISC] " Jan Henrik Weinstock 2021-02-05 14:43 ` Stafford Horne 2021-02-05 14:43 ` [OpenRISC] " Stafford Horne 2021-02-05 16:07 ` Geert Uytterhoeven 2021-02-05 16:07 ` [OpenRISC] " Geert Uytterhoeven 2021-02-05 22:36 ` Stafford Horne 2021-02-05 22:36 ` [OpenRISC] " Stafford Horne 2021-02-06 9:33 ` Geert Uytterhoeven 2021-02-06 9:33 ` [OpenRISC] " Geert Uytterhoeven 2021-02-08 12:16 ` Stafford Horne 2021-02-08 12:22 ` Geert Uytterhoeven 2021-02-08 12:22 ` [OpenRISC] " Geert Uytterhoeven 2021-01-30 10:37 ` [PATCH] " Geert Uytterhoeven 2021-01-30 10:37 ` [OpenRISC] " Geert Uytterhoeven
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=2a018afc-d797-3a91-ffab-e55ae3b0a795@rwth-aachen.de \ --to=jan.weinstock@rwth-aachen.de \ --cc=geert@linux-m68k.org \ --cc=jonas@southpole.se \ --cc=linux-kernel@vger.kernel.org \ --cc=openrisc@lists.librecores.org \ --cc=shorne@gmail.com \ --cc=stefan.kristiansson@saunalahti.fi \ /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.