* [PATCH] RISC-V: Add preprocessor directives for boot_sec_cpu
@ 2018-08-02 15:22 Zong Li
2018-08-02 18:13 ` Atish Patra
0 siblings, 1 reply; 3+ messages in thread
From: Zong Li @ 2018-08-02 15:22 UTC (permalink / raw)
To: palmer, aou, hch, linux-riscv, linux-kernel, greentime; +Cc: Zong Li
The boot_sec_cpu is for hutplug CPU on SMP system. It should
be conditional compiling. Otherwise, it causes undefined reference
to `smp_callin' when compiling uniprocessor kernel.
Signed-off-by: Zong Li <zong@andestech.com>
---
arch/riscv/kernel/head.S | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/riscv/kernel/head.S b/arch/riscv/kernel/head.S
index 11066d5..9b6f601 100644
--- a/arch/riscv/kernel/head.S
+++ b/arch/riscv/kernel/head.S
@@ -154,6 +154,7 @@ END(_start)
.section .text
.global boot_sec_cpu
+#ifdef CONFIG_HOTPLUG_CPU
boot_sec_cpu:
/* clear all pending flags */
csrw sip, zero
@@ -162,6 +163,7 @@ boot_sec_cpu:
fence
tail smp_callin
+#endif
__PAGE_ALIGNED_BSS
/* Empty zero page */
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] RISC-V: Add preprocessor directives for boot_sec_cpu
2018-08-02 15:22 [PATCH] RISC-V: Add preprocessor directives for boot_sec_cpu Zong Li
@ 2018-08-02 18:13 ` Atish Patra
2018-08-03 0:27 ` Zong Li
0 siblings, 1 reply; 3+ messages in thread
From: Atish Patra @ 2018-08-02 18:13 UTC (permalink / raw)
To: Zong Li, palmer, aou, hch, linux-riscv, linux-kernel, greentime
On 8/2/18 8:23 AM, Zong Li wrote:
> The boot_sec_cpu is for hutplug CPU on SMP system. It should
> be conditional compiling. Otherwise, it causes undefined reference
> to `smp_callin' when compiling uniprocessor kernel.
>
> Signed-off-by: Zong Li <zong@andestech.com>
> ---
> arch/riscv/kernel/head.S | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/riscv/kernel/head.S b/arch/riscv/kernel/head.S
> index 11066d5..9b6f601 100644
> --- a/arch/riscv/kernel/head.S
> +++ b/arch/riscv/kernel/head.S
> @@ -154,6 +154,7 @@ END(_start)
> .section .text
> .global boot_sec_cpu
>
> +#ifdef CONFIG_HOTPLUG_CPU
> boot_sec_cpu:
> /* clear all pending flags */
> csrw sip, zero
> @@ -162,6 +163,7 @@ boot_sec_cpu:
> fence
>
> tail smp_callin
> +#endif
>
> __PAGE_ALIGNED_BSS
> /* Empty zero page */
>
Thanks for the fix. If you don't mind, I would like to merge it on top
of my cpu hotplug patch and resubmit together to avoid confusion.
Regards,
Atish
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] RISC-V: Add preprocessor directives for boot_sec_cpu
2018-08-02 18:13 ` Atish Patra
@ 2018-08-03 0:27 ` Zong Li
0 siblings, 0 replies; 3+ messages in thread
From: Zong Li @ 2018-08-03 0:27 UTC (permalink / raw)
To: atish.patra
Cc: Zong Li, Palmer Dabbelt, aou, hch, linux-riscv,
Linux Kernel Mailing List, greentime
Atish Patra <atish.patra@wdc.com> 於 2018年8月3日 週五 上午2:14寫道:
>
> On 8/2/18 8:23 AM, Zong Li wrote:
> > The boot_sec_cpu is for hutplug CPU on SMP system. It should
> > be conditional compiling. Otherwise, it causes undefined reference
> > to `smp_callin' when compiling uniprocessor kernel.
> >
> > Signed-off-by: Zong Li <zong@andestech.com>
> > ---
> > arch/riscv/kernel/head.S | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/riscv/kernel/head.S b/arch/riscv/kernel/head.S
> > index 11066d5..9b6f601 100644
> > --- a/arch/riscv/kernel/head.S
> > +++ b/arch/riscv/kernel/head.S
> > @@ -154,6 +154,7 @@ END(_start)
> > .section .text
> > .global boot_sec_cpu
> >
> > +#ifdef CONFIG_HOTPLUG_CPU
> > boot_sec_cpu:
> > /* clear all pending flags */
> > csrw sip, zero
> > @@ -162,6 +163,7 @@ boot_sec_cpu:
> > fence
> >
> > tail smp_callin
> > +#endif
> >
> > __PAGE_ALIGNED_BSS
> > /* Empty zero page */
> >
>
> Thanks for the fix. If you don't mind, I would like to merge it on top
> of my cpu hotplug patch and resubmit together to avoid confusion.
>
OK, just take it:)
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-08-03 0:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-02 15:22 [PATCH] RISC-V: Add preprocessor directives for boot_sec_cpu Zong Li
2018-08-02 18:13 ` Atish Patra
2018-08-03 0:27 ` Zong Li
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).