* [PATCH V2] MIPS: CPU#0 is not hotpluggable
@ 2020-07-16 10:40 Huacai Chen
2020-07-25 6:37 ` Huacai Chen
2020-07-26 8:25 ` Thomas Bogendoerfer
0 siblings, 2 replies; 11+ messages in thread
From: Huacai Chen @ 2020-07-16 10:40 UTC (permalink / raw)
To: Thomas Bogendoerfer
Cc: linux-mips, Fuxin Zhang, Zhangjin Wu, Huacai Chen, Jiaxun Yang,
Huacai Chen, stable
Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
/system/cpu/cpu0/online which confuses some user-space tools.
Cc: stable@vger.kernel.org
Signed-off-by: Huacai Chen <chenhc@lemote.com>
---
arch/mips/kernel/topology.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/mips/kernel/topology.c b/arch/mips/kernel/topology.c
index cd3e1f8..08ad637 100644
--- a/arch/mips/kernel/topology.c
+++ b/arch/mips/kernel/topology.c
@@ -20,7 +20,7 @@ static int __init topology_init(void)
for_each_present_cpu(i) {
struct cpu *c = &per_cpu(cpu_devices, i);
- c->hotpluggable = 1;
+ c->hotpluggable = !!i;
ret = register_cpu(c, i);
if (ret)
printk(KERN_WARNING "topology_init: register_cpu %d "
--
2.7.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-16 10:40 [PATCH V2] MIPS: CPU#0 is not hotpluggable Huacai Chen
@ 2020-07-25 6:37 ` Huacai Chen
2020-07-25 6:49 ` Greg KH
2020-07-26 8:25 ` Thomas Bogendoerfer
2020-07-26 8:25 ` Thomas Bogendoerfer
1 sibling, 2 replies; 11+ messages in thread
From: Huacai Chen @ 2020-07-25 6:37 UTC (permalink / raw)
To: Thomas Bogendoerfer
Cc: open list:MIPS, Fuxin Zhang, Zhangjin Wu, Jiaxun Yang, stable
Hi, Thomas,
What do you think about this patch? Other archs also do the same thing
except those support hotplug CPU#0.
grep hotpluggable arch -rwI
arch/riscv/kernel/setup.c: cpu->hotpluggable = cpu_has_hotplug(i);
arch/powerpc/kernel/sysfs.c: BUG_ON(!c->hotpluggable);
arch/powerpc/kernel/sysfs.c: c->hotpluggable = 1;
arch/powerpc/kernel/sysfs.c: if (cpu_online(cpu) || c->hotpluggable) {
arch/arm/kernel/setup.c: cpuinfo->cpu.hotpluggable =
platform_can_hotplug_cpu(cpu);
arch/sh/kernel/topology.c: c->hotpluggable = 1;
arch/ia64/kernel/topology.c: * CPEI target, then it is hotpluggable
arch/ia64/kernel/topology.c: sysfs_cpus[num].cpu.hotpluggable = 1;
arch/xtensa/kernel/setup.c: cpu->hotpluggable = !!i;
arch/s390/kernel/smp.c: c->hotpluggable = 1;
arch/mips/kernel/topology.c: c->hotpluggable = 1;
arch/arm64/kernel/cpuinfo.c: * In case the boot CPU is hotpluggable,
we record its initial state and
arch/arm64/kernel/setup.c: cpu->hotpluggable = cpu_can_disable(i);
arch/x86/kernel/topology.c: per_cpu(cpu_devices,
num).cpu.hotpluggable = 1;
On Thu, Jul 16, 2020 at 6:38 PM Huacai Chen <chenhc@lemote.com> wrote:
>
> Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
> /system/cpu/cpu0/online which confuses some user-space tools.
>
> Cc: stable@vger.kernel.org
> Signed-off-by: Huacai Chen <chenhc@lemote.com>
> ---
> arch/mips/kernel/topology.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/kernel/topology.c b/arch/mips/kernel/topology.c
> index cd3e1f8..08ad637 100644
> --- a/arch/mips/kernel/topology.c
> +++ b/arch/mips/kernel/topology.c
> @@ -20,7 +20,7 @@ static int __init topology_init(void)
> for_each_present_cpu(i) {
> struct cpu *c = &per_cpu(cpu_devices, i);
>
> - c->hotpluggable = 1;
> + c->hotpluggable = !!i;
> ret = register_cpu(c, i);
> if (ret)
> printk(KERN_WARNING "topology_init: register_cpu %d "
> --
> 2.7.0
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-25 6:37 ` Huacai Chen
@ 2020-07-25 6:49 ` Greg KH
2020-07-25 6:57 ` Huacai Chen
2020-07-26 8:25 ` Thomas Bogendoerfer
1 sibling, 1 reply; 11+ messages in thread
From: Greg KH @ 2020-07-25 6:49 UTC (permalink / raw)
To: Huacai Chen
Cc: Thomas Bogendoerfer, open list:MIPS, Fuxin Zhang, Zhangjin Wu,
Jiaxun Yang, stable
On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
> Hi, Thomas,
>
> What do you think about this patch? Other archs also do the same thing
> except those support hotplug CPU#0.
>
> grep hotpluggable arch -rwI
> arch/riscv/kernel/setup.c: cpu->hotpluggable = cpu_has_hotplug(i);
> arch/powerpc/kernel/sysfs.c: BUG_ON(!c->hotpluggable);
> arch/powerpc/kernel/sysfs.c: c->hotpluggable = 1;
> arch/powerpc/kernel/sysfs.c: if (cpu_online(cpu) || c->hotpluggable) {
> arch/arm/kernel/setup.c: cpuinfo->cpu.hotpluggable =
> platform_can_hotplug_cpu(cpu);
> arch/sh/kernel/topology.c: c->hotpluggable = 1;
> arch/ia64/kernel/topology.c: * CPEI target, then it is hotpluggable
> arch/ia64/kernel/topology.c: sysfs_cpus[num].cpu.hotpluggable = 1;
> arch/xtensa/kernel/setup.c: cpu->hotpluggable = !!i;
> arch/s390/kernel/smp.c: c->hotpluggable = 1;
> arch/mips/kernel/topology.c: c->hotpluggable = 1;
> arch/arm64/kernel/cpuinfo.c: * In case the boot CPU is hotpluggable,
> we record its initial state and
> arch/arm64/kernel/setup.c: cpu->hotpluggable = cpu_can_disable(i);
> arch/x86/kernel/topology.c: per_cpu(cpu_devices,
> num).cpu.hotpluggable = 1;
>
> On Thu, Jul 16, 2020 at 6:38 PM Huacai Chen <chenhc@lemote.com> wrote:
> >
> > Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
> > /system/cpu/cpu0/online which confuses some user-space tools.
What userspace tools are confused by this? They should be able to
handle a cpu not being able to be removed, right?
> >
> > Cc: stable@vger.kernel.org
> > Signed-off-by: Huacai Chen <chenhc@lemote.com>
> > ---
> > arch/mips/kernel/topology.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/mips/kernel/topology.c b/arch/mips/kernel/topology.c
> > index cd3e1f8..08ad637 100644
> > --- a/arch/mips/kernel/topology.c
> > +++ b/arch/mips/kernel/topology.c
> > @@ -20,7 +20,7 @@ static int __init topology_init(void)
> > for_each_present_cpu(i) {
> > struct cpu *c = &per_cpu(cpu_devices, i);
> >
> > - c->hotpluggable = 1;
> > + c->hotpluggable = !!i;
Seems to be the same as what xtensa did, so it's probably not a big
deal.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-25 6:49 ` Greg KH
@ 2020-07-25 6:57 ` Huacai Chen
2020-07-25 7:45 ` Greg KH
0 siblings, 1 reply; 11+ messages in thread
From: Huacai Chen @ 2020-07-25 6:57 UTC (permalink / raw)
To: Greg KH
Cc: Thomas Bogendoerfer, open list:MIPS, Fuxin Zhang, Zhangjin Wu,
Jiaxun Yang, stable
Hi Greg,
On Sat, Jul 25, 2020 at 2:49 PM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
> > Hi, Thomas,
> >
> > What do you think about this patch? Other archs also do the same thing
> > except those support hotplug CPU#0.
> >
> > grep hotpluggable arch -rwI
> > arch/riscv/kernel/setup.c: cpu->hotpluggable = cpu_has_hotplug(i);
> > arch/powerpc/kernel/sysfs.c: BUG_ON(!c->hotpluggable);
> > arch/powerpc/kernel/sysfs.c: c->hotpluggable = 1;
> > arch/powerpc/kernel/sysfs.c: if (cpu_online(cpu) || c->hotpluggable) {
> > arch/arm/kernel/setup.c: cpuinfo->cpu.hotpluggable =
> > platform_can_hotplug_cpu(cpu);
> > arch/sh/kernel/topology.c: c->hotpluggable = 1;
> > arch/ia64/kernel/topology.c: * CPEI target, then it is hotpluggable
> > arch/ia64/kernel/topology.c: sysfs_cpus[num].cpu.hotpluggable = 1;
> > arch/xtensa/kernel/setup.c: cpu->hotpluggable = !!i;
> > arch/s390/kernel/smp.c: c->hotpluggable = 1;
> > arch/mips/kernel/topology.c: c->hotpluggable = 1;
> > arch/arm64/kernel/cpuinfo.c: * In case the boot CPU is hotpluggable,
> > we record its initial state and
> > arch/arm64/kernel/setup.c: cpu->hotpluggable = cpu_can_disable(i);
> > arch/x86/kernel/topology.c: per_cpu(cpu_devices,
> > num).cpu.hotpluggable = 1;
> >
> > On Thu, Jul 16, 2020 at 6:38 PM Huacai Chen <chenhc@lemote.com> wrote:
> > >
> > > Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
> > > /system/cpu/cpu0/online which confuses some user-space tools.
>
> What userspace tools are confused by this? They should be able to
> handle a cpu not being able to be removed, right?
It causes ltp's "hotplug" test fails, and ltp considers CPUs with a
"online" node be hotpluggable.
>
>
> > >
> > > Cc: stable@vger.kernel.org
> > > Signed-off-by: Huacai Chen <chenhc@lemote.com>
> > > ---
> > > arch/mips/kernel/topology.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/arch/mips/kernel/topology.c b/arch/mips/kernel/topology.c
> > > index cd3e1f8..08ad637 100644
> > > --- a/arch/mips/kernel/topology.c
> > > +++ b/arch/mips/kernel/topology.c
> > > @@ -20,7 +20,7 @@ static int __init topology_init(void)
> > > for_each_present_cpu(i) {
> > > struct cpu *c = &per_cpu(cpu_devices, i);
> > >
> > > - c->hotpluggable = 1;
> > > + c->hotpluggable = !!i;
>
> Seems to be the same as what xtensa did, so it's probably not a big
> deal.
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-25 6:57 ` Huacai Chen
@ 2020-07-25 7:45 ` Greg KH
2020-07-25 8:29 ` Huacai Chen
0 siblings, 1 reply; 11+ messages in thread
From: Greg KH @ 2020-07-25 7:45 UTC (permalink / raw)
To: Huacai Chen
Cc: Thomas Bogendoerfer, open list:MIPS, Fuxin Zhang, Zhangjin Wu,
Jiaxun Yang, stable
On Sat, Jul 25, 2020 at 02:57:31PM +0800, Huacai Chen wrote:
> Hi Greg,
>
> On Sat, Jul 25, 2020 at 2:49 PM Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
> > > Hi, Thomas,
> > >
> > > What do you think about this patch? Other archs also do the same thing
> > > except those support hotplug CPU#0.
> > >
> > > grep hotpluggable arch -rwI
> > > arch/riscv/kernel/setup.c: cpu->hotpluggable = cpu_has_hotplug(i);
> > > arch/powerpc/kernel/sysfs.c: BUG_ON(!c->hotpluggable);
> > > arch/powerpc/kernel/sysfs.c: c->hotpluggable = 1;
> > > arch/powerpc/kernel/sysfs.c: if (cpu_online(cpu) || c->hotpluggable) {
> > > arch/arm/kernel/setup.c: cpuinfo->cpu.hotpluggable =
> > > platform_can_hotplug_cpu(cpu);
> > > arch/sh/kernel/topology.c: c->hotpluggable = 1;
> > > arch/ia64/kernel/topology.c: * CPEI target, then it is hotpluggable
> > > arch/ia64/kernel/topology.c: sysfs_cpus[num].cpu.hotpluggable = 1;
> > > arch/xtensa/kernel/setup.c: cpu->hotpluggable = !!i;
> > > arch/s390/kernel/smp.c: c->hotpluggable = 1;
> > > arch/mips/kernel/topology.c: c->hotpluggable = 1;
> > > arch/arm64/kernel/cpuinfo.c: * In case the boot CPU is hotpluggable,
> > > we record its initial state and
> > > arch/arm64/kernel/setup.c: cpu->hotpluggable = cpu_can_disable(i);
> > > arch/x86/kernel/topology.c: per_cpu(cpu_devices,
> > > num).cpu.hotpluggable = 1;
> > >
> > > On Thu, Jul 16, 2020 at 6:38 PM Huacai Chen <chenhc@lemote.com> wrote:
> > > >
> > > > Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
> > > > /system/cpu/cpu0/online which confuses some user-space tools.
> >
> > What userspace tools are confused by this? They should be able to
> > handle a cpu not being able to be removed, right?
> It causes ltp's "hotplug" test fails, and ltp considers CPUs with a
> "online" node be hotpluggable.
Is that always true?
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-25 7:45 ` Greg KH
@ 2020-07-25 8:29 ` Huacai Chen
2020-07-25 8:57 ` Greg KH
0 siblings, 1 reply; 11+ messages in thread
From: Huacai Chen @ 2020-07-25 8:29 UTC (permalink / raw)
To: Greg KH
Cc: Thomas Bogendoerfer, open list:MIPS, Fuxin Zhang, Zhangjin Wu,
Jiaxun Yang, stable
Hi, Greg,
On Sat, Jul 25, 2020 at 3:45 PM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Sat, Jul 25, 2020 at 02:57:31PM +0800, Huacai Chen wrote:
> > Hi Greg,
> >
> > On Sat, Jul 25, 2020 at 2:49 PM Greg KH <gregkh@linuxfoundation.org> wrote:
> > >
> > > On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
> > > > Hi, Thomas,
> > > >
> > > > What do you think about this patch? Other archs also do the same thing
> > > > except those support hotplug CPU#0.
> > > >
> > > > grep hotpluggable arch -rwI
> > > > arch/riscv/kernel/setup.c: cpu->hotpluggable = cpu_has_hotplug(i);
> > > > arch/powerpc/kernel/sysfs.c: BUG_ON(!c->hotpluggable);
> > > > arch/powerpc/kernel/sysfs.c: c->hotpluggable = 1;
> > > > arch/powerpc/kernel/sysfs.c: if (cpu_online(cpu) || c->hotpluggable) {
> > > > arch/arm/kernel/setup.c: cpuinfo->cpu.hotpluggable =
> > > > platform_can_hotplug_cpu(cpu);
> > > > arch/sh/kernel/topology.c: c->hotpluggable = 1;
> > > > arch/ia64/kernel/topology.c: * CPEI target, then it is hotpluggable
> > > > arch/ia64/kernel/topology.c: sysfs_cpus[num].cpu.hotpluggable = 1;
> > > > arch/xtensa/kernel/setup.c: cpu->hotpluggable = !!i;
> > > > arch/s390/kernel/smp.c: c->hotpluggable = 1;
> > > > arch/mips/kernel/topology.c: c->hotpluggable = 1;
> > > > arch/arm64/kernel/cpuinfo.c: * In case the boot CPU is hotpluggable,
> > > > we record its initial state and
> > > > arch/arm64/kernel/setup.c: cpu->hotpluggable = cpu_can_disable(i);
> > > > arch/x86/kernel/topology.c: per_cpu(cpu_devices,
> > > > num).cpu.hotpluggable = 1;
> > > >
> > > > On Thu, Jul 16, 2020 at 6:38 PM Huacai Chen <chenhc@lemote.com> wrote:
> > > > >
> > > > > Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
> > > > > /system/cpu/cpu0/online which confuses some user-space tools.
> > >
> > > What userspace tools are confused by this? They should be able to
> > > handle a cpu not being able to be removed, right?
> > It causes ltp's "hotplug" test fails, and ltp considers CPUs with a
> > "online" node be hotpluggable.
>
> Is that always true?
Yes, someone who meet the same problem report a bug to LTP, and LTP
maintainer said that this should be fixed in kernel.
Huacai
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-25 8:29 ` Huacai Chen
@ 2020-07-25 8:57 ` Greg KH
2020-07-26 2:17 ` Jiaxun Yang
0 siblings, 1 reply; 11+ messages in thread
From: Greg KH @ 2020-07-25 8:57 UTC (permalink / raw)
To: Huacai Chen
Cc: Thomas Bogendoerfer, open list:MIPS, Fuxin Zhang, Zhangjin Wu,
Jiaxun Yang, stable
On Sat, Jul 25, 2020 at 04:29:28PM +0800, Huacai Chen wrote:
> Hi, Greg,
>
> On Sat, Jul 25, 2020 at 3:45 PM Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > On Sat, Jul 25, 2020 at 02:57:31PM +0800, Huacai Chen wrote:
> > > Hi Greg,
> > >
> > > On Sat, Jul 25, 2020 at 2:49 PM Greg KH <gregkh@linuxfoundation.org> wrote:
> > > >
> > > > On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
> > > > > Hi, Thomas,
> > > > >
> > > > > What do you think about this patch? Other archs also do the same thing
> > > > > except those support hotplug CPU#0.
> > > > >
> > > > > grep hotpluggable arch -rwI
> > > > > arch/riscv/kernel/setup.c: cpu->hotpluggable = cpu_has_hotplug(i);
> > > > > arch/powerpc/kernel/sysfs.c: BUG_ON(!c->hotpluggable);
> > > > > arch/powerpc/kernel/sysfs.c: c->hotpluggable = 1;
> > > > > arch/powerpc/kernel/sysfs.c: if (cpu_online(cpu) || c->hotpluggable) {
> > > > > arch/arm/kernel/setup.c: cpuinfo->cpu.hotpluggable =
> > > > > platform_can_hotplug_cpu(cpu);
> > > > > arch/sh/kernel/topology.c: c->hotpluggable = 1;
> > > > > arch/ia64/kernel/topology.c: * CPEI target, then it is hotpluggable
> > > > > arch/ia64/kernel/topology.c: sysfs_cpus[num].cpu.hotpluggable = 1;
> > > > > arch/xtensa/kernel/setup.c: cpu->hotpluggable = !!i;
> > > > > arch/s390/kernel/smp.c: c->hotpluggable = 1;
> > > > > arch/mips/kernel/topology.c: c->hotpluggable = 1;
> > > > > arch/arm64/kernel/cpuinfo.c: * In case the boot CPU is hotpluggable,
> > > > > we record its initial state and
> > > > > arch/arm64/kernel/setup.c: cpu->hotpluggable = cpu_can_disable(i);
> > > > > arch/x86/kernel/topology.c: per_cpu(cpu_devices,
> > > > > num).cpu.hotpluggable = 1;
> > > > >
> > > > > On Thu, Jul 16, 2020 at 6:38 PM Huacai Chen <chenhc@lemote.com> wrote:
> > > > > >
> > > > > > Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
> > > > > > /system/cpu/cpu0/online which confuses some user-space tools.
> > > >
> > > > What userspace tools are confused by this? They should be able to
> > > > handle a cpu not being able to be removed, right?
> > > It causes ltp's "hotplug" test fails, and ltp considers CPUs with a
> > > "online" node be hotpluggable.
> >
> > Is that always true?
> Yes, someone who meet the same problem report a bug to LTP, and LTP
> maintainer said that this should be fixed in kernel.
So the action _always_ has to succeed and can never return an error?
That feels wrong even for normal systems.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-25 8:57 ` Greg KH
@ 2020-07-26 2:17 ` Jiaxun Yang
0 siblings, 0 replies; 11+ messages in thread
From: Jiaxun Yang @ 2020-07-26 2:17 UTC (permalink / raw)
To: Greg KH, Huacai Chen
Cc: Thomas Bogendoerfer, open list:MIPS, Fuxin Zhang, Zhangjin Wu, stable
在 2020/7/25 下午4:57, Greg KH 写道:
> On Sat, Jul 25, 2020 at 04:29:28PM +0800, Huacai Chen wrote:
>> Hi, Greg,
>>
>> On Sat, Jul 25, 2020 at 3:45 PM Greg KH <gregkh@linuxfoundation.org> wrote:
>>> On Sat, Jul 25, 2020 at 02:57:31PM +0800, Huacai Chen wrote:
>>>> Hi Greg,
>>>>
>>>> On Sat, Jul 25, 2020 at 2:49 PM Greg KH <gregkh@linuxfoundation.org> wrote:
>>>>> On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
>>>>>> Hi, Thomas,
>>>>>>
>>>>>> What do you think about this patch? Other archs also do the same thing
>>>>>> except those support hotplug CPU#0.
>>>>>>
>>>>>> grep hotpluggable arch -rwI
>>>>>> arch/riscv/kernel/setup.c: cpu->hotpluggable = cpu_has_hotplug(i);
>>>>>> arch/powerpc/kernel/sysfs.c: BUG_ON(!c->hotpluggable);
>>>>>> arch/powerpc/kernel/sysfs.c: c->hotpluggable = 1;
>>>>>> arch/powerpc/kernel/sysfs.c: if (cpu_online(cpu) || c->hotpluggable) {
>>>>>> arch/arm/kernel/setup.c: cpuinfo->cpu.hotpluggable =
>>>>>> platform_can_hotplug_cpu(cpu);
>>>>>> arch/sh/kernel/topology.c: c->hotpluggable = 1;
>>>>>> arch/ia64/kernel/topology.c: * CPEI target, then it is hotpluggable
>>>>>> arch/ia64/kernel/topology.c: sysfs_cpus[num].cpu.hotpluggable = 1;
>>>>>> arch/xtensa/kernel/setup.c: cpu->hotpluggable = !!i;
>>>>>> arch/s390/kernel/smp.c: c->hotpluggable = 1;
>>>>>> arch/mips/kernel/topology.c: c->hotpluggable = 1;
>>>>>> arch/arm64/kernel/cpuinfo.c: * In case the boot CPU is hotpluggable,
>>>>>> we record its initial state and
>>>>>> arch/arm64/kernel/setup.c: cpu->hotpluggable = cpu_can_disable(i);
>>>>>> arch/x86/kernel/topology.c: per_cpu(cpu_devices,
>>>>>> num).cpu.hotpluggable = 1;
>>>>>>
>>>>>> On Thu, Jul 16, 2020 at 6:38 PM Huacai Chen <chenhc@lemote.com> wrote:
>>>>>>> Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
>>>>>>> /system/cpu/cpu0/online which confuses some user-space tools.
>>>>> What userspace tools are confused by this? They should be able to
>>>>> handle a cpu not being able to be removed, right?
>>>> It causes ltp's "hotplug" test fails, and ltp considers CPUs with a
>>>> "online" node be hotpluggable.
>>> Is that always true?
>> Yes, someone who meet the same problem report a bug to LTP, and LTP
>> maintainer said that this should be fixed in kernel.
> So the action _always_ has to succeed and can never return an error?
> That feels wrong even for normal systems.
If that returns error then it means kernel's hotplug function is not
working as
expected. Thus, LTP as a testsuit will consider the error as a indicator
of malfunction
kernel.
Thanks
- Jiaxun
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-25 6:37 ` Huacai Chen
2020-07-25 6:49 ` Greg KH
@ 2020-07-26 8:25 ` Thomas Bogendoerfer
2020-07-26 11:30 ` Jiaxun Yang
1 sibling, 1 reply; 11+ messages in thread
From: Thomas Bogendoerfer @ 2020-07-26 8:25 UTC (permalink / raw)
To: Huacai Chen; +Cc: open list:MIPS, Fuxin Zhang, Zhangjin Wu, Jiaxun Yang, stable
On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
> Hi, Thomas,
>
> What do you think about this patch? Other archs also do the same thing
> except those support hotplug CPU#0.
I'm ok with the patch, I'm just wondering if this is a hardware or
software limitation. If it's the latter, what needs to be done to
support it ?
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-16 10:40 [PATCH V2] MIPS: CPU#0 is not hotpluggable Huacai Chen
2020-07-25 6:37 ` Huacai Chen
@ 2020-07-26 8:25 ` Thomas Bogendoerfer
1 sibling, 0 replies; 11+ messages in thread
From: Thomas Bogendoerfer @ 2020-07-26 8:25 UTC (permalink / raw)
To: Huacai Chen
Cc: linux-mips, Fuxin Zhang, Zhangjin Wu, Huacai Chen, Jiaxun Yang, stable
On Thu, Jul 16, 2020 at 06:40:23PM +0800, Huacai Chen wrote:
> Now CPU#0 is not hotpluggable on MIPS, so prevent to create /sys/devices
> /system/cpu/cpu0/online which confuses some user-space tools.
>
> Cc: stable@vger.kernel.org
> Signed-off-by: Huacai Chen <chenhc@lemote.com>
> ---
> arch/mips/kernel/topology.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
applied to mips-next.
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V2] MIPS: CPU#0 is not hotpluggable
2020-07-26 8:25 ` Thomas Bogendoerfer
@ 2020-07-26 11:30 ` Jiaxun Yang
0 siblings, 0 replies; 11+ messages in thread
From: Jiaxun Yang @ 2020-07-26 11:30 UTC (permalink / raw)
To: Thomas Bogendoerfer, Huacai Chen
Cc: open list:MIPS, Fuxin Zhang, Zhangjin Wu, stable
在 2020/7/26 下午4:25, Thomas Bogendoerfer 写道:
> On Sat, Jul 25, 2020 at 02:37:52PM +0800, Huacai Chen wrote:
>> Hi, Thomas,
>>
>> What do you think about this patch? Other archs also do the same thing
>> except those support hotplug CPU#0.
> I'm ok with the patch, I'm just wondering if this is a hardware or
> software limitation. If it's the latter, what needs to be done to
> support it ?
It should be a software limitation, x86 have already deal with that [1].
But there is no reason to spend extra effort on CPU0 hotplug. I don't
think any user is expecting this feature.
Thanks.
[1]:
https://events.static.linuxfound.org/sites/events/files/lcjpcojp13_fenghua.pdf
- Jiaxun
>
> Thomas.
>
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2020-07-26 11:31 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-16 10:40 [PATCH V2] MIPS: CPU#0 is not hotpluggable Huacai Chen
2020-07-25 6:37 ` Huacai Chen
2020-07-25 6:49 ` Greg KH
2020-07-25 6:57 ` Huacai Chen
2020-07-25 7:45 ` Greg KH
2020-07-25 8:29 ` Huacai Chen
2020-07-25 8:57 ` Greg KH
2020-07-26 2:17 ` Jiaxun Yang
2020-07-26 8:25 ` Thomas Bogendoerfer
2020-07-26 11:30 ` Jiaxun Yang
2020-07-26 8:25 ` Thomas Bogendoerfer
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.