From: Conor Dooley <conor@kernel.org> To: Sunil V L <sunilvl@ventanamicro.com> Cc: Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu>, "Rafael J . Wysocki" <rafael@kernel.org>, Len Brown <lenb@kernel.org>, Thomas Gleixner <tglx@linutronix.de>, Marc Zyngier <maz@kernel.org>, Daniel Lezcano <daniel.lezcano@linaro.org>, Jonathan Corbet <corbet@lwn.net>, linux-riscv@lists.infradead.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, Anup Patel <apatel@ventanamicro.com>, Andrew Jones <ajones@ventanamicro.com>, Atish Patra <atishp@rivosinc.com> Subject: Re: [PATCH 14/24] RISC-V: ACPI: smpboot: Add function to retrieve the hartid Date: Thu, 9 Feb 2023 20:30:40 +0000 [thread overview] Message-ID: <Y+VX8EclC1cE2fdI@spud> (raw) In-Reply-To: <20230130182225.2471414-15-sunilvl@ventanamicro.com> [-- Attachment #1: Type: text/plain, Size: 3189 bytes --] Hey Sunil, Drew, @drew, a question below that I'm sorta aiming at you... On Mon, Jan 30, 2023 at 11:52:15PM +0530, Sunil V L wrote: > hartid is in RINTC structuire in MADT table. Instead of parsing Nit: missing articles before RINTC and MADT. Also typo "structure". Perhaps you'd benefit from a spell checker in your git editor. > the ACPI table every time we need for a cpu, cache it and provide > a function to read it. > > This is similar to acpi_get_madt_gicc() in arm64. -ENOTFOUND, do you mean acpi_cpu_get_madt_gicc()? > > Signed-off-by: Sunil V L <sunilvl@ventanamicro.com> > --- > arch/riscv/include/asm/acpi.h | 14 +++++++++++++- > arch/riscv/kernel/smpboot.c | 21 +++++++++++++++++++++ > 2 files changed, 34 insertions(+), 1 deletion(-) > > diff --git a/arch/riscv/include/asm/acpi.h b/arch/riscv/include/asm/acpi.h > index d1f1e53ec657..69a880b7257a 100644 > --- a/arch/riscv/include/asm/acpi.h > +++ b/arch/riscv/include/asm/acpi.h > @@ -65,6 +65,18 @@ int acpi_numa_get_nid(unsigned int cpu); > static inline int acpi_numa_get_nid(unsigned int cpu) { return NUMA_NO_NODE; } > #endif /* CONFIG_ACPI_NUMA */ > > -#endif > +struct acpi_madt_rintc *acpi_get_madt_rintc(int cpu); > +struct acpi_madt_rintc *acpi_cpu_get_madt_rintc(int cpu); > +static inline u32 get_acpi_id_for_cpu(int cpu) > +{ > + return acpi_cpu_get_madt_rintc(cpu)->uid; > +} > +#else > +static inline u32 get_acpi_id_for_cpu(int cpu) > +{ > + return -1; > +} > + > +#endif /* CONFIG_ACPI */ > > #endif /*_ASM_ACPI_H*/ > diff --git a/arch/riscv/kernel/smpboot.c b/arch/riscv/kernel/smpboot.c > index e48cf88d0bc1..3a8b7a9eb5ac 100644 > --- a/arch/riscv/kernel/smpboot.c > +++ b/arch/riscv/kernel/smpboot.c > @@ -73,6 +73,25 @@ void __init smp_prepare_cpus(unsigned int max_cpus) > > #ifdef CONFIG_ACPI > static unsigned int cpu_count = 1; > +static unsigned int intc_count; > +static struct acpi_madt_rintc cpu_madt_rintc[NR_CPUS]; > + > +struct acpi_madt_rintc *acpi_get_madt_rintc(int cpu) > +{ > + return &cpu_madt_rintc[cpu]; > +} > + > +struct acpi_madt_rintc *acpi_cpu_get_madt_rintc(int cpu) > +{ > + int i; Since we are C11 now, you don't even need to declare this outside of the loop, right? > + > + for (i = 0; i < NR_CPUS; i++) { @drew, perhaps you know since you were fiddling not too long ago with cpumask stuff - at what point does for_each_possible_cpu() become usable? I had a bit of a poke & couldn't immediately tell if it'd be okay to use it here. > + if (riscv_hartid_to_cpuid(cpu_madt_rintc[i].hart_id) == cpu) > + return &cpu_madt_rintc[i]; > + } > + return NULL; Another nit: newline before return please :) > +} > +EXPORT_SYMBOL_GPL(acpi_cpu_get_madt_rintc); > > static int __init > acpi_parse_rintc(union acpi_subtable_headers *header, > @@ -92,6 +111,8 @@ acpi_parse_rintc(union acpi_subtable_headers *header, > hart = processor->hart_id; > if (hart < 0) > return 0; > + > + cpu_madt_rintc[intc_count++] = *processor; > if (hart == cpuid_to_hartid_map(0)) { > BUG_ON(found_boot_cpu); > found_boot_cpu = 1; > -- > 2.38.0 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Conor Dooley <conor@kernel.org> To: Sunil V L <sunilvl@ventanamicro.com> Cc: Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu>, "Rafael J . Wysocki" <rafael@kernel.org>, Len Brown <lenb@kernel.org>, Thomas Gleixner <tglx@linutronix.de>, Marc Zyngier <maz@kernel.org>, Daniel Lezcano <daniel.lezcano@linaro.org>, Jonathan Corbet <corbet@lwn.net>, linux-riscv@lists.infradead.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, Anup Patel <apatel@ventanamicro.com>, Andrew Jones <ajones@ventanamicro.com>, Atish Patra <atishp@rivosinc.com> Subject: Re: [PATCH 14/24] RISC-V: ACPI: smpboot: Add function to retrieve the hartid Date: Thu, 9 Feb 2023 20:30:40 +0000 [thread overview] Message-ID: <Y+VX8EclC1cE2fdI@spud> (raw) In-Reply-To: <20230130182225.2471414-15-sunilvl@ventanamicro.com> [-- Attachment #1.1: Type: text/plain, Size: 3189 bytes --] Hey Sunil, Drew, @drew, a question below that I'm sorta aiming at you... On Mon, Jan 30, 2023 at 11:52:15PM +0530, Sunil V L wrote: > hartid is in RINTC structuire in MADT table. Instead of parsing Nit: missing articles before RINTC and MADT. Also typo "structure". Perhaps you'd benefit from a spell checker in your git editor. > the ACPI table every time we need for a cpu, cache it and provide > a function to read it. > > This is similar to acpi_get_madt_gicc() in arm64. -ENOTFOUND, do you mean acpi_cpu_get_madt_gicc()? > > Signed-off-by: Sunil V L <sunilvl@ventanamicro.com> > --- > arch/riscv/include/asm/acpi.h | 14 +++++++++++++- > arch/riscv/kernel/smpboot.c | 21 +++++++++++++++++++++ > 2 files changed, 34 insertions(+), 1 deletion(-) > > diff --git a/arch/riscv/include/asm/acpi.h b/arch/riscv/include/asm/acpi.h > index d1f1e53ec657..69a880b7257a 100644 > --- a/arch/riscv/include/asm/acpi.h > +++ b/arch/riscv/include/asm/acpi.h > @@ -65,6 +65,18 @@ int acpi_numa_get_nid(unsigned int cpu); > static inline int acpi_numa_get_nid(unsigned int cpu) { return NUMA_NO_NODE; } > #endif /* CONFIG_ACPI_NUMA */ > > -#endif > +struct acpi_madt_rintc *acpi_get_madt_rintc(int cpu); > +struct acpi_madt_rintc *acpi_cpu_get_madt_rintc(int cpu); > +static inline u32 get_acpi_id_for_cpu(int cpu) > +{ > + return acpi_cpu_get_madt_rintc(cpu)->uid; > +} > +#else > +static inline u32 get_acpi_id_for_cpu(int cpu) > +{ > + return -1; > +} > + > +#endif /* CONFIG_ACPI */ > > #endif /*_ASM_ACPI_H*/ > diff --git a/arch/riscv/kernel/smpboot.c b/arch/riscv/kernel/smpboot.c > index e48cf88d0bc1..3a8b7a9eb5ac 100644 > --- a/arch/riscv/kernel/smpboot.c > +++ b/arch/riscv/kernel/smpboot.c > @@ -73,6 +73,25 @@ void __init smp_prepare_cpus(unsigned int max_cpus) > > #ifdef CONFIG_ACPI > static unsigned int cpu_count = 1; > +static unsigned int intc_count; > +static struct acpi_madt_rintc cpu_madt_rintc[NR_CPUS]; > + > +struct acpi_madt_rintc *acpi_get_madt_rintc(int cpu) > +{ > + return &cpu_madt_rintc[cpu]; > +} > + > +struct acpi_madt_rintc *acpi_cpu_get_madt_rintc(int cpu) > +{ > + int i; Since we are C11 now, you don't even need to declare this outside of the loop, right? > + > + for (i = 0; i < NR_CPUS; i++) { @drew, perhaps you know since you were fiddling not too long ago with cpumask stuff - at what point does for_each_possible_cpu() become usable? I had a bit of a poke & couldn't immediately tell if it'd be okay to use it here. > + if (riscv_hartid_to_cpuid(cpu_madt_rintc[i].hart_id) == cpu) > + return &cpu_madt_rintc[i]; > + } > + return NULL; Another nit: newline before return please :) > +} > +EXPORT_SYMBOL_GPL(acpi_cpu_get_madt_rintc); > > static int __init > acpi_parse_rintc(union acpi_subtable_headers *header, > @@ -92,6 +111,8 @@ acpi_parse_rintc(union acpi_subtable_headers *header, > hart = processor->hart_id; > if (hart < 0) > return 0; > + > + cpu_madt_rintc[intc_count++] = *processor; > if (hart == cpuid_to_hartid_map(0)) { > BUG_ON(found_boot_cpu); > found_boot_cpu = 1; > -- > 2.38.0 > [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 161 bytes --] _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2023-02-09 20:30 UTC|newest] Thread overview: 132+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-01-30 18:22 [PATCH 00/24] Add basic ACPI support for RISC-V Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 01/24] riscv: move sbi_init() earlier before jump_label_init() Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 02/24] ACPICA: MADT: Add RISC-V INTC interrupt controller Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-08 19:59 ` Conor Dooley 2023-02-08 19:59 ` Conor Dooley 2023-02-13 5:13 ` Sunil V L 2023-02-13 5:13 ` Sunil V L 2023-01-30 18:22 ` [PATCH 03/24] ACPICA: Add structure definitions for RISC-V RHCT Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 04/24] RISC-V: ACPI: Add empty headers to enable ACPI core Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-08 19:55 ` Conor Dooley 2023-02-08 19:55 ` Conor Dooley 2023-01-30 18:22 ` [PATCH 05/24] RISC-V: ACPI: Add basic functions to build " Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-08 20:58 ` Conor Dooley 2023-02-08 20:58 ` Conor Dooley 2023-02-13 15:16 ` Sunil V L 2023-02-13 15:16 ` Sunil V L 2023-01-30 18:22 ` [PATCH 06/24] RISC-V: ACPI: Add PCI " Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-08 21:26 ` Conor Dooley 2023-02-08 21:26 ` Conor Dooley 2023-02-13 15:23 ` Sunil V L 2023-02-13 15:23 ` Sunil V L 2023-02-13 17:14 ` Conor Dooley 2023-02-13 17:14 ` Conor Dooley 2023-02-13 17:26 ` Jessica Clarke 2023-02-13 17:26 ` Jessica Clarke 2023-02-14 4:42 ` Sunil V L 2023-02-14 4:42 ` Sunil V L 2023-01-30 18:22 ` [PATCH 07/24] RISC-V: ACPI: Enable ACPI build infrastructure Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-08 21:31 ` Conor Dooley 2023-02-08 21:31 ` Conor Dooley 2023-02-13 15:23 ` Sunil V L 2023-02-13 15:23 ` Sunil V L 2023-01-30 18:22 ` [PATCH 08/24] ACPI: Enable ACPI_PROCESSOR for RISC-V Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 09/24] ACPI: OSL: Make should_use_kmap() 0 " Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 10/24] ACPI: processor_core: RISC-V: Enable mapping processor to the hartid Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 11/24] RISC-V: ACPI: irqchip/riscv-intc: Add ACPI support Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 23:38 ` Jessica Clarke 2023-01-30 23:38 ` Jessica Clarke 2023-01-31 9:11 ` Sunil V L 2023-01-31 9:11 ` Sunil V L 2023-02-08 21:49 ` Conor Dooley 2023-02-08 21:49 ` Conor Dooley 2023-02-13 15:25 ` Sunil V L 2023-02-13 15:25 ` Sunil V L 2023-01-30 18:22 ` [PATCH 12/24] RISC-V: ACPI: smpboot: Create wrapper smp_setup() Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-08 21:34 ` Conor Dooley 2023-02-08 21:34 ` Conor Dooley 2023-01-30 18:22 ` [PATCH 13/24] RISC-V: ACPI: smpboot: Add ACPI support in smp_setup() Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-08 22:10 ` Conor Dooley 2023-02-08 22:10 ` Conor Dooley 2023-02-13 15:27 ` Sunil V L 2023-02-13 15:27 ` Sunil V L 2023-01-30 18:22 ` [PATCH 14/24] RISC-V: ACPI: smpboot: Add function to retrieve the hartid Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 20:30 ` Conor Dooley [this message] 2023-02-09 20:30 ` Conor Dooley 2023-02-13 17:00 ` Sunil V L 2023-02-13 17:00 ` Sunil V L 2023-01-30 18:22 ` [PATCH 15/24] clocksource/timer-riscv: Refactor riscv_timer_init_dt() Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 20:54 ` Conor Dooley 2023-02-09 20:54 ` Conor Dooley 2023-02-13 17:22 ` Sunil V L 2023-02-13 17:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 16/24] RISC-V: ACPI: clocksource/timer-riscv: Add ACPI support Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 20:58 ` Conor Dooley 2023-02-09 20:58 ` Conor Dooley 2023-02-13 17:39 ` Sunil V L 2023-02-13 17:39 ` Sunil V L 2023-01-30 18:22 ` [PATCH 17/24] ACPI: RISC-V: drivers/acpi: Add RHCT related code Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 18/24] RISC-V: ACPI: time.c: Add ACPI support for time_init() Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 18:22 ` [PATCH 19/24] RISC-V: ACPI: cpufeature: Add ACPI support in riscv_fill_hwcap() Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 21:47 ` Conor Dooley 2023-02-09 21:47 ` Conor Dooley 2023-02-13 17:51 ` Sunil V L 2023-02-13 17:51 ` Sunil V L 2023-01-30 18:22 ` [PATCH 20/24] RISC-V: ACPI: cpu: Enable cpuinfo for ACPI systems Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 21:13 ` Conor Dooley 2023-02-09 21:13 ` Conor Dooley 2023-02-13 17:42 ` Sunil V L 2023-02-13 17:42 ` Sunil V L 2023-01-30 18:22 ` [PATCH 21/24] RISC-V: ACPI: Add ACPI initialization in setup_arch() Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 21:53 ` Conor Dooley 2023-02-09 21:53 ` Conor Dooley 2023-02-13 17:52 ` Sunil V L 2023-02-13 17:52 ` Sunil V L 2023-01-30 18:22 ` [PATCH 22/24] RISC-V: ACPI: Enable ACPI in defconfig Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-01-30 23:47 ` Conor Dooley 2023-01-30 23:47 ` Conor Dooley 2023-01-31 8:41 ` Sunil V L 2023-01-31 8:41 ` Sunil V L 2023-01-30 18:22 ` [PATCH 23/24] MAINTAINERS: Add entry for drivers/acpi/riscv Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 21:54 ` Conor Dooley 2023-02-09 21:54 ` Conor Dooley 2023-02-13 17:53 ` Sunil V L 2023-02-13 17:53 ` Sunil V L 2023-01-30 18:22 ` [PATCH 24/24] Documentation/kernel-parameters.txt: Add RISC-V for ACPI parameter Sunil V L 2023-01-30 18:22 ` Sunil V L 2023-02-09 2:02 ` Bagas Sanjaya 2023-02-09 2:02 ` Bagas Sanjaya 2023-02-13 15:29 ` Sunil V L 2023-02-13 15:29 ` Sunil V L 2023-01-30 19:11 ` [PATCH 00/24] Add basic ACPI support for RISC-V Rafael J. Wysocki 2023-01-30 19:11 ` Rafael J. Wysocki 2023-02-08 18:28 ` Conor Dooley 2023-02-08 18:28 ` Conor Dooley 2023-02-08 18:50 ` Conor Dooley 2023-02-08 18:50 ` Conor Dooley 2023-02-13 4:51 ` Sunil V L 2023-02-13 4:51 ` Sunil V L
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=Y+VX8EclC1cE2fdI@spud \ --to=conor@kernel.org \ --cc=ajones@ventanamicro.com \ --cc=aou@eecs.berkeley.edu \ --cc=apatel@ventanamicro.com \ --cc=atishp@rivosinc.com \ --cc=corbet@lwn.net \ --cc=daniel.lezcano@linaro.org \ --cc=lenb@kernel.org \ --cc=linux-acpi@vger.kernel.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=maz@kernel.org \ --cc=palmer@dabbelt.com \ --cc=rafael@kernel.org \ --cc=sunilvl@ventanamicro.com \ --cc=tglx@linutronix.de \ /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.